Re: [PATCH 3/4] build: put iptraf into sbindir
On Monday 2011-04-11 14:31, Nikola Pajkovsky wrote:
Jan Engelhardt <jengelh(a)medozas.de> writes:
> Superuser permissions are required to run it anyway.
Yes it is. Iptraf-ng can be compiled with -D_ALLOWUSERS and can be
set suid to binary hence user can run iptraf-ng without root
permissions. Nevertheless this is not good behavior, because you can run
iptraf-ng only once right now. It creates pid and check if other
instance is running thus only one user can run it.
It's just a matter of speaking. sbin is not a place for programs
that are only executable by root, but ones that are usually for root.
Think traceroute.
> -bin_PROGRAMS = iptraf-ng rvnamed-ng
> +sbin_PROGRAMS = iptraf-ng rvnamed-ng
>
> NULL =
> TUI_C =
Patch is not complete. It need to be patched spec file also, but if you
are not familiar I'll fix it. Taken.
Usually, packages are not supposed to carry around distro-specific
pieces, because they are either not maintained, or too specific.
(Cf. removal of "debian" dirs from upstream projects.)