FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Infrastructure

 
 
LinkBack Thread Tools
 
Old 09-13-2008, 08:09 PM
Ricky Zhou
 
Default modules/plague/templates/sync-committers.sh.erb Change

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
_______________________________________________
Fedora-infrastructure-list mailing list
Fedora-infrastructure-list@redhat.com
https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
 
Old 09-13-2008, 11:18 PM
Mike McGrath
 
Default modules/plague/templates/sync-committers.sh.erb Change

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
 

Thread Tools




All times are GMT. The time now is 05:58 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org