Bug 191101 - [PATCH] dns/p5-Net-DNS: fix broken functionality that breaks SpamAssassin!
Summary: [PATCH] dns/p5-Net-DNS: fix broken functionality that breaks 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 Many People
Assignee: Adam Weinberger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-16 19:32 UTC by Daniel Austin
Modified: 2014-06-20 18:29 UTC (History)
2 users (show)

See Also:

Attachments
20140616-p5-Net-DNS.diff (914 bytes, patch)
2014-06-16 19:32 UTC, Daniel Austin 		no flags Details | Diff
diff for mail/spamassassin (1.71 KB, patch)
2014-06-20 08:14 UTC, Kurt Jaeger 		no flags Details | Diff
patch for japanese/spamassassin (477 bytes, patch)
2014-06-20 08:14 UTC, Kurt Jaeger 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Austin 2014-06-16 19:32:28 UTC 
Created attachment 143846 [details]
20140616-p5-Net-DNS.diff

A bug in the rushed release of 0.76 broke SpamAssassin (and presumably others) processing.

Fix:
Patch below updates to 0.77 and fixes the errors in SpamAssassin.

Maintainer CC'd
Comment 1 Daniel Austin 2014-06-16 19:33:35 UTC 
Actually, this doesn't seem to have fixed things *sigh*

I assume we'll need to wait for 0.78 (or revert back to 0.75)
Comment 2 Kurt Jaeger freebsd_committer freebsd_triage 2014-06-19 19:33:55 UTC 
See also 189912, 191082
Comment 3 Kurt Jaeger freebsd_committer freebsd_triage 2014-06-20 08:14:21 UTC 
Created attachment 143950 [details]
diff for mail/spamassassin

this and the patch for japanese/spamassassin should fix the problem. please test and approve.
Comment 4 Kurt Jaeger freebsd_committer freebsd_triage 2014-06-20 08:14:44 UTC 
Created attachment 143951 [details]
patch for japanese/spamassassin
Comment 5 Adam Weinberger freebsd_committer freebsd_triage 2014-06-20 18:06:12 UTC 
These diffs are just PORTREVISION bumps. Is that all that is needed?
Comment 6 Daniel Austin 2014-06-20 18:08:27 UTC 
I see a new patch in the files folder in the diff - but the BZ web diff doesn't show it.  Works if you download though.

(i've not tested Kurt's patch, just commenting!)
Comment 7 Kurt Jaeger freebsd_committer freebsd_triage 2014-06-20 18:12:52 UTC 
(In reply to Adam Weinberger from comment #5)
> These diffs are just PORTREVISION bumps. Is that all that is needed?

The mail/spamassassin has one little files/patch-... file included.
The bugzilla webinterface does not show this file (TODO: needs a PR about this).

The japanese/spamassassin is a slave port and only needs a revision bump.
Comment 8 Kurt Jaeger freebsd_committer freebsd_triage 2014-06-20 18:16:39 UTC 
(In reply to Kurt Jaeger from comment #7)
> (In reply to Adam Weinberger from comment #5)
> > These diffs are just PORTREVISION bumps. Is that all that is needed?
> 
> The mail/spamassassin has one little files/patch-... file included.
> The bugzilla webinterface does not show this file (TODO: needs a PR about
> this).

There already is a bug report:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=190724
Comment 9 Adam Weinberger freebsd_committer freebsd_triage 2014-06-20 18:29:54 UTC 
Thank you for preparing this patch! It is now committed.