Bug 254495 - mail/sendmail: overhaul cyruslookup.patch
Summary: mail/sendmail: overhaul cyruslookup.patch
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Dirk Meyer
URL:
Keywords: feature, patch
Depends on:
Blocks:
 
Reported: 2021-03-23 06:14 UTC by Norikatsu Shigemura
Modified: 2021-03-25 03:48 UTC (History)
0 users

See Also:
dinoex: maintainer-feedback+

Attachments
Patches for Makefile and files/cyruslookup.patch (9.06 KB, patch)
2021-03-23 06:14 UTC, Norikatsu Shigemura 		no flags Details | Diff
Patch for Makefile (8.93 KB, patch)
2021-03-23 06:32 UTC, Norikatsu Shigemura 		no flags Details | Diff
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 Norikatsu Shigemura 2021-03-23 06:14:00 UTC 
Created attachment 223513 [details]
Patches for Makefile and files/cyruslookup.patch

Hi dinoex.

I recently built two environments for cyrus-imapd and sendmail.
At that time, I found some bugs in cyruslookup.patch and fixed
them.  I found the following bug in cyruslookup.patch:

 * A socket file to smmapd does not match current cyrus-imapd's provided it.

 * FEATURE (mrs_cyrus) requires the definition of MAILER (cyrusv2d), but
   it does not exist.  According to the (*) site, New release (B) of cyrusv2
   mailer needs to be installed,  The original sendmail has an older version
   installed.

  (*) http://jmaimon.com/sendmail/anfi.homeunix.net/sendmail/rtcyrus2.html
      http://jmaimon.com/sendmail/anfi.homeunix.net/sendmail/cyrusv2.html

 * In the old version, error handling fails when there is no user in
   cyrus-imapd.  We only need to write "example.org  mrs_cyrus_mailertable:"
   in the /etc/mail/mailertable, but must write
   "example.org  mrs_cyrus_mailertable:error:5.1.1:550 User unknown" :-(.

The attached patch fixes the above problem. 

P.S.
  Makefile installs "*.orig"s in cf directories, so I remove them.
Comment 1 Norikatsu Shigemura 2021-03-23 06:32:43 UTC 
Created attachment 223514 [details]
Patch for Makefile

Oops, sorry.
I missed take care of pkg-plist.  So I re-fixed Makefile.
Comment 2 Dirk Meyer freebsd_committer freebsd_triage 2021-03-25 03:39:45 UTC 
Looks fine.
Comment 3 commit-hook freebsd_committer freebsd_triage 2021-03-25 03:41:23 UTC 
A commit references this bug:

Author: dinoex
Date: Thu Mar 25 03:40:32 UTC 2021
New revision: 569183
URL: https://svnweb.freebsd.org/changeset/ports/569183

Log:
  - update option CYRUSLOOKUP
  PR:		254495
  Submitted by:	nork(at)inth-nine.com

Changes:
  head/mail/sendmail/Makefile
  head/mail/sendmail/files/cyruslookup.patch