Closed giuseppe closed 4 weeks ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: giuseppe
The full list of commands accepted by this bot can be found here.
The pull request process is described here
LGTM
got 2 LGTMs, can we merge?
This is still marked as draft.
I suppose the relevant reference is https://github.com/containers/composefs/blob/3d2af03cd1b1eecfdbb38cfb9f6757a4de8fd713/tools/composefs-info.c#L88-L92 ; and that program does not call setlocale
.
/lgtm
To avoid a mismatch with the C composefs library that uses isgraph() and isprint(), do not use the unicode package but provide the expected definition for these functions.