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

exec: Don't exit on exec failure if interactive #87

Merged
merged 1 commit into from
Nov 15, 2024
Merged

Conversation

magicant
Copy link
Owner

@magicant magicant commented Nov 14, 2024

This commit implements the new POSIX.1-2024 requirement.

Previously, failure in the exec built-in on an interactive shell caused the shell to exit only if the POSIXly-correct mode was off. The mode no longer affects the shell behavior.


This PR is the yash counterpart to magicant/yash-rs#424.

@magicant magicant added the enhancement New feature or request label Nov 14, 2024
@magicant magicant added this to the 2.58 milestone Nov 14, 2024
@magicant magicant self-assigned this Nov 14, 2024
This commit implements the new POSIX.1-2024 requirement.

Previously, failure in the exec built-in on an interactive shell caused
the shell to exit only if the POSIXly-correct mode was off. The mode no
longer affects the shell behavior.

The test file "exec-p.tst" is synced with:
https://github.com/magicant/yash-rs/blob/218275d85908442e4aa4c06dbe41d93f6cf62a37/yash-cli/tests/scripted_test/exec-p.sh
@magicant magicant merged commit d05054f into trunk Nov 15, 2024
3 checks passed
@magicant magicant deleted the exec-exit branch November 15, 2024 15:15
@magicant magicant mentioned this pull request Nov 15, 2024
47 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant