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

Fix endless loop on CTRL-C with bash 5.2 #656

Conversation

martin-schulze-vireso
Copy link
Member

Fixes #654

@martin-schulze-vireso martin-schulze-vireso requested a review from a team as a code owner September 18, 2022 19:39
From [Bash 5.2 release notes](https://mail.gnu.org/archive/html/bug-bash/2022-07/msg00129.html):
x. New shell option: patsub_replacement. When enabled, a `&' in the replacement
   string of the pattern substitution expansion is replaced by the portion of
   the string that matched the pattern. Backslash will escape the `&' and
   insert a literal `&'.
Otherwise would contain the \
@martin-schulze-vireso martin-schulze-vireso force-pushed the fix/issue_654_endless_loop_on_sigint_with_bash_5.2 branch from 06d5913 to bf7b7e8 Compare September 23, 2022 21:57
@martin-schulze-vireso martin-schulze-vireso merged commit c0d2ca1 into bats-core:master Sep 24, 2022
yarikoptic pushed a commit to neurodebian/bats that referenced this pull request Sep 26, 2022
These patches have been extracted from upstream PR #656 ("Fix endless
loop on CTRL-C with bash 5.2").

bats-core/bats-core#656

Closes: #1020017
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.

SIGINT throws bats into endless loop on Bash 5.2 (RC)
1 participant