Bug 139962 - [Patch] mail/vpopmail: fix maildir++ quota with spamassassin
Summary: [Patch] mail/vpopmail: fix maildir++ quota with spamassassin
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Peter Pentchev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-25 16:00 UTC by Alex Dupre
Modified: 2009-11-30 12:14 UTC (History)
0 users

See Also:


Attachments
vpopmail.diff (4.15 KB, patch)
2009-10-25 16:00 UTC, Alex Dupre
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Dupre freebsd_committer freebsd_triage 2009-10-25 16:00:08 UTC
When spamassassin is used, the Maildir++ S=nnn value is different from the actual message size,
since SA modify the original message and the filename is constructed before.
When I initially wrote the patch I didn't care, but actually it's better to be coherent here
and set the correct message size.

Fix: Stat the 'tmp' file after processing and use the real size to create the 'new' filename.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2009-10-25 16:00:25 UTC
Responsible Changed
From-To: freebsd-ports-bugs->roam

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Peter Pentchev freebsd_committer freebsd_triage 2009-11-27 09:07:01 UTC
State Changed
From-To: open->analyzed

Whups, I missed this one in the 5.4.28 update.  Sorry, I'll take care of 
it ASAP now.
Comment 3 dfilter service freebsd_committer freebsd_triage 2009-11-30 12:09:44 UTC
roam        2009-11-30 12:09:31 UTC

  FreeBSD ports repository

  Modified files:
    mail/vpopmail        Makefile 
    mail/vpopmail/files  patch-vdelivermail.c 
  Log:
  Fix the master sites so that vpopmail is actually fetchable even with
  -A present in FETCH_ARGS, as it is on most systems.
  Fix the maildir++ file size calculations in the SpamAssassin code as
  per the PR.
  
  PR:             139962 (the maildir++ fix)
  Submitted by:   Alex Dupre <ale@freebsd.org>
  
  Revision  Changes    Path
  1.78      +3 -1      ports/mail/vpopmail/Makefile
  1.13      +71 -8     ports/mail/vpopmail/files/patch-vdelivermail.c
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 4 Peter Pentchev freebsd_committer freebsd_triage 2009-11-30 12:13:55 UTC
State Changed
From-To: analyzed->closed

I've committed your fix, adapting it slightly to the refreshed patch. 
Thanks!