On Wed, Nov 20, 2013 at 7:55 PM, Xyne <xyne@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