Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support pipefail with POSIX:2022 #258

Closed
tgross35 opened this issue Jun 16, 2022 · 8 comments
Closed

Support pipefail with POSIX:2022 #258

tgross35 opened this issue Jun 16, 2022 · 8 comments

Comments

@tgross35
Copy link

Hello all,

As I understand, POSIX:2022 will include the option to set -o pipefail. As Alpine uses some form of the POSIX-compliant sh, would it be possible to add this option in one of the upcoming releases?

@ncopa
Copy link
Contributor

ncopa commented Jun 16, 2022

This is great news! Do you have any reference for this?

Adding support for pipefail sounds like a great thing, and I would like to create a feature request for upstream busybox.

@tgross35
Copy link
Author

@ncopa glad you're on board! Didn't realize that sh came from busybox.

Finding references is tough, this is the best I can do: https://www.austingroupbugs.net/view.php?id=789. The POSIX docs are pretty locked down it seems, so I'm relying on some word of mouth (and that thread is pretty convincing). Even if the POSIX spec doesn't come out until 2023, I definitely don't think it's a bad thing to add before then.

Is the busybox request something you'll do, or that you'd like me to do? (I just don't know where since it looks like only the mirror is on GH)

@ncopa
Copy link
Contributor

ncopa commented Jun 16, 2022

Would be great if you could help me with a busybox request. You can do so by creating an issue here: https://bugs.busybox.net/ and I would also recommend that you write an email to the busybox mailing list.

The maintainers are generally slow on responding but the sooner this gets attention the better. I might even have a look at implementing this myself and send them a patch if I get the time.

@tgross35
Copy link
Author

Sounds good, I will do so later today and let you know. Thanks for the info!

@tgross35
Copy link
Author

OK cool, bugz is here https://bugs.busybox.net/show_bug.cgi?id=14851, and I got your email from a patch and CC'd you on what I sent to the mailing list

@tgross35
Copy link
Author

Also linking a better source here, this answer is made by somebody named Stéphane https://unix.stackexchange.com/questions/654885/who-is-responsible-for-providing-set-o-pipefail/654932#654932 which I'd have to imagine is the same stephane on the austingroupbugs site

@ncopa
Copy link
Contributor

ncopa commented Jun 20, 2022

It says that busybox ash already has support for it. That was news for me! It means we can close this?

@tgross35
Copy link
Author

Hey, that's a good point - easy fix. Thanks Natanael

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

No branches or pull requests

2 participants