[Buildroot] [PATCH 1/2] coreutils: Do not install uptime
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Sep 10 20:45:29 UTC 2014
Dear Vicente Olivert Riera,
On Wed, 10 Sep 2014 15:30:05 +0100, Vicente Olivert Riera wrote:
> coreutils' uptime doesn't work and its execution fails with a message
> like this one:
>
> uptime: couldn't get boot time: No such file or directory
>
> busybox and procps-ng provide a working alternative.
>
> This is a common old problem as we can see here:
>
> https://lists.openwrt.org/pipermail/openwrt-devel/2011-March/010006.html
>
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
> Reviewed-by: Markos.Chandras <Markos.Chandras at imgtec.com>
> ---
> package/coreutils/coreutils.mk | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/package/coreutils/coreutils.mk b/package/coreutils/coreutils.mk
> index 07a1347..7457b79 100644
> --- a/package/coreutils/coreutils.mk
> +++ b/package/coreutils/coreutils.mk
> @@ -61,7 +61,8 @@ COREUTILS_CONF_ENV = ac_cv_c_restrict=no \
> PERL=missing
>
> COREUTILS_CONF_OPT = --disable-rpath \
> - --enable-install-program=hostname
> + --enable-install-program=hostname \
> + --enable-no-install-program=uptime
I think this is not the proper fix. A more appropriate fix is to
actually get uptime to work, which is a oneliner. See my patch posted
at http://patchwork.ozlabs.org/patch/387971/.
I'll mark your patch as Rejected in patchwork. Of course, if you
disagree, feel free to turn back your patch to the "New" state with an
explanation.
Thanks a lot for noticing this issue in the first place!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list