[Buildroot] [PATCH 1/1] iputils: set the permissions with IPUTILS_PERMISSIONS

[Arnout Vandecappelle <arnout@mind.be>]

On 10/06/2019 22:06, Petr Vorel wrote:
> Hi Arnout,
> 
>>  Applied to master, thanks. I updated the commit message a little.
> Thanks! You've been faster, I might try to think about setcap (xattr) use if
> USE_CAP=true, and setuid if USE_CAP=false.

 Great!

> Is there an easy way to detect
> iso9660 usage?

 BR2_TARGET_ROOTFS_ISO9660 of course. Well, that one is special actually because
in case of BR2_TARGET_ROOTFS_INITRAMFS or BR2_TARGET_ROOTFS_ISO9660_INITRD, the
rootfs is not actually stored in the iso9660.

 But I don't know which other filesystems lack support for xattrs.

 So maybe just assume the filesystem supports xattrs, we can fix things later if
needed.

> (+ setcap being available on host).

 setcap can't be used, since setting xattrs can only be done by root and we're
not root while building. It could be used when running in a fakeroot
environment, but I believe fakeroot is a bit flakey when it comes to
capabilities. So the appropriate xattrs should instead be set with
IPUTILS_PERMISSIONS.

 Regards,
 Arnout


> I'm not aware any other xattr limitations thatn iso9660.
> 
>>> it'd be great also create symlink ping -> ping6, but don't know how.
> 
>>  This can be done with a post-install hook.
> Thanks, I'll do it in separate patch sometimes this week.
> 
>>  Also, the bump to s20190515 should not have removed the traceroute6 program.
>> Could you make a second patch that adds it back?
> Sure, I'll try to do it this week.
> 
>>  Regards,
>>  Arnout
> 
> Kind regards,
> Petr
>