Repository navigation
Regex aliases do not allow forward-slash '/' character #982
Description
The hledger docs show a nice way to rewrite all account names to allow using a period character . instead of a colon : as the account separator:
alias /\./=:
Changing this to an escaped forward slash results in an unexpected syntax error:
hledger: hledger.journal:5:10:
|
5 | alias /\// = :
| ^
unexpected '/'
expecting '='
It seems like the forward slash isn't escaped, so the regex ends early.
Activity
simonmichael commentedon Mar 5, 2019
Thanks for the report.
raehik commentedon Mar 5, 2019
have done a tiny bit of Haskell before and might have a look at this tomorrow :)
raehik commentedon Mar 9, 2019
Looks like regex alias parsing occurs at JournalReader.hs#L438. Line 442 reads the regex. But any
/ends it, escaping isn't considered.raehik commentedon Mar 11, 2019
Would love to fix this so I can pretend my accounts are directories like a true *nix addict, but I'm not sure how to begin. Rewrite
\/to/and continue reading, perhaps? @simonmichael any chance you could point me to similar/related code, or some tutorials for thedoconstruct?simonmichael commentedon Mar 11, 2019
@raehik I don't know either, off the top of my head. It's a parsing problem, not too hard I think. If I were you I would (after skimming a few of its tutorials, maybe) get familiar with megaparsec's api/combinators, something in there is bound to be helpful. Once you've done that you could also ask for ideas on #haskell.
simonmichael commentedon Mar 11, 2019
(which you can find by starting at http://hackage.haskell.org/package/megaparsec)
raehik commentedon Mar 13, 2019
Cheers for the links! I'll do some reading when I can, & I believe building locally worked fine so I can mess around.
fix: alias: Allow escaped characters in regular expression aliases. (h…
fix: alias: Allow escaped characters in regular expression aliases. (#…
hledger, hledger-lib: Update to 1.28
hledger, hledger-lib: Update to 1.28
hledger, hledger-lib: Update to 1.28