[pacman-dev] [PATCH] Added mirror support to makepkg's source array.

[Ido Rosen]

On Wed, Nov 20, 2013 at 7:55 PM, Xyne <xyne at archlinux.ca> wrote:

> On 2013-11-18 12:09 +1000
> Allan McRae wrote:
>
> >What did you think about the proposal in an earlier reply to this thread:
> >
> >source=("mirror://_foo/blah/blah/foo.tar.gz")
> >_foo=("http://foo.com/" "http://bar.com/")
> >
> >I think we can bet safely that "mirror://" will not become a valid
> >protocol.  And we are replacing the start of the URL so just using the
> >initial "/" as the delimiter is fine.
> >
> >Allan
>
> That works, but I would prefer differential syntax to prevent confusion
> with
> regular URLs. "MIRROR://" should be enough. It stands out, and we already
> use
> upper-case for "SKIP" in the checksums array, so I think it is
> stylistically
> coherent.
>
> Regards,
> Xyne
>

In the interest of getting mirror support into PKGBUILD/makepkg/pacman,
regardless of style or aesthetic considerations.  Seems like these are the
two current options being considered:

(a) my original patch (multiple same-filenamed entries in $source), or,
(b) the proposal of having
    [dest_filename::]mirror://<_identifier>/[path_prefix/]<filename>
...scheme that references a shell variable or function name to get the list
from.

>From my perspective:   (a) is already written and easy to grok, involving
relatively minor changes to makepkg.sh.in;  (b) is more flexible but
involves duplication of code in makepkg, multiple entrypoints into
download_* functions...

So, do we vote or does Allan decide or do we roll dice? :-)  I'm happy to
resubmit the patch either way.

Ido