Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Arnout Vandecappelle <arnout@mind.be>
Cc: Conrad Ratschan <conrad.ratschan@collins.com>, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1] package/coreutils: Prevent overwriting of fakedate
Date: Sat, 28 Aug 2021 14:22:56 +0200	[thread overview]
Message-ID: <20210828122256.GI2661@scaer> (raw)
In-Reply-To: <af866dec-70f5-fea2-3f59-01908afe35a6@mind.be>

Arnout, All,

On 2021-08-28 13:47 +0200, Arnout Vandecappelle spake thusly:
> On 27/08/2021 23:14, Conrad Ratschan via buildroot wrote:
> > When BR2_REPRODUCIBLE is set and host-coreutils needs to be built, the
> > fakedate script installed to `host/bin/date` will be overwritten by
> > host-coreutils. Disable installing the `date` binary in host-coreutils
> > when BR2_REPRODUCIBLE is set.
> > 
> > Signed-off-by: Conrad Ratschan <conrad.ratschan@collins.com>
> > ---
> >  package/coreutils/coreutils.mk | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/package/coreutils/coreutils.mk b/package/coreutils/coreutils.mk
> > index 65234a113e..1bdfe2f9cf 100644
> > --- a/package/coreutils/coreutils.mk
> > +++ b/package/coreutils/coreutils.mk
> > @@ -164,5 +164,10 @@ HOST_COREUTILS_CONF_OPTS = \
> >  	--without-gmp \
> >  	--enable-install-program=ln,realpath
>  We were supposed already to only install ln and realpath, so this issue
> shouldn't have happened to begin with. Could you investigate in a bit more
> detail why this --enable-install-program doesn't seem to do the trick?

See the comment just above the HOST_COREUTILS_CONF_OPTS:

    # A lot of other programs still get installed, however, but disabling
    # them does not gain much at build time, and is a loooong list that is
    # difficult to maintain...

> > +# Avoid overwriting fakedate when creating a reproducible build
> > +ifeq ($(BR2_REPRODUCIBLE),y)
>  If there really is no way to get enable-install-program to work correctly, then
> we should probably not make this conditional on BR2_REPRODUCIBLE anyway.

Yes, we should really just disable it unconditionally...

So, I extended the commit log, the comment to explain all that, and
uncoditionally disabled installing date.

Applied to master, thanks.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

  reply	other threads:[~2021-08-28 12:23 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-27 21:14 [Buildroot] [PATCH 1/1] package/coreutils: Prevent overwriting of fakedate Conrad Ratschan via buildroot
2021-08-28 11:47 ` Arnout Vandecappelle
2021-08-28 12:22   ` Yann E. MORIN [this message]
2021-09-07 12:40 ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210828122256.GI2661@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=arnout@mind.be \
    --cc=buildroot@buildroot.org \
    --cc=conrad.ratschan@collins.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox