On 9/25/19 6:28 PM, Eric Biggers wrote: > On Wed, Sep 25, 2019 at 12:23:25PM -0500, Eric Sandeen wrote: >> On 8/12/19 12:56 PM, Eric Biggers wrote: >>> From: Eric Biggers <ebiggers@xxxxxxxxxx> >>> >>> get_encpolicy uses the FS_IOC_GET_ENCRYPTION_POLICY ioctl to retrieve >>> the file's encryption policy, then displays it. But that only works for >>> v1 encryption policies. A new ioctl, FS_IOC_GET_ENCRYPTION_POLICY_EX, >>> has been introduced which is more flexible and can retrieve both v1 and >>> v2 encryption policies. >> >> ... >> ... >>> + fprintf(stderr, >>> + "%s: unexpected error checking for FS_IOC_GET_ENCRYPTION_POLICY_EX support: %s\n", >> >> Darrick also mentioned to me off-list that the io/encrypt.c code is chock full of >> strings that really need to be _("translatable") >> > > Sure, I can do that, though is this really something that people want? These > commands are only intended for testing, and the xfsprogs translations don't seem > actively maintained (only 1 language was updated in the last 10 years?). True that there's not a ton of translation going on now, but it's easy enough to toss in _("foo") and keep the possibility open, and be consistent. So thanks :) -Eric