[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: cvs.sh patch
From:       Adriaan de Groot <groot () kde ! org>
Date:       2004-09-25 21:00:43
Message-ID: 200409252300.44080.groot () kde ! org
[Download RAW message or body]

On Saturday 25 September 2004 20:31, Anatoli Gorchetchnikov wrote:
> cvs.sh seems to be broken. The following patch fixed it for me, but I am no
> scripting guru, and I don't seem to have package* files...
>
> Index: cvs.sh
> ===================================================================
> RCS file: /home/kde/kde-common/admin/cvs.sh,v
> retrieving revision 1.130.2.2
> diff -u -3 -p -r1.130.2.2 cvs.sh
> --- cvs.sh      17 Sep 2004 00:02:29 -0000      1.130.2.2
> +++ cvs.sh      24 Sep 2004 22:23:05 -0000
> @@ -647,7 +647,7 @@ fi
>  ### Main
>  ###
>
> -arg=`echo $1 | tr '\-.' __`
> +arg=`echo $1 | tr .- _`

That looks reasonable to me, just to avoid the possibility of the - ending up 
interpreted as an option indicator, though you'll perhaps need two 
underscores there for it to tr to.

CC-ing core-devel, where the folks with clout to commit this live.

-- 
   KPilot - www.kpilot.org - HotSync Solutions for KDE
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]