modules/plague/templates/sync-committers.sh.erb Change
Mike McGrath
mmcgrath at redhat.com
Sat Sep 13 23:18:00 UTC 2008
On Sat, 13 Sep 2008, Ricky Zhou wrote:
> Sorry, I completely forgot we were in a change freeze now. I just
> pushed a change (diff below) to make the sync-committers script not
> fail. Should I revert this, or does it look all right?
>
> diff --git a/modules/plague/templates/sync-committers.sh.erb b/modules/plague/templates/sync-committers.sh.erb
> index d3c8e63..4e51d44 100644
> --- a/modules/plague/templates/sync-committers.sh.erb
> +++ b/modules/plague/templates/sync-committers.sh.erb
> @@ -12,7 +12,7 @@ fi
>
> for user in `cat ${users}|cut -d, -f2`
> do
> - plague-user-manager.py ${userdb} add $user own_jobs > $cruft 2>&1
> + /usr/bin/plague-user-manager ${userdb} add $user own_jobs > $cruft 2>&1
> if [ $? != 0 ]; then
> if ! grep -qi 'already exists' $cruft; then
> echo "Error adding user $user was:"
>
I'll +1 it just in case. This could be considered an outage so I think no
harm done.
-Mike
More information about the infrastructure
mailing list