From owner-svn-ports-head@FreeBSD.ORG Tue Aug 27 21:37:48 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id DD770718; Tue, 27 Aug 2013 21:37:47 +0000 (UTC) (envelope-from rene@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id CA5FE246D; Tue, 27 Aug 2013 21:37:47 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.7/8.14.7) with ESMTP id r7RLblq7076896; Tue, 27 Aug 2013 21:37:47 GMT (envelope-from rene@svn.freebsd.org) Received: (from rene@localhost) by svn.freebsd.org (8.14.7/8.14.5/Submit) id r7RLblb7076894; Tue, 27 Aug 2013 21:37:47 GMT (envelope-from rene@svn.freebsd.org) Message-Id: <201308272137.r7RLblb7076894@svn.freebsd.org> From: Rene Ladan Date: Tue, 27 Aug 2013 21:37:47 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r325489 - in head/net/boinc-client: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Aug 2013 21:37:48 -0000 Author: rene Date: Tue Aug 27 21:37:47 2013 New Revision: 325489 URL: http://svnweb.freebsd.org/changeset/ports/325489 Log: Overhaul option handling: - Only use the rc framework for boinc-client when the CLIENT option is selected - Always set CLIENT and X11 options when MANAGER option is selected - Unset USER option when CLIENT option is not selected - Create the share/boinc directory when the MANAGER or X11 option is set instead of when the USER option is set - Prefer negation over empty() when checking options - Update description for the USER option In pkg-install.in : - Only install the CA bundle and only adjust file ownership of ~boinc when the USER option is selected - Only install skins when both the USER and MANAGER options are selected While here: - create share/pixmaps in PREFIX instead of LOCALBASE - wrap a long line PR: ports/176724 (part) Submitted by: Andrey Simonenko Modified: head/net/boinc-client/Makefile head/net/boinc-client/files/pkg-install.in Modified: head/net/boinc-client/Makefile ============================================================================== --- head/net/boinc-client/Makefile Tue Aug 27 21:26:36 2013 (r325488) +++ head/net/boinc-client/Makefile Tue Aug 27 21:37:47 2013 (r325489) @@ -33,8 +33,6 @@ SUB_LIST= BOINC_CLIENT_HOME="${BOINC_CLI BOINC_CLIENT_GROUP="${BOINC_CLIENT_GROUP}" \ PREFIX="${PREFIX}" LOCALBASE="${LOCALBASE}" -USE_RC_SUBR= boinc-client - CONFIGURE_ARGS= --disable-server CPPFLAGS+= -I${LOCALBASE}/include @@ -47,7 +45,7 @@ CLIENT_DESC= Build BOINC client MANAGER_DESC= Build BOINC manager GUI X11_DESC= Build graphics API LINUX_DESC= Accept Linux science applications -USER_DESC= Create/check BOINC client user +USER_DESC= Create/check BOINC client user and directory SKINS_DESC= Install more skins for BOINC manager OPTIMIZE_DESC= Enable compiler optimization flags SCRIPT_DESC= Command line options in script @@ -61,12 +59,7 @@ BROKEN= Does not install on ia64, power .endif .if ${PORT_OPTIONS:MMANAGER} -. if empty(PORT_OPTIONS:MCLIENT) -PORT_OPTIONS+= CLIENT -. endif -. if empty(PORT_OPTIONS:MX11) -PORT_OPTIONS+= X11 -. endif +PORT_OPTIONS+= CLIENT X11 .endif .if ${PORT_OPTIONS:MX11} @@ -90,18 +83,22 @@ CONFIGURE_ARGS+=--with-sqlite3-prefix=${ CPPFLAGS+= -DNO_PER_THREAD_LOCALE . endif PLIST_SUB+= BOINC_MANAGER="" +SUB_LIST+= OPTION_MANAGER="true" .else CONFIGURE_ARGS+=--disable-manager --with-wx-config=false --without-x PLIST_SUB+= BOINC_MANAGER="@comment " +SUB_LIST+= OPTION_MANAGER="" .endif .if ${PORT_OPTIONS:MCLIENT} +USE_RC_SUBR= boinc-client LIB_DEPENDS+= curl:${PORTSDIR}/ftp/curl RUN_DEPENDS+= ${LOCALBASE}/share/certs/ca-root-nss.crt:${PORTSDIR}/security/ca_root_nss PLIST_SUB+= BOINC_CLIENT="" SUB_LIST+= OPTION_NO_GUI_RPC=${PORT_OPTIONS:MNO_GUI_RPC:S/^NO_GUI_RPC$/--no_gui_rpc/} \ OPTION_NO_NET_INFO=${PORT_OPTIONS:MNO_NET_INFO:S/^NO_NET_INFO$/--suppress_net_info/} .else +PORT_OPTIONS:= ${PORT_OPTIONS:NUSER} CONFIGURE_ARGS+=--disable-client PLIST_SUB+= BOINC_CLIENT="@comment " .endif @@ -125,14 +122,18 @@ PLIST_SUB+= SKINS="@comment " .if ${PORT_OPTIONS:MUSER} PLIST_SUB+= USER="" +SUB_LIST+= OPTION_USER="true" .else PLIST_SUB+= USER="@comment " +SUB_LIST+= OPTION_USER="" .endif .if ${PORT_OPTIONS:MOPTIMIZE} CONFIGURE_ARGS+=--enable-optimize .endif +PORT_OPTIONS:= ${PORT_OPTIONS:O:u} + post-patch: @${REINPLACE_CMD} -e 's|client/scripts||' ${WRKSRC}/Makefile.in @${REINPLACE_CMD} -e 's|-lcrypto -ldl|-lcrypto|' ${WRKSRC}/configure @@ -141,19 +142,20 @@ post-patch: .else @${REINPLACE_CMD} -e 's| locale||' ${WRKSRC}/Makefile.in .endif -.if empty(PORT_OPTIONS:MX11) +.if ! ${PORT_OPTIONS:MX11} @${REINPLACE_CMD} -e 's|@BUILD_GRAPHICS_API_TRUE@|#&|' ${WRKSRC}/api/Makefile.in .endif post-install: -.if ${PORT_OPTIONS:MUSER} @PKG_PREFIX=${PREFIX} ${SH} ${PKGINSTALL} ${PKGNAME} POST-INSTALL +.if ${PORT_OPTIONS:MMANAGER} || ${PORT_OPTIONS:MX11} + ${MKDIR} ${PREFIX}/share/boinc .endif .if ${PORT_OPTIONS:MMANAGER} - ${MKDIR} ${PREFIX}/share/boinc - ${MKDIR} ${LOCALBASE}/share/pixmaps + ${MKDIR} ${PREFIX}/share/pixmaps . for name in 16 32 48 - ${INSTALL_DATA} ${WRKSRC}/clientgui/res/boincmgr.${name}x${name}.png ${LOCALBASE}/share/pixmaps + ${INSTALL_DATA} ${WRKSRC}/clientgui/res/boincmgr.${name}x${name}.png \ + ${LOCALBASE}/share/pixmaps . endfor . if ${PORT_OPTIONS:MSKINS} ${CP} -R ${WRKSRC}/clientgui/skins ${PREFIX}/share/boinc @@ -170,9 +172,6 @@ post-install: api/txf_util.h ${INSTALL_DATA} ${WRKSRC}/${name} ${PREFIX}/include/boinc . endfor -. if empty(PORT_OPTIONS:MMANAGER) - ${MKDIR} ${PREFIX}/share/boinc -. endif ${CP} -R ${WRKSRC}/api/txf ${PREFIX}/share/boinc .endif Modified: head/net/boinc-client/files/pkg-install.in ============================================================================== --- head/net/boinc-client/files/pkg-install.in Tue Aug 27 21:26:36 2013 (r325488) +++ head/net/boinc-client/files/pkg-install.in Tue Aug 27 21:37:47 2013 (r325489) @@ -8,17 +8,21 @@ PATH=/bin:/usr/sbin:/usr/bin:/usr/sbin user=%%BOINC_CLIENT_USER%% group=%%BOINC_CLIENT_GROUP%% home="%%BOINC_CLIENT_HOME%%" +option_manager=%%OPTION_MANAGER%% +option_user=%%OPTION_USER%% case $2 in POST-INSTALL) - if [ ! -e "${home}/skins" -a ! -L "${home}/skins" ]; then - ln -s "%%PREFIX%%/share/boinc/skins" "${home}/skins" + if [ -n "${option_user}" ] ; then + if [ -n "${option_manager}" -a ! -e "${home}/skins" -a ! -L "${home}/skins" ]; then + ln -s "%%PREFIX%%/share/boinc/skins" "${home}/skins" + fi + if [ ! -e "${home}/ca-bundle.crt" -a ! -L "${home}/ca-bundle.crt" ]; then + ln -s "%%LOCALBASE%%/share/certs/ca-root-nss.crt" "${home}/ca-bundle.crt" + fi + echo "Adjusting file ownership in \"${home}\" to ${user}:${group}" + chown -hR ${user}:${group} "${home}" fi - if [ ! -e "${home}/ca-bundle.crt" -a ! -L "${home}/ca-bundle.crt" ]; then - ln -s "%%LOCALBASE%%/share/certs/ca-root-nss.crt" "${home}/ca-bundle.crt" - fi - echo "Adjusting file ownership in \"${home}\" to ${user}:${group}" - chown -hR ${user}:${group} "${home}" ;; esac