[Buildroot] [PATCH v2 4/7] arch/xtensa: accept the overlay to be an URL
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sun Jul 9 15:24:58 UTC 2017
Hello,
On Sun, 9 Jul 2017 15:57:18 +0200, Yann E. MORIN wrote:
> > Another issue that I have is that we're not checking its hash, even if
> > the file used in the qemu xtensa defconfigs doesn't have any version
> > information in its name, making it quite likely that the contents of
> > the file will change without us knowing.
>
> Hmm... IIRC, in the following patch, we're using the file from a specific
> git sha1, aren't we? In this case, we don't really expect it to change...
Yes, we are fetching from a specific git sha1. And anyway since it's a
custom download, there's no way to store a hash.
> > In any case, I'd like to hear about a direction to solve the
> > per-package DL_DIR issue.
>
> I think this will be as simple as using:
>
> ARCH_XTENSA_OVERLAY_FILE = $($(PKG)_DL_DIR)/$(notdir $(BR_ARCH_XTENSA_OVERLAY_FILE))
>
> since this will be expanded in the commands for the .stamp_downloaded
> stamp file.
Indeed, so it's fine for me. I've applied the patches.
> But as you say, this means the file will be duplicated for each
> component that needs the overlay.
Not a big deal IMO. It's a 90 KB file, and it's only for the specific
case of Xtensa, so I don't think it's worth extending the infra to
optimize for this.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list