[pacman-dev] [PATCH] makepkg: tidy version package tests

Allan McRae allan at archlinux.org
Thu Jan 1 23:44:49 EST 2009


Dan McGee wrote:
> On Thu, Jan 1, 2009 at 10:37 PM, Allan McRae <allan at archlinux.org> wrote:
>   
>> The use if "! -z" to check if a string is not null is not good practice
>> so replace with the "-n" option. Also use the AND comparison within one
>> test rather than on two separate tests.
>>
>> Signed-off-by: Allan McRae <allan at archlinux.org>
>>     
>
> Looks good to me, maint- worthy?
>   

I'm not worried on this one really.  It is not an actual problem, just a 
potential one so up to you.

> I really need to jump on making this darnded release, don't I..

We really need to get a fix for that whole umask issue first (now we 
have figured out the cause).

Allan





More information about the pacman-dev mailing list