Changeset View
Changeset View
Standalone View
Standalone View
databases/rubygem-dbm/Makefile
- This file was added.
Property | Old Value | New Value |
---|---|---|
svn:eol-style | null | native \ No newline at end of property |
svn:keywords | null | FreeBSD=%H \ No newline at end of property |
svn:mime-type | null | text/plain \ No newline at end of property |
# $FreeBSD$ | |||||
PORTNAME= dbm | |||||
PORTVERSION= 1.0.0 | |||||
CATEGORIES= databases rubygems | |||||
MASTER_SITES= RG | |||||
MAINTAINER= yasu@utahime.org | |||||
COMMENT= Wrapper for the UNIX-style Database Manager Library | |||||
LICENSE= BSD2CLAUSE | |||||
USES= gem | |||||
USE_RUBY= yes | |||||
OPTIONS_SINGLE= BACKEND | |||||
OPTIONS_SINGLE_BACKEND= BDB GDBM LIBC QDBM | |||||
OPTIONS_DEFAULT= BDB | |||||
sunpoet: I think you could simplify the option names by removing BACKEND_ prefix. | |||||
BACKEND_DESC= Select backend dbm library | |||||
BDB_USES= bdb | |||||
BDB_CONFIGURE_ON= --with-dbm-type=db${BDB_VER} | |||||
GDBM_CONFIGURE_ON= --with-dbm-type=gdbm_compat | |||||
GDBM_LIB_DEPENDS= libgdbm.so:databases/gdbm | |||||
LIBC_DESC= libc ndbm support | |||||
LIBC_CONFIGURE_ON= --with-dbm-type=libc | |||||
QDBM_DESC= QDBM support | |||||
QDBM_CONFIGURE_ON= --with-dbm-type=qdbm | |||||
QDBM_LIB_DEPENDS= libqdbm.so:databases/qdbm | |||||
pre-configure-GDBM-on: | |||||
@if [ ! -f ${LOCALBASE}/lib/libgdbm_compat.a ]; then ${ECHO_CMD} You need to reinstall database/gdbm with COMPAT option ON.; exit 1; fi | |||||
Not Done Inline ActionsPlease 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. sunpoet: Please group settings of each option (e.g. BDB_*) together. It's easier to read.
And it's… | |||||
Not Done Inline ActionsIIUC, it should be LOCALBASE, not PREFIX. sunpoet: IIUC, it should be LOCALBASE, not PREFIX. | |||||
.include <bsd.port.mk> | |||||
Not Done Inline ActionsIt is not needed. This gem does not require libunwind. sunpoet: It is not needed. This gem does not require libunwind. |
I think you could simplify the option names by removing BACKEND_ prefix.