Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#35104 closed enhancement (fixed)

Opendkim version bump (was fix for opendkim crashing on darwin 9)

Reported by: danielluke (Daniel J. Luke) Owned by: pixilla (Bradley Giesbrecht)
Priority: Normal Milestone:
Component: ports Version: 2.1.1
Keywords: haspatch Cc:
Port: opendkim

Description

I've been running opendkim (2.6.2) outside of MacPorts on a Darwin 9 (ppc) box.

It appears that the resolver on darwin9 isn't thread safe, so opendkim crashes a lot (and even manages to kernel panic the machine).

I've got a simple patch to configure.ac that fixes it there. I'm not sure about the thread safety of the resolver on later darwin releases, though (this might be needed for more recent OS releases as well).

Attachments (4)

opendkim-configure.ac.diff (337 bytes) - added by danielluke (Daniel J. Luke) 12 years ago.
configure.ac patch for opendkim 2.6.2 on darwin9
opendkim_2.6.3.diff (905 bytes) - added by danielluke (Daniel J. Luke) 12 years ago.
version bump to 2.6.3, add test target
opendkim_2.6.4.diff (905 bytes) - added by danielluke (Daniel J. Luke) 12 years ago.
2.6.4 upgrade
opendkim_2.6.6.diff (961 bytes) - added by danielluke (Daniel J. Luke) 12 years ago.
upgrade to 2.6.6, enable port test

Download all attachments as: .zip

Change History (13)

Changed 12 years ago by danielluke (Daniel J. Luke)

Attachment: opendkim-configure.ac.diff added

configure.ac patch for opendkim 2.6.2 on darwin9

comment:1 Changed 12 years ago by danielluke (Daniel J. Luke)

Something very similar to this patch was accepted upstream and is in the current (2.6.3) release.

I'm attaching a patch to update the port to 2.6.3.

Changed 12 years ago by danielluke (Daniel J. Luke)

Attachment: opendkim_2.6.3.diff added

version bump to 2.6.3, add test target

comment:2 Changed 12 years ago by danielluke (Daniel J. Luke)

Keywords: haspatch added

comment:3 Changed 12 years ago by danielluke (Daniel J. Luke)

Summary: fix for opendkim crashing on darwin 9Opendkim version bump (was fix for opendkim crashing on darwin 9)

comment:4 Changed 12 years ago by danielluke (Daniel J. Luke)

... and of course, 2.6.4 was released today, attaching a new patch for it.

Changed 12 years ago by danielluke (Daniel J. Luke)

Attachment: opendkim_2.6.4.diff added

2.6.4 upgrade

comment:5 Changed 12 years ago by danielluke (Daniel J. Luke)

It actually looks like the upstream version of the fix doesn't work correctly, so we probably want to add --enable-arlib to the configure arguments until there's a new upstream release that fixes it.

comment:6 Changed 12 years ago by danielluke (Daniel J. Luke)

2.6.6 was released today and includes the build fix (to automatically add arlib to the build on Darwin). I'm attaching a new patch.

Changed 12 years ago by danielluke (Daniel J. Luke)

Attachment: opendkim_2.6.6.diff added

upgrade to 2.6.6, enable port test

comment:7 Changed 12 years ago by danielluke (Daniel J. Luke)

2.6.7 released today, patch generation now an exercise for the reader ...

comment:8 Changed 12 years ago by pixilla (Bradley Giesbrecht)

Resolution: fixed
Status: newclosed

See r96895

dluke: Thanks. I add openmaintainer to the port. Should configure(.ac) still need patching let me know or have at it yourself.

comment:9 Changed 12 years ago by danielluke (Daniel J. Luke)

2.6.7 is good, been running it without patching for a while on my darwin9/ppc box with no issues.

Note: See TracTickets for help on using tickets.