Bug 201241
| Summary: | [PATCH] mail/opensmtpd: Update to 5.7.1p1 | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Bernard Spil <brnrd> | ||||||
| Component: | Individual Port(s) | Assignee: | Ashish SHUKLA <ashish> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Only Me | CC: | andrey, xmj | ||||||
| Priority: | --- | Keywords: | patch | ||||||
| Version: | Latest | Flags: | ashish:
maintainer-feedback+
|
||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| Attachments: |
|
||||||||
Hi, Sorry for the delay. The diff looks good to me, except I don't understand the point of moving CA_BUNDLE_* lines, and also it's a typo in the name of ca_root_nss port, so I've just ignored that part. Other than it looks perfect, works fine on my 10.x host, thus have my approval. I'll do a test build and commit it. Thanks for working on it. Ashish Hi Ashish, Not a problem, not in a hurry! As far as I know, options should be alphabetically sorted, that's the only reason for the re-ordering. Just now I ran into an issue with 5.7.1, I don't know (yet) what caused it but some QA is in place. I had enabled PAM and that was working before with 5.4.6. With 5.7.1 I got 501 5.5.2 Syntax error: Syntax error whatever I did. Reverting back to 5.4.6 solved that (but was complaining that /var/spool/smtpd/purge wasn't 700) Kind regards, Bernard Created attachment 158379 [details]
poudrier build log of mail/opensmtpd
(In reply to Bernard Spil from comment #2) > Just now I ran into an issue with 5.7.1, I don't know (yet) what caused it but some QA is in place. I had enabled PAM and that was working before with 5.4.6. With 5.7.1 I got > 501 5.5.2 Syntax error: Syntax error > whatever I did. Reverting back to 5.4.6 solved that (but was complaining that /var/spool/smtpd/purge wasn't 700) Hi Bernard, Could you provide more details on this issue, or your configuration, so I can try to reproduce it my end ? If configuration is not suitable for public report, then can email to ashish@ . Thanks! Ashish Random and offtopic: Can someone reproduce this behavior with 5.7.1, or was it fixed already? https://github.com/OpenSMTPD/OpenSMTPD/issues/617 A commit references this bug: Author: ashish Date: Wed Jul 15 15:53:22 UTC 2015 New revision: 392161 URL: https://svnweb.freebsd.org/changeset/ports/392161 Log: - Update to 5.7.1p1 - Add TABLE_DB option to add support for aliases db - Sort options in alphabetical order PR: 201241 Submitted by: brnrd Changes: head/mail/opensmtpd/Makefile head/mail/opensmtpd/distinfo head/mail/opensmtpd/pkg-plist Thanks for providing the diff, it's committed now. (In reply to Johannes Jost Meixner from comment #5) > Can someone reproduce this behavior with 5.7.1, or was it fixed already? This issue is reproducible with 5.7.1p1 as well. Thanks for reporting to upstream. |
Created attachment 158210 [details] svn diff for mail/opensmtpd Hi Ashish, Created a patch to update OpenSMPTd to 5.7.1 table_db is now a configure knob so added as an option and using OPTIONS_SUB for the files.