Bug 193693
| Summary: | mail/dspam: Merge mail/dspam-devel in to mail/dspam | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | danny | ||||||||||||
| Component: | Individual Port(s) | Assignee: | Kurt Jaeger <pi> | ||||||||||||
| Status: | Closed FIXED | ||||||||||||||
| Severity: | Affects Some People | CC: | cs, hsn, ohauer, pi | ||||||||||||
| Priority: | --- | ||||||||||||||
| Version: | Latest | ||||||||||||||
| Hardware: | Any | ||||||||||||||
| OS: | Any | ||||||||||||||
| Attachments: |
|
||||||||||||||
|
Description
danny
2014-09-16 20:51:56 UTC
Created attachment 147393 [details]
dspam-merge-20140916 (IN-PROGRESS)
Attaching an in-progress patch for merging dspam-devel in to dspam, updating to dspam 3.10.2 and rolling up all other applicable open issues.
This is not ready to be applied as I am still testing and rectifying some discrepancies between dspam and dspam-devel.
However, I wanted to get what I had done out there since this is probably the bulk of the bigger changes.
Created attachment 147430 [details]
dspam-merge-20140917 (IN-PROGRESS)
Comment on attachment 147430 [details] dspam-merge-20140917 (IN-PROGRESS) Continuing work on rollup patch for merge: * including more cleanup changes from Olli Hauer's patch from bug #148485 * including changelog with credit for patch submitters (need to hunt and peck through svn trees for both dspam and dspam-devel to find any that aren't based on PRs) * including all upstream changes since 3.9.0 in changelog * merged in changes to rc.d/dspam * cleaned up etc/dspam.conf.sample * simplified pkg-message * fixes from build testing Created attachment 147461 [details] dspam-merge-20140918 Latest progress on merge patch: * sticking mostly to cosmetic changes here, as I spent most of the day testing the build and don't want to shake it up too much * clarified a few descriptions on makefile options * misc doc tweaks Moved the patch over to github, located here: https://github.com/dannywarren/FreeBSD-Ports/tree/pr_193693/mail/dspam So that I could push changes to redports for verification, as seen here: https://redports.org/buildarchive/20140919070550-39414/ I did some scratch testing of the build in a test environment, and the options I tested compiled correctly. All of the fixes we merged in from dspam-devel seem to be intact as well. I also installed this in place of dspam-devel on one of my hosted mail servers, completely wiped my dspam setup, trained and configured it from scratch. Working great! The only two failures I am seeing from redports don't seem to be related to mail/dspam (though I should probably report them both). I think we might be ready to commit this (though I will probably do another round of checking since this is a big messy merge). So, we need to figure out: 1) Who wants to do the commit (I will email Carlo Strub tomorrow and ask) 2) What is the cleanest way to obsolete the mail/dspam-devel plugin (I will double check the porters handbook tomorrow) 3) If any ports depend on dspam-devel Created attachment 147628 [details] dspam-merge-20140923 Here is the latest merge patch, I think we might be ready to commit at this point. This includes a small fix for an incorrect pidfile location introduced in dspam 10.3.2 upstream. I'm getting really strange results from redports, at this point I am going to assume that not seeing a major obvious build failure means that it is ok. Examples: My uncommitted patch: https://redports.org/buildarchive/20140924025042-25991/ Running test again on same code, different result from redports: https://redports.org/buildarchive/20140924035335-38166/ Known good version of mail/dspam, same random fails from redports: https://redports.org/buildarchive/20140924031800-49163/ I verified that no other ports depend on mail/dspam-devel, so we should be save to remove it after the merge. Proposed entry to /usr/ports/MOVED: mail/dspam-devel|mail/dspam|2014-09-23|Merged into mail/dspam Just wanted to note that I will be unavailable until Sunday 10/6, just in case there is some movement with this and I'm not around to respond. When I get back I will bug Carlo Strub again about getting this mess committed, or check with the ports mailing list. Comment on attachment 147628 [details]
dspam-merge-20140923
Adding maintainer approval flag to dspam-merge-20140923 patch
No contact from Carlo Strub for almost a month, I will contact the ports mailing list and see if anyone is willing to help get this mess committed. Created attachment 148490 [details]
dspam-3.10.2-10-amd64.log
Attaching successful poudriere log for the above 20140923 patch, build with 10.0-RELEASE/amd64.
It also passed against 10.0-RELEASE/i386 and 11-CURRENT/amd64.
I made test-builds on poudriere, builds fine. Logs see: http://people.freebsd.org/~pi/logs/mail__dspam-10a-1413782576.txt http://people.freebsd.org/~pi/logs/mail__dspam-91a-1413782576.txt http://people.freebsd.org/~pi/logs/mail__dspam-84i-1413782576.txt (yes, i fixed the @dir stuff afterwards 8-) From what I understand, 3.10.2 is approx. 2 years older than 3.9.0. I had a look at the changelog and it's huge, but there's no linkable changelog anywhere ? So, how can one summarize the changes between the two versions ? Thanks Kurt! Sounds good on the @dir stuff, I didn't know enough backstory on that to make a call myself. Yea, the main CHANGELOG with all the juicy details lives inside the tarball and they don't publish it to any purposeful spot online. We could link to the CHANGELOG in their svn tree: http://sourceforge.net/p/dspam/code/ci/master/tree/CHANGELOG Or link to their announcement blog posts for each version: http://sourceforge.net/p/dspam/news/2011/08/dspam-3100-released/ http://sourceforge.net/p/dspam/news/2011/08/dspam-3101-released/ http://sourceforge.net/p/dspam/news/2012/04/dspam-3102-released/ Let me know if you would prefer to link to the changes using one of the above, or if we should still attempt to summarize them ourselves. I'll get to work on summarizing in case you still want to do that. Just to double check, did you see the changelog summary in UPDATING? I was assuming you did and you wanted it to be more detailed to use as a commit log, but maybe this works as-is? ########################################################################### # dspam-3.10.2 # Upstream Changes: - Support for Postgresql 9.1 added - Support for Postgresql <8.1 dropped (EOL) - Support for MySQL 5.5 added - Support for MySQL <5.0 dropped - Updated license from GPLv2 to AGPLv3 - Bug in RSET handling during LHLO phase - Location of configuration files in dspam home (/var/dspam) can be changed through dspam.conf - Various fixes to the webui templates - Various small bugs and memleaks - Missing documentation added to tarball - Many bugfixes Port Changes: - Merge mail/dspam-devel and mail/dspam (PR #193693) - Update to OptionsNG - Fix pidfile location discrepancy introduced in 3.10.2 - Port cleanup by various submitters, thanks! [1][2][3][4][5] [1] Patch from <yds@CoolRat.org> (PR #180695, #191610) [2] Patch from Radim Kolar <hsn@sendmail.cz> (PR #167289) [3] Patch from Olli Hauer <ohauer@FreeBSD.org> (PR #148485) [4] Patch from <mtomasek4@gmail.com> (PR #191610) [5] Patch from Rene Ladan <rene@FreeBSD.org> (PR #191797) A commit references this bug: Author: pi Date: Mon Oct 20 18:48:34 UTC 2014 New revision: 371290 URL: https://svnweb.freebsd.org/changeset/ports/371290 Log: mail/dspam: 3.9.0 -> 3.10.2 Upstream Changes: - Support for Postgresql 9.1 added - Support for Postgresql <8.1 dropped (EOL) - Support for MySQL 5.5 added - Support for MySQL <5.0 dropped - Updated license from GPLv2 to AGPLv3 - Bug in RSET handling during LHLO phase - Location of configuration files in dspam home (/var/dspam) can be changed through dspam.conf - Various fixes to the webui templates - Various small bugs and memleaks - Missing documentation added to tarball - Many bugfixes Port Changes: - Update to OptionsNG - Fix pidfile location discrepancy introduced in 3.10.2 - Port cleanup by various submitters, thanks! [1][2][3][4][5] [1] Patch from <yds@CoolRat.org> (PR #180695, #191610) [2] Patch from Radim Kolar <hsn@sendmail.cz> (PR #167289) [3] Patch from Olli Hauer <ohauer@FreeBSD.org> (PR #148485) [4] Patch from <mtomasek4@gmail.com> (PR #191610) [5] Patch from Rene Ladan <rene@FreeBSD.org> (PR #191797) PR: 193693 Submitted by: danny@dannywarren.com (maintainer) Changes: head/mail/dspam/Makefile head/mail/dspam/distinfo head/mail/dspam/files/UPDATING head/mail/dspam/files/dspam.in head/mail/dspam/files/patch-scripts__train.pl head/mail/dspam/files/patch-src__dspam.c head/mail/dspam/files/patch-src__dspam.conf.in head/mail/dspam/pkg-descr head/mail/dspam/pkg-message head/mail/dspam/pkg-plist A commit references this bug: Author: pi Date: Mon Oct 20 18:51:30 UTC 2014 New revision: 371291 URL: https://svnweb.freebsd.org/changeset/ports/371291 Log: mail/dspam-devel: merged with mail/dspam PR: 193693 Submitted by: danny@dannywarren.com (maintainer) Changes: head/MOVED head/mail/Makefile head/mail/dspam-devel/ (In reply to danny from comment #17) > Just to double check, did you see the changelog summary in UPDATING? No, this is perfect as commit message. Thanks. All things committed, thank you very much! Thanks Kurt, I really appreciate the help! *** Bug 177045 has been marked as a duplicate of this bug. *** *** Bug 148485 has been marked as a duplicate of this bug. *** *** Bug 167289 has been marked as a duplicate of this bug. *** |
