env: fix only long options are allowed to contain '='#7008
env: fix only long options are allowed to contain '='#7008cakebaker merged 8 commits intouutils:mainfrom
Conversation
|
GNU testsuite comparison: |
|
great progress, thanks!
Looks like a regression |
|
I'm having trouble replicating the printenv test failure from github-actions. |
|
GNU testsuite comparison: |
|
The log says: |
|
i guess it is this line: |
|
Oh, got it. I was ensuring that all short options could not contain '=', but that is not correct. It should just be the short unset option (-u) that cannot contain '=' as its first character. I committed a change to fix this. I've tested this with that line of code directly (not sure why it wasn't catching on my system as it was running the same tests; I suppose it could be because I am using fish shell to run these tests), and run all relevant tests. I believe it should pass now. |
|
GNU testsuite comparison: |
|
Thanks for your PR :) |
This fixes #6165 and was adapted from #6711.
This PR contains changes that ensure short options (-u) do not have '=' as their arguments' first character in accordance with GNU/POSIX standards.
Tests were added to ensure this change does not regress.