Opened 11 years ago

Closed 10 years ago

#24731 closed submission (wontfix)

db50: new port of Berkeley DB version 5.0.21

Reported by: pixilla (Bradley Giesbrecht) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 1.8.2
Keywords: Cc: blair (Blair Zajac), MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Port: db50

Description

I saw on unix-porting@… some talking about Berkeley DB version 5.0.21. Since there is no db50 in macports I decided to create on as an exercise. I don't use bdb directly so I don't know how well this works. I created a patch for Makefile.in that fixes a couple configure options that are ignored making for a nicer install.

Also configure checks for sed but Makefile.in only uses it once leaning on a hardcoded "sed" command elsewhere which seemed dumb but maybe there is a reason. I'm thinking if the patch works well here I may offer it upstream.

Mostly I just want some critical review by MP's committers as to the quality of the Portfile, if time allows:)

Thanks, Bradley Giesbrecht

Attachments (2)

Portfile (2.8 KB) - added by pixilla (Bradley Giesbrecht) 11 years ago.
patch-dist-Makefile.in.diff (1.6 KB) - added by pixilla (Bradley Giesbrecht) 11 years ago.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by pixilla (Bradley Giesbrecht)

Attachment: Portfile added

Changed 11 years ago by pixilla (Bradley Giesbrecht)

Attachment: patch-dist-Makefile.in.diff added

comment:1 Changed 11 years ago by ryandesign (Ryan Schmidt)

Cc: blair@… mcalhoun@… added
Port: db50 added
Type: enhancementsubmission

Perhaps our existing db46 and db47 maintainers could take a look. Ideally, all the BerkeleyDB ports would be kept similar and maintained by the same maintainer(s).

My comments after glancing at your submission are that the epoch and revision lines should be deleted, and I don't think there's any particular need to replace "sed" with "$(SED)" in the Makefile.

comment:2 Changed 10 years ago by pixilla (Bradley Giesbrecht)

Resolution: wontfix
Status: newclosed

Closing due to inactivity

Note: See TracTickets for help on using tickets.