[Buildroot] [PATCH 2/2] package/iputils: fix IPUTILS_PERMISSIONS

[Thomas Petazzoni]

Hello,

On Mon, 17 Jun 2019 22:32:36 +0200
Petr Vorel <petr.vorel at gmail.com> wrote:

> The only thing to consider:
> Upstream doesn't care much about systems without usr merge.
> I wonder if systems without it (unset BR2_ROOTFS_MERGED_USR),
> whether we want to have /bin/ping and symlinked it into /usr/bin/ping.
> I see this is handled for coreutils (COREUTILS_BIN_PROGS).

What COREUTILS_CLEANUP_BIN is doing is not to install a symlink
between /bin/foo and /usr/bin/foo. Indeed, such a symlink is not
necessary: both /bin and /usr/bin are in the PATH, so having /bin/foo
and /usr/bin/foo is totally useless.

What COREUTILS_CLEANUP_BIN does is to make sure that in a
non-BR2_ROOTFS_MERGED_USR case, the symlinks to coreutils are installed
in the same location as the corresponding Busybox applets, so that we
properly handle the conflict between Busybox applets and full-blown
variants, as I explained in my reply to your comments on PATCH 1/2.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com