[patch #9754] find: new '-attr' predicate to test inode flags

[Matt Whitlock]

Follow-up Comment #2, patch #9754 (project findutils):

Inode flags are not specific to Linux's Extended File System, but they _are_
specific to Linux. They were lifted from Ext to Linux's (filesystem-agnostic)
VFS layer because they're used by multiple file system implementations. The
following note from <linux/fs.h> may be instructive:


 * Note: for historical reasons, these flags were originally used and
 * defined for use by ext2/ext3, and then other file systems started
 * using these flags so they wouldn't need to write their own version
 * of chattr/lsattr (which was shipped as part of e2fsprogs).


As it happens, I mostly use these flags on XFS volumes.

Regarding the "Casefold" and "Verity" flags you mentioned, I do not see these
in the <linux/fs.h> header. I would guess that they are specific to Ext and
are accessed by filesystem-specific APIs. If they are ever promoted to the VFS
layer, then it might make sense to add them to Find.

I have added an Autoconf check and preprocessor guard directives so that -attr
is implemented only on systems with a working <linux/fs.h> header.

I do not see the code style warnings that you quoted, but I have incorporated
fixes that should eliminate them.

I am indeed willing to assign copyrights for my contribution to the Free
Software Foundation.

(file #46811)
    _______________________________________________________

Additional Item Attachment:

File name: 0002-find-support-attr-only-if-OS-supports-inode-flags.patch
Size:13 KB
   
<https://savannah.gnu.org/file/0002-find-support-attr-only-if-OS-supports-inode-flags.patch?file_id=46811>



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?9754>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/