[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