#1064795 iproute2: installs same filename to both bin and sbin

Package:
iproute2
Source:
iproute2
Description:
networking and traffic control tools
Submitter:
Date:
2024-10-08 14:27:02 UTC
Severity:
normal
#1064795#5
Date:
2024-02-25 22:52:14 UTC
From:
To:
Dear Maintainer,

your package installs the filename `ip` to both bin and sbin as opposed to just one of those locations.

This causes a problem on a filesystem layout where bin and sbin are merged into a single real directory, typically by sbin being a symlink to bin. Such a filesystem layout has become standard on some distributions now, and others are moving onto in their next releases.

Please pick one location and install it only there. /usr/bin is preferred over any other location.

Thank you for maintaining software in debian.

#1064795#16
Date:
2024-08-16 15:21:38 UTC
From:
To:
Colin Watson <cjwatson@debian.org> writes:

I think it probably was just a coincidence, since it looks like the
change was made in order to fix #1064795 which was reported on
25 Feb 2024.

Luka, how about temporarily reverting this change to give people a
chance to prepare for it?

BTW I'm not directly affected by this AFAIK, so I'm not asking for me.

It just strikes me as obvious that removing any long-standing binary
path in Debian is pretty-much bound to break someone's system, and if
you want to do that you really ought to at least check, and preferably
try to work out a way of warning them about it, or fixing the breakage
first.

I note that neither the Changelog nor the NEWS file mentioned this as a
breaking change or issued anything like a warning about it.

https://salsa.debian.org/kernel-team/iproute2/-/commit/c4bb148dd4ed0601ca32ee8a458007d0c348d6c3

Cheers, Phil.

#1064795#21
Date:
2024-08-16 15:54:02 UTC
From:
To:
Ah, good to know, thanks.  I didn't notice that since it wasn't
mentioned in the iproute2 changelog.

Quite.  If nothing else, I think the code actually in the Debian archive
that relies on the old path ought to be changed _first_, e.g. via an
MBF.  I see a bunch of cases that are relatively subtle and might suck a
lot of other people's time trying to debug them from cold, such as
AppArmor profiles and example scripts, and it's just good manners to
give maintainers an explicit heads-up.

#1064795#26
Date:
2024-08-16 20:32:02 UTC
From:
To:
Or, of course, leave it forever since it causes no problems...
#1064795#31
Date:
2024-08-22 00:11:59 UTC
From:
To:
I've made a team upload of iproute2 (version 6.10.0-2) with this change
reverted.

Luca, please leave the symlink in place at least as long as there are
packages that rely on it.

Ben.