New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #35104 (closed enhancement: fixed)

Opened 11 months ago

Last modified 9 months ago

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

Reported by: dluke@… Owned by: pixilla@…
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

opendkim-configure.ac.diff (337 bytes) - added by dluke@… 11 months ago.
configure.ac patch for opendkim 2.6.2 on darwin9
opendkim_2.6.3.diff (905 bytes) - added by dluke@… 11 months ago.
version bump to 2.6.3, add test target
opendkim_2.6.4.diff (905 bytes) - added by dluke@… 10 months ago.
2.6.4 upgrade
opendkim_2.6.6.diff (961 bytes) - added by dluke@… 10 months ago.
upgrade to 2.6.6, enable port test

Change History

Changed 11 months ago by dluke@…

configure.ac patch for opendkim 2.6.2 on darwin9

comment:1 Changed 11 months ago by dluke@…

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 11 months ago by dluke@…

version bump to 2.6.3, add test target

comment:2 Changed 11 months ago by dluke@…

  • Keywords haspatch added

comment:3 Changed 10 months ago by dluke@…

  • Summary changed from fix for opendkim crashing on darwin 9 to Opendkim version bump (was fix for opendkim crashing on darwin 9)

comment:4 Changed 10 months ago by dluke@…

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

Changed 10 months ago by dluke@…

2.6.4 upgrade

comment:5 Changed 10 months ago by dluke@…

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 10 months ago by dluke@…

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 10 months ago by dluke@…

upgrade to 2.6.6, enable port test

comment:7 Changed 10 months ago by dluke@…

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

comment:8 Changed 9 months ago by pixilla@…

  • Status changed from new to closed
  • Resolution set to fixed

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 9 months ago by dluke@…

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.