Skip to content

Napraw pobieranie pliku .sha256 w instrukcji - #12

Closed
ldziedziul wants to merge 1 commit into
kprzybys:mainfrom
ldziedziul:fix/macos-install-curl
Closed

ldziedziul wants to merge 1 commit into
kprzybys:mainfrom
ldziedziul:fix/macos-install-curl

Conversation

@ldziedziul

Copy link
Copy Markdown

Fixes #11

curl -LO url1 url2 - opcja -O dotyczy tylko pierwszego URL-a, więc .sha256 z rozwinięcia {,.sha256} trafiał na stdout zamiast na dysk. W efekcie shasum -c kończył się błędem, tar się nie wykonywał, a install nie znajdował binarki.

Zmiana: -LO → -L --remote-name-all w docs/INSTALL.md (macOS) i README.md (szybki start, Linux - ten sam problem).

Sprawdzone na macOS arm64 (0.1.4): ksef-0.1.4-osx-arm64.tar.gz: OK, archiwum rozpakowane, binarka na miejscu.

Opcja -O w curl dotyczy tylko jednego, następującego po niej URL-a.
Rozwinięcie {,.sha256} daje dwa URL-e, więc plik .sha256 trafiał na
stdout zamiast na dysk. Przez to shasum/sha256sum kończył się błędem,
tar się nie wykonywał, a install nie znajdował binarki.

--remote-name-all zapisuje wszystkie podane URL-e.

Fixes kprzybys#11
@ldziedziul
ldziedziul marked this pull request as ready for review November 2, 2026 10:41
@kprzybys

kprzybys commented Nov 2, 2026

Copy link
Copy Markdown
Owner

Dzięki! Twoja poprawka była pierwsza i dokładnie trafiła w przyczynę — przepraszam, że przeoczyłem ten PR i zrobiłem tę samą zmianę osobno w #13 (jest już w main, więc ten PR ma teraz konflikt).

#13 zawiera Twoją zmianę (curl -L --remote-name-all w README i INSTALL) oraz dwie rzeczy wyłapane przy testach: bloki poleceń bez komentarzy, bo zsh w domyślnych ustawieniach macOS bierze # za polecenie i gubi zmienne v/a, oraz mkdir -p ~/.config/ksef przed ksef cert new (na czystym systemie katalogu nie ma).

Zamykam jako zastąpiony przez #13; w notatkach następnego wydania będzie wzmianka o Twoim zgłoszeniu i poprawce. Jeszcze raz dziękuję — zarówno za zgłoszenie, jak i za PR.

@kprzybys kprzybys closed this Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Instalacja MacOs nie działa

2 participants