[Buildroot] [PATCH] infra: add the transient download mechanism
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Thu Jan 16 10:05:38 UTC 2020
On Thu, 16 Jan 2020 10:01:09 +0000
Nicolas Carrier <nicolas.carrier at orolia.com> wrote:
> Just to be sure I understand, when you say "when TRANSIENT_DOWNLOAD is
> NO.", it'd be the same as if TRANSIENT_DOWNLOAD download is not defined
> at all, right?
Yes.
There is maybe a little bit of an issue in Yann's current
implementation as he does:
$(if $($(2)_DOWNLOAD_TRANSIENT),-F) \
so in fact, -F would be passed as soon as <pkg>_DOWNLOAD_TRANSIENT is
non-empty, so <pkg>_DOWNLOAD_TRANSIENT = NO would in fact pass -F.
But this bug already exists for <pkg>_GIT_SUBMODULES:
$(if $($(2)_GIT_SUBMODULES),-r) \
And it is trivial to fix.
> If it's the case, then that seems to me like a good idea, because it
> renders explicit the risk taken by the user when using a branch (which
> can be done unknowingly ATM).
Absolutely.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list