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
_______________________________________________
Fedora-infrastructure-list mailing list
Fedora-infrastructure-list@redhat.com
https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
