Skip to content

Makefile.defs Use /bin/sh for $SHELL#1530

Closed
Baitinq wants to merge 1 commit into
koreader:masterfrom
Baitinq:posix_sh
Closed

Makefile.defs Use /bin/sh for $SHELL#1530
Baitinq wants to merge 1 commit into
koreader:masterfrom
Baitinq:posix_sh

Conversation

@Baitinq

@Baitinq Baitinq commented Oct 6, 2022

Copy link
Copy Markdown
Contributor

As we just use posix compatible shell scripts. Related #1529


This change is Reviewable

As we just use posix compatible shell scripts. Related koreader#1529
@NiLuJe

NiLuJe commented Oct 6, 2022

Copy link
Copy Markdown
Member

The buildbot doesn't agree ;o).

({} list construct, but there may be others, so I wouldn't necessarily rock the boat there, as I guarantee that I may rant and rage
in the future if I can't use bashisms ;p).

@Frenzie

Frenzie commented Oct 6, 2022

Copy link
Copy Markdown
Member

Well, I meant in the Makefile(s) which isn't the same thing as the separate scripts of course. ;-) Anyway, clearly we use {}. :-)

@Frenzie Frenzie closed this Oct 6, 2022
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.

3 participants