Tickets :: [#4110] Vacation Procmail problem

6.0.0-beta1

10/19/25

Summary	Vacation Procmail problem
Queue	Ingo
Queue Version	1.1.1
Type	Bug
State	Resolved
Priority	2. Medium
Owners	ben (at)
Requester	car (at) cespi (dot) unlp (dot) edu (dot) ar
Created	07/03/2006 (7048 days ago)
Due
Updated	07/07/2006 (7044 days ago)
Assigned	07/04/2006 (7047 days ago)
Resolved	07/07/2006 (7044 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

07/07/2006 04:02:55 PM	ben	Comment #4 State ⇒ Resolved	Reply to this comment
Fixed in CVS. I took a slightly different approach, but it has the same end result.

07/04/2006 06:35:24 PM	ben	Assigned to ben State ⇒ Assigned

07/03/2006 07:01:25 PM	car (at) cespi (dot) unlp (dot) edu (dot) ar	Comment #3	Reply to this comment
I forgot to mention that the problem can be corrected changing line 434 in lib/Script/procmail.php file from ingo's tree directory [Show Quoted Text - 29 lines][Hide Quoted Text] The solution I propose is to change the line: DUMMY=`test $FILEDATE -le $DATE && rm -f 'vacation.user@domain.tld'` by DUMMY=`test ${FILEDATE:-0} -le $DATE && rm -f 'vacation.user@domain.tld'` When the .procmailrc script is generated it does not consider the first case when the file vacation.user@domain.tld it does not exists. The generated script looks like: :0 { FILEDATE=`ls -lcn --time-style=+%s 'vacation.user@domain.tld' \| awk '{ print $6 + (604800) }'` DATE=`date +%s` DUMMY=`test $FILEDATE -le $DATE && rm -f 'vacation.user@domain.tld'` :0 Whc: vacation.lock * $^To:.user@domain.tld !^X-Loop: user@domain.tld * !^FROM_DAEMON \| formail -rD 8192 vacation.user@domain.tld :0 ehc \| (formail -rI"Precedence: junk" \ -A"X-Loop: user@domain.tld" \ -i"Subject: Vacation test" ; \ echo "I'm not at office" \ ) \| $SENDMAIL -oi -t }

07/03/2006 06:46:52 PM	car (at) cespi (dot) unlp (dot) edu (dot) ar	Comment #2	Reply to this comment
The solution I propose is to change the line: DUMMY=`test $FILEDATE -le $DATE && rm -f 'vacation.user@domain.tld'` by DUMMY=`test ${FILEDATE:-0} -le $DATE && rm -f 'vacation.user@domain.tld'` [Show Quoted Text - 23 lines][Hide Quoted Text] When the .procmailrc script is generated it does not consider the first case when the file vacation.user@domain.tld it does not exists. The generated script looks like: :0 { FILEDATE=`ls -lcn --time-style=+%s 'vacation.user@domain.tld' \| awk '{ print $6 + (604800) }'` DATE=`date +%s` DUMMY=`test $FILEDATE -le $DATE && rm -f 'vacation.user@domain.tld'` :0 Whc: vacation.lock * $^To:.user@domain.tld !^X-Loop: user@domain.tld * !^FROM_DAEMON \| formail -rD 8192 vacation.user@domain.tld :0 ehc \| (formail -rI"Precedence: junk" \ -A"X-Loop: user@domain.tld" \ -i"Subject: Vacation test" ; \ echo "I'm not at office" \ ) \| $SENDMAIL -oi -t }

07/03/2006 06:44:33 PM	car (at) cespi (dot) unlp (dot) edu (dot) ar	Comment #1 State ⇒ Unconfirmed Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ Vacation Procmail problem Queue ⇒ Ingo	Reply to this comment
When the .procmailrc script is generated it does not consider the first case when the file vacation.user@domain.tld it does not exists. The generated script looks like: :0 { FILEDATE=`ls -lcn --time-style=+%s 'vacation.user@domain.tld' \| awk '{ print $6 + (604800) }'` DATE=`date +%s` DUMMY=`test $FILEDATE -le $DATE && rm -f 'vacation.user@domain.tld'` :0 Whc: vacation.lock * $^To:.user@domain.tld !^X-Loop: user@domain.tld * !^FROM_DAEMON \| formail -rD 8192 vacation.user@domain.tld :0 ehc \| (formail -rI"Precedence: junk" \ -A"X-Loop: user@domain.tld" \ -i"Subject: Vacation test" ; \ echo "I'm not at office" \ ) \| $SENDMAIL -oi -t }