[pacman-dev] FS #6246 - makepkg fails if the extensions is wrong on the archive

James Rosten seinfeld90 at gmail.com
Mon Jan 22 16:26:09 EST 2007


On 1/22/07, Johannes Weiner <hannes at saeurebad.de> wrote:
> No, this is not readable. What do you have against
>
>         *.gz|application/x-gzip)
>                 cmd="gunzip -f $file" ;;
>         *.bz2|application/x-bzip2)
>                 cmd="bunzip -f $file" ;;
>
> There is an error at the *.bz2 detection also, you check twice for
> application/x-gzip where you meant application/x-bzip2.
>
> Please don't mix up quoting and not-quoting.
>
> Do we need to check for the file-extension AND the `file' result? I
> think if they mismatch, we should rather trust `file'.
>
> Thank you very much and please forgive me for being so picky..
> Hannes

Nowhere in that entire patch do I check the file-extension....did you
notice the -'s in front of the lines with file extensions...that means
they are going to be removed.  But thank you for noticing the spelling
error.

~ Jamie / yankees26




More information about the pacman-dev mailing list