On Thu, Aug 02, 2007 at 01:16:56PM +0200, DI Roman Fiedler wrote:
> Possible patch??
>
> host:~/Tmp$ diff -U 3 /tmp/savelog /usr/bin/savelog
> --- /tmp/savelog 2007-08-01 12:54:33.000000000 +0200
> +++ /usr/bin/savelog 2007-03-05 06:43:41.000000000 +0100
> @@ -194,11 +194,7 @@
>        if [ -z "$savedir" ]; then
>                savedir=.
>        fi
> - if [ "${rolldir%%/*}" = "" ] ; then
> - savedir=$rolldir
> - else
> - savedir="$savedir/$rolldir"
> - fi
> + savedir="$savedir/$rolldir"
>        if [ ! -d "$savedir" ]; then
>                mkdir -p -- "$savedir"
>                if [ "$?" -ne 0 ]; then

I'll change the string comparison (in your way, -r "" will cause savedir
to be set to the empty string, whereas now it would be set to
"$savedir/").  Other than that it seems like a good idea.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to