[Buildroot] [PATCH] util-linux: add menu items for login utilities

[Carlos Santos <casantos@datacom.ind.br>]

> From: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>
> To: "Carlos Santos" <casantos@datacom.ind.br>
> Cc: buildroot at buildroot.org
> Sent: Wednesday, August 23, 2017 10:42:52 AM
> Subject: Re: [Buildroot] [PATCH] util-linux: add menu items for login utilities

> Hello,
> 
> On Sat, 19 Aug 2017 20:55:33 -0300, Carlos Santos wrote:
> 
>> +config BR2_PACKAGE_UTIL_LINUX_LOGIN_UTILS
>> +	bool "login utilities"
>> +	depends on BR2_USE_MMU # fork() (login, runuser, su, sulogin)
>> +	depends on (BR2_ENABLE_LOCALE && BR2_USE_WCHAR) # linux-pam
>> +	depends on !BR2_STATIC_LIBS
>> +	depends on !BR2_TOOLCHAIN_USES_MUSL # linux-pam
>> +	select BR2_PACKAGE_UTIL_LINUX_LAST
>> +	select BR2_PACKAGE_UTIL_LINUX_LOGIN
>> +	select BR2_PACKAGE_UTIL_LINUX_RUNUSER
>> +	select BR2_PACKAGE_UTIL_LINUX_SU
>> +	select BR2_PACKAGE_UTIL_LINUX_SULOGIN
> 
> I don't really see the need to keep this option. We have
> BR2_PACKAGE_UTIL_LINUX_BINARIES, because it matches the upstream
> --enable-all-programs option. But here,
> BR2_PACKAGE_UTIL_LINUX_LOGIN_UTILS doesn't match anything provided by
> upstream, it's just a random category created by Buildroot.
> 
> So let's get rid of this option, with appropriate Config.in.legacy
> handling selecting all the individual tools, and only have sub-options
> per tools, matching the --enable-<foo> options provided by util-linux
> configure script.

I'm not particularly fond of this option but maintained it because
it is usefull reduce the size of the defconfig files. Anyway, I will
send a new patch removing the login utils group.

-- 
Carlos Santos (Casantos) - DATACOM, P&D
?The greatest triumph that modern PR can offer is the transcendent 
success of having your words and actions judged by your reputation, 
rather than the other way about.? ? Christopher Hitchens