Skip to content

POSIX compliance - #4

Open
u20n wants to merge 3 commits into
Bladtman242:masterfrom
u20n:master
Open

POSIX compliance#4
u20n wants to merge 3 commits into
Bladtman242:masterfrom
u20n:master

Conversation

@u20n

@u20n u20n commented Oct 23, 2023

Copy link
Copy Markdown
  • shebang changed from bash to sh
  • replaced bash for loops with POSIX adjacent
  • replaced echo with printf

- shebang changed from bash to sh
- replaced bash for loops with POSIX adjacent
- replaced echo with printf
@u20n

u20n commented Oct 23, 2023

Copy link
Copy Markdown
Author

I noted that the bell echo produced an extra newline, I assumed this wasn't intentional behavior. A simple fix to revert behavior is of course to just insert a newline before the status message in the relevant printf calls.

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.

1 participant