modules/plague/templates/sync-committers.sh.erb Change

[Ricky Zhou]

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:"

Thanks,
Ricky

[Mike McGrath]

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