On Thu, Feb 25, 2010 at 3:36 AM, Eric Bélanger <[email protected]> wrote: <snip> > for arch in ${arch...@]}; do > > + IS_LOCKED=0 > + count=0 > + while [ $count -le $LOCK_TRIAL ]; do > + if repo_lock $reponame $arch ; then > + IS_LOCKED=1 > + let count=$LOCK_TRIAL+1 > + continue
I'd just use a 'break' here, instead of making the conditional expression false and jumping to the check again. > + fi > + sleep $LOCK_DELAY > + let count=$count+1 > + done > + > + if [ $IS_LOCKED -eq 0 ]; then <snip> Thanks for creating this patch. :)
