Page MenuHomeFreeBSD
Differential D5602 Diff 14225 databases/percona56-server/Makefile

Changeset View

Standalone View

View Options

databases/percona56-server/Makefile

# Created by: Alex Dupre <ale@FreeBSD.org> # Created by: Alex Dupre <ale@FreeBSD.org>
# $FreeBSD$ # $FreeBSD$
PORTNAME?= percona PORTNAME?= percona
DISTVERSION= 5.6.27-75.0 DISTVERSION= 5.6.29-76.2
PORTREVISION?= 0 PORTREVISION?= 0
CATEGORIES= databases ipv6 CATEGORIES= databases ipv6
MASTER_SITES= http://www.percona.com/downloads/Percona-Server-5.6/Percona-Server-${DISTVERSION}/source/tarball/ MASTER_SITES= http://www.percona.com/downloads/Percona-Server-5.6/Percona-Server-${DISTVERSION}/source/tarball/
PKGNAMESUFFIX?= 56-server PKGNAMESUFFIX?= 56-server
DISTNAME= percona-server-${DISTVERSION} DISTNAME= percona-server-${DISTVERSION}
MAINTAINER= flo@FreeBSD.org MAINTAINER= flo@FreeBSD.org
COMMENT?= Multithreaded SQL database (server) COMMENT?= Multithreaded SQL database (server)
SLAVEDIRS= databases/percona56-client SLAVEDIRS= databases/percona56-client
USES= bison cmake perl5 shebangfix USES= bison cmake perl5 shebangfix
OPTIONS_DEFINE= OPENSSL FASTMTX INNODBMEMCACHED OPTIONS_DEFINE= OPENSSL FASTMTX INNODBMEMCACHED TOKUDB
OPTIONS_DEFAULT= OPENSSL INNODBMEMCACHED OPTIONS_DEFAULT= OPENSSL INNODBMEMCACHED
OPTIONS_SUB= yes OPTIONS_SUB= yes
FASTMTX_DESC= Replace mutexes with spinlocks FASTMTX_DESC= Replace mutexes with spinlocks
FASTMTX_CMAKE_ON= -DWITH_FAST_MUTEXES=1
INNODBMEMCACHED_DESC= InnoDB Memcached plugin INNODBMEMCACHED_DESC= InnoDB Memcached plugin
matUnsubmitted
Not Done
Inline Actions

This should be named "extrapatch-tokudb" to be more consistent with the rest of the tree.

mat: This should be named "extrapatch-tokudb" to be more consistent with the rest of the tree.
INNODBMEMCACHED_CMAKE_ON=-DWITH_INNODB_MEMCACHED=1 INNODBMEMCACHED_CMAKE_ON=-DWITH_INNODB_MEMCACHED=1
OPENSSL_CMAKE_ON= -DWITH_SSL=bundled
TOKUDB_DESC= TokuDB Engine (experimental)
TOKUDB_CMAKE_ON=-DUSE_CTAGS=0
TOKUDB_VARS= ONLY_FOR_ARCHS="amd64" \
ONLY_FOR_ARCHS_REASON="TokuDB engine does not compile on ${ARCH}"
TOKUDB_EXTRA_PATCHES=${PATCHDIR}/extrapatch-tokudb
matUnsubmitted
Not Done
Inline Actions

Thinking about it, this is a "server" only option, maybe you should add:

OPTIONS_EXCLUDE=TOKUDB

To databases/percona56-client/Makefile, so that it's not available as an option there.

mat: Thinking about it, this is a "server" only option, maybe you should add…
DATADIR= ${PREFIX}/share/mysql DATADIR= ${PREFIX}/share/mysql
CMAKE_ARGS+= -DINSTALL_DOCDIR="share/doc/mysql" \ CMAKE_ARGS+= -DINSTALL_DOCDIR="share/doc/mysql" \
-DINSTALL_DOCREADMEDIR="share/doc/mysql" \ -DINSTALL_DOCREADMEDIR="share/doc/mysql" \
-DINSTALL_INCLUDEDIR="include/mysql" \ -DINSTALL_INCLUDEDIR="include/mysql" \
-DINSTALL_INFODIR="info" \ -DINSTALL_INFODIR="info" \
-DINSTALL_LIBDIR="lib/mysql" \ -DINSTALL_LIBDIR="lib/mysql" \
Show All 11 LinesCMAKE_ARGS+= -DINSTALL_DOCDIR="share/doc/mysql" \
-DWITH_LIBWRAP=1 \ -DWITH_LIBWRAP=1 \
-DHAVE_PAM=0 -DHAVE_PAM=0
SHEBANG_FILES= scripts/*.pl* scripts/*.sh SHEBANG_FILES= scripts/*.pl* scripts/*.sh
.ifdef USE_MYSQL .ifdef USE_MYSQL
.error You have `USE_MYSQL' variable defined either in environment or in make(1) arguments. Please undefine and try again. .error You have `USE_MYSQL' variable defined either in environment or in make(1) arguments. Please undefine and try again.
.endif .endif
.include <bsd.port.options.mk>
.if ${PORT_OPTIONS:MOPENSSL}
CMAKE_ARGS+= -DWITH_SSL=bundled
.endif
.if ${PORT_OPTIONS:MFASTMTX}
CMAKE_ARGS+= -DWITH_FAST_MUTEXES=1
.endif
matUnsubmitted
Not Done
Inline Actions

That's not needed any more. (Sorry, should have said earlier.)

mat: That's not needed any more. (Sorry, should have said earlier.)
# MySQL-Server part # MySQL-Server part
.if !defined(CLIENT_ONLY) .if !defined(CLIENT_ONLY)
USE_MYSQL= yes USE_MYSQL= yes
WANT_MYSQL_VER= 56p WANT_MYSQL_VER= 56p
CONFLICTS_INSTALL= mysql*-server-* \ CONFLICTS_INSTALL= mysql*-server-* \
mariadb*-server-* \ mariadb*-server-* \
matUnsubmitted
Not Done
Inline Actions

I think these should be set using helpers and sooner in the Makefile, for example:

TOKUDB_CMAKE_ON=      -DUSE_CTAGS=0
TOKUDB_VARS=          ONLY_FOR_ARCH=amd64
ONLY_FOR_ARCH_REASON= TokuDB engine does not compile on ${ARCH}

The ONLY_FOR_ARCH_REASON can be set regardless as it's a no-op if ONLY_FOR_ARCH is not defined.

(while there, you could convert the two options blocs just before to helpers too.)

mat: I think these should be set using helpers and sooner in the Makefile, for example…
percona5[0-57-9]-server-* percona5[0-57-9]-server-*
USE_RC_SUBR= mysql-server USE_RC_SUBR= mysql-server
USERS= mysql USERS= mysql
GROUPS= mysql GROUPS= mysql
INFO= mysql INFO= mysql
Show All 27 Lines