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
next prev parent 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