* René van Bevern <[EMAIL PROTECTED]> [May 26. 2005 12:11]:
> On 26.05.05, Frederik Juul Christiani wrote:
> > The /usr/bin/rem script lets the shell expand the repeat parameter.
> 
> Upstream has decided to drop "rem" from future versions of
> remind.  Instead, "remind" will use "~/.reminders" as reminder
> file if nothing else is given at the command line. (instead of
> resulting in an error)

That does sound like the sensible thing to do.

> I will probably fix the bug by obsoleting "rem" instead of
> patching it once again.

Very understandable.
However, if you choose to patch it, you should probably use this
revised patch instead, since the first one breaks if no
parameter is given. Apparently remind doesn't take to kindly to
an empty parameter.

$ echo | remind -
No reminders.

vs.

$ echo | remind - ""

REMIND 03.00.22 (English version) Copyright 1992-1998 David F.  Skoll
Copyright 1999-2000 Roaring Penguin Software Inc.
Usage: remind [options] filename [date] [time] [*rep]
Options:
 -n     Output next occurrence of reminders in simple format
 -r     Disable RUN directives
<snip>


Sorry for not testing the first patch. I really hope this one
works.


--- /usr/bin/rem        2005-05-05 10:13:09.000000000 +0200
+++ rem 2005-05-26 13:01:08.496475259 +0200
@@ -42,4 +42,11 @@
        esac
 done
 
-$EXECUTABLE "[EMAIL PROTECTED]" $DEFAULT $PARAMETERS
+if [ "$PARAMETERS" != "" ]; then
+  $EXECUTABLE "[EMAIL PROTECTED]" $DEFAULT "$PARAMETERS"
+else
+  $EXECUTABLE "[EMAIL PROTECTED]" $DEFAULT
+fi
+

-- 
Frederik



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

Reply via email to