Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 3 Oct 2017 20:54:29 +0000 (UTC)
From:      Zach Leslie <zleslie@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r451170 - head/sysutils/facter
Message-ID:  <201710032054.v93KsT4j035540@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: zleslie
Date: Tue Oct  3 20:54:29 2017
New Revision: 451170
URL: https://svnweb.freebsd.org/changeset/ports/451170

Log:
  sysutils/facter: Drop slave port, add Java option default
  
  Here we drop support for the slave port, since it never quite worked as
  desired, nor was it ever submitted, and removes the conditionals to support it.
  Also, we add a new default option FACTER_JAVA to build support for Java and
  install facter.jar to be used by sysutils/puppetserver5.
  
  Approved by:	swills(mentor)
  Differential Revision:	D12498

Modified:
  head/sysutils/facter/Makefile
  head/sysutils/facter/pkg-plist

Modified: head/sysutils/facter/Makefile
==============================================================================
--- head/sysutils/facter/Makefile	Tue Oct  3 20:38:27 2017	(r451169)
+++ head/sysutils/facter/Makefile	Tue Oct  3 20:54:29 2017	(r451170)
@@ -2,7 +2,7 @@
 
 PORTNAME=	facter
 PORTVERSION=	3.9.0
-PORTREVISION?=	2
+PORTREVISION?=	3
 CATEGORIES?=	sysutils
 MASTER_SITES=	http://downloads.puppetlabs.com/facter/
 
@@ -12,10 +12,8 @@ COMMENT=	Cross-platform library for retrieving facts f
 LICENSE=	APACHE20
 LICENSE_FILE=	${WRKSRC}/LICENSE
 
-USES=		cmake
+USES=		cmake compiler:c++11-lib ssl
 
-.if !defined(PKGNAMEPREFIX)
-
 BROKEN_DragonFly=	DragonFly is not supported upstream
 BROKEN_powerpc64=	fails to compile: collection.cc: 'stoi' was not declared in this scope
 
@@ -26,13 +24,24 @@ LIB_DEPENDS=	libboost_system.so:devel/boost-libs \
 		libleatherman_locale.so:devel/leatherman \
 		libyaml-cpp.so:devel/yaml-cpp
 
-PLIST_SUB+=	PORTVERSION="${PORTVERSION}"
+PLIST_SUB=	PORTVERSION="${PORTVERSION}"
 
 USE_LDCONFIG=	yes
 USE_RUBY=	yes
-USES+=		compiler:c++11-lib ssl
 CMAKE_ARGS+=	-DMAN_PATH=${MANPREFIX}/man
 
+OPTIONS_DEFINE=	FACTER_JAVA
+FACTER_JAVA_DESC=	Build with Java bindings for puppetserver
+FACTER_JAVA_USE=	java=yes
+FACTER_JAVA_VARS=	JAVA_BUILD=yes
+FACTER_JAVA_CMAKE_ON=	-DJAVA_HOME=${JAVA_HOME}
+
+OPTIONS_DEFAULT=	FACTER_JAVA
+
+OPTIONS_SUB=	yes
+
+.include <bsd.port.options.mk>
+
 post-patch:
 	${REINPLACE_CMD} -e 's|%%PREFIX%%|${PREFIX}|g' \
 		${WRKSRC}/lib/src/facts/posix/collection.cc \
@@ -42,28 +51,9 @@ test: build
 	cd ${WRKSRC}/lib && bundle install --path vendor
 	cd ${WRKSRC} && ${MAKE_CMD} test
 
-.elif ${PKGNAMEPREFIX} == java-
-
-LIB_DEPENDS=	libfacter.so:sysutils/facter
-
-ALL_TARGET=	facter-jruby
-USE_JAVA=	yes
-JAVA_BUILD=	yes
-CMAKE_ARGS+=	-DJAVA_HOME=${JAVA_HOME}
-
-# Ignore sysutils/facter's pkg-plist by providing a non-existent file
-PLIST=		${WRKDIR}/no-plist
-PLIST_FILES=	${PREFIX}/lib/ruby/vendor_ruby/facter.jar
-
-do-install:
-	${MKDIR} ${STAGEDIR}${PREFIX}/lib/ruby/vendor_ruby
-	cd ${WRKSRC} && ${INSTALL_DATA} lib/facter.jar ${STAGEDIR}${PREFIX}/lib/ruby/vendor_ruby
-
-.endif
-
 .include <bsd.port.pre.mk>
 
-.if !defined(PKGNAMEPREFIX) && ${OPSYS} == FreeBSD && ${COMPILER_FEATURES:Mlibstdc++}
+.if ${OPSYS} == FreeBSD && ${COMPILER_FEATURES:Mlibstdc++}
 BROKEN=		Build with system libstdc++ is unsupported
 .endif
 

Modified: head/sysutils/facter/pkg-plist
==============================================================================
--- head/sysutils/facter/pkg-plist	Tue Oct  3 20:38:27 2017	(r451169)
+++ head/sysutils/facter/pkg-plist	Tue Oct  3 20:54:29 2017	(r451170)
@@ -21,3 +21,4 @@ lib/libfacter.so
 lib/libfacter.so.%%PORTVERSION%%
 lib/ruby/vendor_ruby/facter.rb
 man/man8/facter.8.gz
+%%FACTER_JAVA%%lib/ruby/vendor_ruby/facter.jar



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201710032054.v93KsT4j035540>