[PATCH] Set LC_ALL=C when calling sed in cmd_show

Allan Odgaard lists+pass at simplit.com
Mon Mar 30 15:28:05 CEST 2020


On 30 Mar 2020, at 19:58, Rémi Lapeyre wrote:

>> Can you provide steps to reproduce?
> In my case the issue was with 'Productivite\xcc\x81’ but it’s 
> harder to reproduce with an emoji symbol. It’s possible to bypass 
> the issue by installing gnu-sed on MacOS but it requires to install an 
> external program, to change the PATH, and PassFF still fails when 
> doing that.
>
> This solution is not perfect since as you see, the byte sequence is 
> replaced and not ignored and passed though as is.

Thanks, as expected, the issue is not with `sed`. Instead it is `tree` 
which has some odd default behaviour in that it “escapes” (mangles) 
characters (byte sequences) it deems “unprintable”.

     % mkdir test && touch test/Æblegrød && tree test
     test
     └── ?\206blegrød

This can be changed by passing the `-N` option:

     % tree -N test
     test
     └── Æblegrød

Please verify that adding `-N` to `tree` works for you, and if so, I 
suggest updating your patch to use this solution instead of changing the 
locale.


More information about the Password-Store mailing list