bootstrap uses arithmetic not available in plain sh #204

Closed
opened 2026-01-29 20:39:47 +00:00 by claunia · 1 comment
Owner

Originally created by @Sp1l on GitHub (Nov 29, 2017).

0ad94eed00/bootstrap (L1)

Even though the shebang indicates this uses plain shell, it uses arithmetic not available in plain shell.
Hard depending on bash would make sense here (though zsh works as well).

Originally created by @Sp1l on GitHub (Nov 29, 2017). https://github.com/google/brotli/blob/0ad94eed00420bf1154cb16a289aa27efbb30c01/bootstrap#L1 Even though the shebang indicates this uses plain shell, it uses arithmetic not available in plain shell. Hard depending on bash would make sense here (though zsh works as well).
Author
Owner

@eustas commented on GitHub (Nov 29, 2017):

Thanks for the report. Going to switch to plain shell + expr. Will it be better?

@eustas commented on GitHub (Nov 29, 2017): Thanks for the report. Going to switch to plain shell + expr. Will it be better?
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/brotli#204