slaesvuo/hledger
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix:Hledger.Utils.String: quoteForCommandLine no longer quotes "7" [#2468]

Browse Source 
7 is no longer considered a scary shell character.

(This appears to have been harmless, used only for formatting the
balance assertion failure message.)
This commit is contained in:
Simon Michael 2025-10-01 17:56:49 -10:00
parent 7dbdbf1e17
commit 71b1293f06
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hledger-lib/Hledger/Utils/String.hs
View File

@ -184,7 +184,7 @@ quotechars, whitespacechars, redirectchars, shellchars :: [Char]
quotechars = "'\"" quotechars = "'\""
whitespacechars = " \t\n\r" whitespacechars = " \t\n\r"
redirectchars = "<>" redirectchars = "<>"
shellchars = "<>(){}[]$7?#!~`" shellchars = "<>(){}[]$?#!~`"
-- | Quote-aware version of words - don't split on spaces which are inside quotes. -- | Quote-aware version of words - don't split on spaces which are inside quotes.
-- NB correctly handles "a'b" but not "''a''". Can raise an error if parsing fails. -- NB correctly handles "a'b" but not "''a''". Can raise an error if parsing fails.