script: quote filename variable in link format bash script #1915
pull aso20455 wants to merge 1 commits into bitcoin:master from aso20455:master changing 1 files +1 −1-
aso20455 commented at 8:43 am on August 8, 2025: noneQuote the $fname variable in grep command to properly handle filenames containing spaces or special characters. This follows bash best practices and prevents potential shell injection or parsing issues.
-
fix: quote filename variable in link format checker script e190f44300
-
jonatack added the label Proposed BIP modification on Aug 8, 2025
-
jonatack removed the label Proposed BIP modification on Aug 8, 2025
-
jonatack added the label CI on Aug 8, 2025
-
jonatack commented at 5:30 pm on August 15, 2025: memberThank you for your proposal. This isn’t strictly needed here but seems fine to do. We usually no longer directly merge typo and code style fixups in favor of batching them into an end-of-month cleanup that will credit you as (co-)author. Thanks!
-
jonatack added the label Typo-to-be-collected on Aug 15, 2025
-
jonatack closed this on Aug 15, 2025
-
jonatack renamed this:
fix: quote filename variable in link format checker script
script: quote filename variable in link format bash script
on Aug 15, 2025 -
aso20455 commented at 6:40 pm on August 15, 2025: noneThanks for the review!
Labels
Typo-to-be-collected
CI
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-08-19 23:10 UTC
This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-08-19 23:10 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me