[pacman-dev] [PATCH] scripts/*.sh.in: Clean up and fix a few bugs

[Andrew Fyfe]

Dan McGee wrote:
> On 8/29/07, Andrew Fyfe <andrew at neptune-one.net> wrote:
>> Ignore this patch there's a small bug in repo-add, I'm sending the fixed
>> patch in a second.
>>
>> Andrew
> 
> A few comments on this patch:
> 1. If we use the message functions in repo-add/repo-remove and
> friends, then we should honor the color preference set in makepkg.conf
> since we source it (correct me if I'm wrong).
Removed colours for now.

> 2. At some point we should put all these shell script functions we
> reuse (msg2, error, etc.) into wherever the FHS says they belong so we
> aren't duplicating so much code, and have each script source them.
> (Similar to the initscripts /etc/rc.d/functions file).
/usr/lib/pacman?

> 3. In repo-add, you used both error() and the text "ERROR". This will
> result in it getting printed twice.
Fixed
> 4. This didn't look quite right (missing an '%s'):
> -	printf "$(gettext "ERROR: /etc/makepkg.conf not found. Can not
> continue.")\n" >&2
> +	error "$(gettext "/etc/makepkg.conf not found. Can not continue.")"
> "@sysconfdir@/makepkg.conf"
> 
Fixed

The patch is on my ready_to_pull branch.

Andrew