Skip to content

bash: update to 5.3 patch level 12#29683

Merged
BKPepe merged 1 commit into
openwrt:masterfrom
Yang-Wei-Ting:bash
Jun 9, 2026
Merged

bash: update to 5.3 patch level 12#29683
BKPepe merged 1 commit into
openwrt:masterfrom
Yang-Wei-Ting:bash

Conversation

@Yang-Wei-Ting

Copy link
Copy Markdown
Contributor

📦 Package Details

Maintainer: Me

Description:
Update bash to 5.3 patch level 12.


🧪 Run Testing Details

  • OpenWrt Version: master
  • OpenWrt Target/Subtarget: Arm SystemReady (EFI) compliant / 64-bit (armv8) machines
  • OpenWrt Device: QEMU

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
    make package/<your-package>/refresh V=s
  • It is structured in a way that it is potentially upstreamable
    (e.g., subject line, commit description, etc.)
    We must try to upstream patches to reduce maintenance burden.

- Fix loop in subshells calling wait builtin with inherited job list
- Fix mapfile problem when callback unsets the variable it is modifying
- Fix subshells inappropriately running the EXIT trap if they receive a
  fatal signal before resetting traps

Signed-off-by: Wei-Ting Yang <williamatcg@gmail.com>
@BKPepe

BKPepe commented Jun 9, 2026

Copy link
Copy Markdown
Member

Nitpick: Do we need to rename all these patches?

@Yang-Wei-Ting

Copy link
Copy Markdown
Contributor Author

I follow the rule in

https://openwrt.org/docs/guide-developer/toolchain/use-patches-with-buildsystem#naming_patches

All patches should be named 'NNN-lowercase_shortname.patch'

If that is not necessary, I can revert renames.

@BKPepe

BKPepe commented Jun 9, 2026

Copy link
Copy Markdown
Member

Okay:-) Lets merge this.

@BKPepe BKPepe merged commit 3085718 into openwrt:master Jun 9, 2026
12 checks passed
@Yang-Wei-Ting Yang-Wei-Ting deleted the bash branch June 9, 2026 11:17
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.

2 participants