[pacman-dev] [PATCH] makepkg: tidy version package tests
Dan McGee
dpmcgee at gmail.com
Thu Jan 1 23:54:50 EST 2009
On Thu, Jan 1, 2009 at 10:44 PM, Allan McRae <allan at archlinux.org> wrote:
> 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).
OK we'll blame the late release on you then! :)
As long as we aren't going to have more message updates, I can at
least try to round up the translations sometime this week and then
hopefully solidify what will go in the package.
-Dan
More information about the pacman-dev
mailing list