diff options
author | Eli Schwartz <eschwartz@archlinux.org> | 2019-11-26 01:07:56 +0100 |
---|---|---|
committer | Andrew Gregory <andrew@archlinux.org> | 2020-01-14 10:26:52 +0100 |
commit | 6f1a9e6ea85a49b8aeb3b7bd2454215e9c34f565 (patch) | |
tree | 30efe70380d35d63983efa04e38679aaabb43bb7 /scripts/libmakepkg/source | |
parent | 41c3b1d78c3807beb4246624b20a31e374fb5af1 (diff) | |
download | pacman-6f1a9e6ea85a49b8aeb3b7bd2454215e9c34f565.tar.gz pacman-6f1a9e6ea85a49b8aeb3b7bd2454215e9c34f565.tar.xz |
makepkg: fix regression that broke extraction of file:// sources
In commit 9c817b654996249b8022e189ee7e2692f4668431 we made these sources
extendable, and heuristically determined the correct extraction
functions to use. But our fallback for protos that didn't have an exact
extract_* function didn't take into account that 'extract_file' matches
an actual proto... so we passed the netfile in while the function
expected a file.
Solution: the function should expect a netfile too, thereby allowing us
to delay an attempted resolution of netfile -> file, to the one case
where it is actually used. This makes us slightly more efficient in the
non-file case, makes our functions a bit more consistent, and makes
file:// extraction work again.
Fixes FS#64648
Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>
Signed-off-by: Allan McRae <allan@archlinux.org>
(cherry picked from commit 349c22d043290ccd0cce9f30981f5415e295442a)
Diffstat (limited to 'scripts/libmakepkg/source')
-rw-r--r-- | scripts/libmakepkg/source/file.sh.in | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/libmakepkg/source/file.sh.in b/scripts/libmakepkg/source/file.sh.in index 7297a1c6..df4256f7 100644 --- a/scripts/libmakepkg/source/file.sh.in +++ b/scripts/libmakepkg/source/file.sh.in @@ -83,8 +83,9 @@ download_file() { } extract_file() { - local file=$1 + local netfile=$1 + local file=$(get_filename "$netfile") local filepath=$(get_filepath "$file") rm -f "$srcdir/${file}" ln -s "$filepath" "$srcdir/" |