Page MenuHomeFreeBSD

lang/ruby2[45]: Remove dbm library and replace it with databases/rubygem-dbm

Authored by meta on Apr 22 2019, 3:40 AM.
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:11 AM
Unknown Object (File)
Sat, Mar 16, 3:29 PM
Unknown Object (File)
Feb 4 2024, 2:47 PM
Unknown Object (File)
Jan 27 2024, 6:46 AM
Unknown Object (File)
Jan 14 2024, 4:24 PM
Unknown Object (File)
Jan 5 2024, 8:51 PM
Unknown Object (File)
Jan 5 2024, 8:48 PM
Unknown Object (File)
Dec 22 2023, 1:06 AM


PR:		230289
Submitted by:	Yasuhiro KIMURA <> (initial revision)
		meta (myself)
Reported by:
#Reviewed by:
Approved by:
Obtained from:
MFC after:
Sponsored by:
#Differential Revision:

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

meta added a reviewer: Ruby.

Added some ruby people and who is on the CC list of original PR.

sunpoet requested changes to this revision.Apr 22 2019, 6:09 PM
sunpoet added inline comments.
17 ↗(On Diff #56464)

I think you could simplify the option names by removing BACKEND_ prefix.

20–31 ↗(On Diff #56464)

Please group settings of each option (e.g. BDB_*) together. It's easier to read.

And it's easier to find out duplicate or conflicts, e.g. duplicate BACKEND_BDB_USES in line 21 and 27.

35–38 ↗(On Diff #56464)

It is not needed. This gem does not require libunwind.

24 ↗(On Diff #56464)

Please use (the homepage link of this gem).

136 ↗(On Diff #56464)

I would like to keep it in the same order (ext/dbm ext/gdbm) as the pkg-message.

This revision now requires changes to proceed.Apr 22 2019, 6:09 PM
  • Replace WWW with GitHub
  • Remove BACKEND_ prefix from options and duplicate definitions
  • Remove unnecessary dependency on libunwind
  • Sort modules in Makefile
31 ↗(On Diff #56520)

IIUC, it should be LOCALBASE, not PREFIX.

This revision is now accepted and ready to land.Apr 26 2019, 1:21 PM
This revision was automatically updated to reflect the committed changes.