[pacman-dev] [PATCH] Fix vercmp and add additional tests

Dan McGee dpmcgee at gmail.com
Fri Jul 25 08:11:54 EDT 2008


On Fri, Jul 25, 2008 at 3:59 AM, Xavier <shiningxc at gmail.com> wrote:
> On Wed, Jul 23, 2008 at 6:48 AM, Dan McGee <dan at archlinux.org> wrote:
>>
>> +if [ ! -x "$bin" ]; then
>> +       echo "vercmp binary ($bin) could not be located"
>> +       exit 1
>> +fi
>> +
>> +echo "Beginning vercmp tests"
>> +
>
> This does not work when vercmptest is run without arguments and $bin
> defaults to vercmp.
> What about doing like in makepkg instead :
> if [ ! $(type -p "$bin") ]; then

Argh, I thought I tested this- good catch.

-Dan




More information about the pacman-dev mailing list