[Buildroot] [PATCH] infra/pkg-generic: use site method for same-site extra downloads
Peter Korsgaard
peter at korsgaard.com
Sat Nov 9 17:13:20 UTC 2019
>>>>> "Yann" == Yann E MORIN <yann.morin.1998 at free.fr> writes:
> When a package specifies extra downloads, it has the option to only name
> the basename of the extra download, in which case that extra download
> will be retrieved from the same location the main download is retrieved
> from.
> In that case, if the extra download contains a '+', it would confuse the
> dl-wrapper, which believes the LHS of the '+' is the site method, and
> the RHS the actual URI, and so the dl-wrapper mangles and damages the
> URI when fetching such extra downloads, like that happens with android
> tools, where the proper URI and mangled URIs of the extra download are,
> respectively:
> https://launchpad.net/ubuntu/+archive/primary/+files/android-tools_4.2.2+git20130218-3ubuntu41.debian.tar.gz
> http://archive/primary/+files/android-tools_4.2.2+git20130218-3ubuntu41.debian.tar.gz
> We fix that by always propagating the site method to extra downloads,
> but only when they are specified as relative to the main download URI.
> For the extra downloads that specify a full URI, it is not systematic
> that it is the same site method. For example, a main download could be a
> git clone, but an extra download a pure http download; in that case we
> can't replicate the site method for extra downloads, so they'll have to
> take appropriate care to specify the required method and encoding if
> needed.
> Reported-by: Jemy Zhang <jemy.zhang at gmail.com>
> Signed-off-by: Yann E. MORIN <yann.morin.1998 at free.fr>
> Cc: Jemy Zhang <jemy.zhang at gmail.com>
Committed to 2019.02.x and 2019.08.x, thanks.
--
Bye, Peter Korsgaard
More information about the buildroot
mailing list