On 14/08/11 06:07, Dave Reisner wrote:
Signed-off-by: Dave Reisner<dreisner@archlinux.org> --- bsdtar can't extract a tarball with the same path/file in it twice, but it will gladly pack it that way... odd. Credit for the AUR for finding this.
scripts/makepkg.sh.in | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index f464014..e5840f1 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1540,6 +1540,10 @@ check_sanity() { error "$(gettext "%s file (%s) does not exist.")" "$i" "$file" ret=1 fi + if in_array "$file" "${source[@]}"; then + error "$(gettext "%s file found in source array: %s")" "$i" "$file" + ret=1 + fi done done
I am sure we already "fixed" this in the past. Did the fix get lost with some of that re-factoring that happened with handling install/changelog files? As background, there was a big discussion a couple of years back about whether we should support the inclusion of install files (changelog was different then) in the source array. The majority opinion (of which I was not part...) then was to do so. So even though this patch goes with my line of thinking about the handling of install files, I think we should fix the actual issue and not pack the same file twice. Allan