[Buildroot] [PATCH v2 4/7] arch/xtensa: accept the overlay to be an URL

[Yann E. MORIN <yann.morin.1998@free.fr>]

On 2017-07-09 15:48 +0200, Thomas Petazzoni spake thusly:
> Hello,
> 
> On Sun,  9 Jul 2017 05:21:58 -0700, Max Filippov wrote:
> 
> > diff --git a/arch/arch.mk.xtensa b/arch/arch.mk.xtensa
> > index ecb9c1086a70..ba3e3025b7d0 100644
> > --- a/arch/arch.mk.xtensa
> > +++ b/arch/arch.mk.xtensa
> > @@ -9,7 +9,15 @@
> >  #       tar xf $(ARCH_XTENSA_OVERLAY_FILE) -C $(@D) --strip-components=1 gcc
> >  #   endif
> >  ################################################################################
> > -ARCH_XTENSA_OVERLAY_FILE = $(call qstrip,$(BR2_XTENSA_OVERLAY_FILE))
> > +BR_ARCH_XTENSA_OVERLAY_FILE = $(call qstrip,$(BR2_XTENSA_OVERLAY_FILE))
> > +ifneq ($(filter http://% https://% ftp://% scp://%,$(BR_ARCH_XTENSA_OVERLAY_FILE)),)
> > +ARCH_XTENSA_OVERLAY_URL = $(BR_ARCH_XTENSA_OVERLAY_FILE)
> > +ARCH_XTENSA_OVERLAY_FILE = $(BR2_DL_DIR)/$(notdir $(BR_ARCH_XTENSA_OVERLAY_FILE))
> 
> We should be using $(DL_DIR) instead of $(BR2_DL_DIR). Also, an issue
> here is that this will not work nicely with the per-package
> sub-directory in DL_DIR that Maxime Hadjinlian is working on.
> 
> Indeed, we have a single download that is done by a lot of different
> packages. Since the downloaded file is quite small, I would advocate
> that we simply re-download it for each of gcc, binutils, linux, etc.
> 
> 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...

> But indeed, I don't see how it can have a hash, since it's a custom
> stuff.
> 
> 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.

But as you say, this means the file will be duplicated for each
component that needs the overlay.

Regards,
Yann E. MORIN.

> Thanks!
> 
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'