Peter Samuelson <[EMAIL PROTECTED]> writes:

> [Goswin von Brederlow]
>> +# dpkg-vendor
>> +#
>> +# Copyright ?? 2004-2005 Scott James Remnant <[EMAIL PROTECTED]>,
>> +# Copyright ?? 1999 Marcus Brinkmann <[EMAIL PROTECTED]>.
>
> Is it really?

Cut&paste error.

>> +get_vendor() {
>> +    if ! [ -z $DEB_VENDOR ]; then
>> +            echo "$DEB_VENDOR"
>
> You need "" around $DEB_VENDOR or this will not do what you expect.
> Also, [ -n "$foo" ] is simpler than ! [ -z "$foo" ].

I know I had -n at first and it didn't do what I needed while ! -z
did. Can't remember what special case it was though. Might have been
the difference between "$foo" and $foo, i.e. only whitespaces in $foo.

>> +get_vendor_file() {
>> +    if ! [ -z $DEB_VENDOR ]; then
>> +            grep -l "^Vendor: $DEB_VENDOR$" /etc/dpkg/origins/*
>
>> +get_build_options() {
>> +    if ! [ -z $DEB_BUILD_OPTIONS ]; then
>> +            echo "$DEB_BUILD_OPTIONS"
>
>> +    if ! [ -z $parent ]; then
>
> Same for these 3 places.
> -- 
> Peter Samuelson | org-tld!p12n!peter | http://p12n.org/

MfG
        Goswin



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to