The dsn patch included in the pflogsumm-1.10-r1 ebuild is incorrect. The "dsn" portion is missing a space, causing the "delivered" section of the pflogsumm output to be zero. ebuild patch contains this line: + /to=<([^>]*)>, (?:orig_to=<[^>]*>, )?relay=([^,]+), (?:conn_use= [^,]+, )?delay=([^,]+), (?:delays=[^,]+, )?(?:dsn=[^,]+,)?status=(\S+)(.*)$/o) > = 4) dsn patch included with postfix-2.3.6 has this line: + /to=<([^>]*)>, (?:orig_to=<[^>]*>, )?relay=([^,]+), (?:conn_use= [^,]+, )?delay=([^,]+), (?:delays=[^,]+, )?(?:dsn=[^,]+, )?status=(\S+)(.*)$/o) >= 4) The error lies in the dsn portion. The ebuild patch says: ... (?:dsn=[^,]+,) ... and it should be: ... (?:dsn=[^,]+, ) ... The space between the comma and the end parenthesis makes all the difference in the world. Reproducible: Always Steps to Reproduce: 1. emerge =pflogsumm-1.1.0-r1 2. run pflogsumm 3. check output for delivered count Actual Results: see link http://forums.gentoo.org/viewtopic-t-539754-highlight-.html Expected Results: delivered count should not be zero
Please, use attachments instead of pasting patches inline... It completely kills formatting and makes the thing unreadable.
(In reply to comment #1) > Please, use attachments instead of pasting patches inline... It completely > kills formatting and makes the thing unreadable. > Sorry about that. I'll do that in the future, however, it's not a patch, it's just a portion of the patch - the relevant part. Is it necessary for me to resubmit as an attachment(s), or is the post currently clear enough to resolve the issue?
Created attachment 111633 [details, diff] corrected patch
Fixed in CVS, and forced on users by a -r2 revision bump. Kept stable keywords, since this is a trivial bugfix. Thanks for your time, Kevin!
