Apply some suggestions by pkglint Also renamed the option "ssl" to "openssl" because asking users to choose between "ssl" and "gnutls" is very confusing. TLS support is now optional too, as gkrellm can be built fine without any TLS libraries.diff -r1.106 -r1.107 pkgsrc/sysutils/gkrellm/Makefile
(pho)
@@ -1,53 +1,53 @@ | @@ -1,53 +1,53 @@ | |||
1 | # $NetBSD: Makefile,v 1.106 2021/11/27 04:43:02 pho Exp $ | 1 | # $NetBSD: Makefile,v 1.107 2021/11/27 05:37:59 pho Exp $ | |
2 | .include "Makefile.common" | 2 | .include "Makefile.common" | |
3 | 3 | |||
4 | PKGREVISION= 45 | 4 | PKGREVISION= 46 | |
5 | 5 | |||
6 | GKRELLM_PKGBASE= gkrellm | 6 | GKRELLM_PKGBASE= gkrellm | |
7 | WRKSRC= ${GKRELLM_SRCDIR} | 7 | WRKSRC= ${GKRELLM_SRCDIR} | |
8 | 8 | |||
9 | COMMENT= GTK2 based system monitor | 9 | COMMENT= GTK2 based system monitor | |
10 | 10 | |||
11 | DEPENDS+= gkrellm-share-${GKRELLM_VERSION}:../../sysutils/gkrellm-share | 11 | DEPENDS+= gkrellm-share-${GKRELLM_VERSION}:../../sysutils/gkrellm-share | |
12 | 12 | |||
13 | USE_TOOLS+= pkg-config | 13 | USE_TOOLS+= pkg-config | |
14 | 14 | |||
15 | .include "../../mk/bsd.prefs.mk" | 15 | .include "../../mk/bsd.prefs.mk" | |
16 | 16 | |||
17 | # NetBSD 3.99.1 and above don't have to be installed setgid. | 17 | # NetBSD 3.99.1 and above don't have to be installed setgid. | |
18 | .if (${OPSYS} == "FreeBSD" || ${OPSYS} == "OpenBSD" || ${OPSYS} == "DragonFly" || \ | 18 | .if (${OPSYS} == "FreeBSD" || ${OPSYS} == "OpenBSD" || ${OPSYS} == "DragonFly" || \ | |
19 | (${OPSYS} == "NetBSD" && (!empty(OS_VERSION:M[123]*) && empty(OS_VERSION:M3.99.*)))) | 19 | (${OPSYS} == "NetBSD" && (!empty(OS_VERSION:M[123].*) && empty(OS_VERSION:M3.99.*)))) | |
20 | SPECIAL_PERMS+= ${PREFIX}/bin/gkrellm ${BINOWN} kmem 2555 | 20 | SPECIAL_PERMS+= ${PREFIX}/bin/gkrellm ${BINOWN} kmem 2555 | |
21 | .endif | 21 | .endif | |
22 | 22 | |||
23 | NOT_FOR_PLATFORM= IRIX-*-* | 23 | NOT_FOR_PLATFORM= IRIX-*-* | |
24 | 24 | |||
25 | .include "options.mk" | 25 | .include "options.mk" | |
26 | 26 | |||
27 | MAKE_ENV+= MODULES=src LINK_FLAGS=${EXPORT_SYMBOLS_LDFLAGS:Q} | 27 | MAKE_ENV+= MODULES=src LINK_FLAGS=${EXPORT_SYMBOLS_LDFLAGS:Q} | |
28 | MAKE_ENV+= SSL_TYPE=${GKRELLM_SSL_TYPE:Q} | 28 | MAKE_ENV+= SSL_TYPE=${GKRELLM_SSL_TYPE:Q} | |
29 | MAKE_ENV+= PKG_INCLUDE=-I${BUILDLINK_PREFIX.${GKRELLM_SSL_TYPE}}/include | 29 | MAKE_ENV+= PKG_INCLUDE=-I${BUILDLINK_PREFIX.${GKRELLM_SSL_TYPE}}/include | |
30 | MAKE_ENV+= PKG_LIBS="-L${BUILDLINK_PREFIX.${GKRELLM_SSL_TYPE}}/lib -Wl,-R${BUILDLINK_PREFIX.${GKRELLM_SSL_TYPE}}/lib" | 30 | MAKE_ENV+= PKG_LIBS="-L${BUILDLINK_PREFIX.${GKRELLM_SSL_TYPE}}/lib ${COMPILER_RPATH_FLAG}${BUILDLINK_PREFIX.${GKRELLM_SSL_TYPE}}/lib" | |
31 | MAKE_ENV+= MANDIR=${PREFIX}/${PKGMANDIR}/man1 | 31 | MAKE_ENV+= MANDIR=${PREFIX}/${PKGMANDIR}/man1 | |
32 | MAKE_ENV+= SMANDIR=${PREFIX}/${PKGMANDIR}/man8 | 32 | MAKE_ENV+= SMANDIR=${PREFIX}/${PKGMANDIR}/man8 | |
33 | MAKE_FLAGS+= BINOWNER=${BINOWN} BINGROUP=${BINGRP} | 33 | MAKE_FLAGS+= BINOWNER=${BINOWN} BINGROUP=${BINGRP} | |
34 | INSTALL_MAKE_FLAGS+= MANDIR=${DESTDIR}${PREFIX}/${PKGMANDIR}/man1 | 34 | INSTALL_MAKE_FLAGS+= MANDIR=${DESTDIR}${PREFIX}/${PKGMANDIR}/man1 | |
35 | INSTALL_MAKE_FLAGS+= SMANDIR=${DESTDIR}${PREFIX}/${PKGMANDIR}/man8 | 35 | INSTALL_MAKE_FLAGS+= SMANDIR=${DESTDIR}${PREFIX}/${PKGMANDIR}/man8 | |
36 | 36 | |||
37 | # Fix PR pkg/40047, Thanks to Ondrej Tuma | 37 | # Fix PR pkg/40047, Thanks to Ondrej Tuma | |
38 | .if ${OBJECT_FMT} == "ELF" | 38 | .if ${OBJECT_FMT} == "ELF" | |
39 | LDFLAGS+= -Wl,-export-dynamic | 39 | LDFLAGS+= -Wl,-export-dynamic | |
40 | .endif | 40 | .endif | |
41 | 41 | |||
42 | SUBST_CLASSES+= gkprefix | |||
42 | SUBST_STAGE.gkprefix= pre-configure | 43 | SUBST_STAGE.gkprefix= pre-configure | |
43 | SUBST_FILES.gkprefix= src/gkrellm.h | 44 | SUBST_FILES.gkprefix= src/gkrellm.h | |
44 | SUBST_SED.gkprefix= -e '/SYSTEM_/{s!/usr!${PREFIX}!;}' | 45 | SUBST_SED.gkprefix= -e '/SYSTEM_/{s!/usr!${PREFIX}!;}' | |
45 | SUBST_CLASSES+= gkprefix | |||
46 | 46 | |||
47 | REQD_DIRS= lib/gkrellm2/plugins lib/gkrellm2 include/gkrellm2 | 47 | REQD_DIRS= lib/gkrellm2/plugins lib/gkrellm2 include/gkrellm2 | |
48 | 48 | |||
49 | .include "../../devel/glib2/buildlink3.mk" | 49 | .include "../../devel/glib2/buildlink3.mk" | |
50 | .include "../../x11/gtk2/buildlink3.mk" | 50 | .include "../../x11/gtk2/buildlink3.mk" | |
51 | .include "../../x11/libSM/buildlink3.mk" | 51 | .include "../../x11/libSM/buildlink3.mk" | |
52 | 52 | |||
53 | .include "../../mk/bsd.pkg.mk" | 53 | .include "../../mk/bsd.pkg.mk" |
@@ -1,28 +1,27 @@ | @@ -1,28 +1,27 @@ | |||
1 | # $NetBSD: Makefile.common,v 1.18 2014/10/05 16:41:08 wiz Exp $ | 1 | # $NetBSD: Makefile.common,v 1.19 2021/11/27 05:37:59 pho Exp $ | |
2 | # | 2 | # | |
3 | # used by sysutils/gkrellm-share/Makefile | 3 | # used by sysutils/gkrellm-share/Makefile | |
4 | # used by sysutils/gkrellm-server/Makefile | 4 | # used by sysutils/gkrellm-server/Makefile | |
5 | 5 | |||
6 | DISTNAME= ${GKRELLM_DISTBASE} | 6 | DISTNAME= ${GKRELLM_DISTBASE} | |
7 | PKGNAME= ${GKRELLM_PKGBASE}-${GKRELLM_VERSION} | 7 | PKGNAME= ${GKRELLM_PKGBASE}-${GKRELLM_VERSION} | |
8 | CATEGORIES= sysutils | 8 | CATEGORIES= sysutils | |
9 | MASTER_SITES= http://members.dslextreme.com/users/billw/gkrellm/ | 9 | MASTER_SITES= http://members.dslextreme.com/users/billw/gkrellm/ | |
10 | EXTRACT_SUFX= .tar.bz2 | 10 | EXTRACT_SUFX= .tar.bz2 | |
11 | 11 | |||
12 | MAINTAINER= pkgsrc-users@NetBSD.org | 12 | MAINTAINER= pkgsrc-users@NetBSD.org | |
13 | HOMEPAGE= http://gkrellm.net/ | 13 | HOMEPAGE= http://gkrellm.net/ | |
14 | 14 | LICENSE= gnu-gpl-v2 | ||
15 | CONFLICTS+= gkrellm-1* | |||
16 | 15 | |||
17 | USE_TOOLS+= gmake | 16 | USE_TOOLS+= gmake | |
18 | USE_PKGLOCALEDIR= YES | 17 | USE_PKGLOCALEDIR= YES | |
19 | 18 | |||
20 | MAKE_ENV+= LOCALEDIR=${PREFIX}/${PKGLOCALEDIR}/locale | 19 | MAKE_ENV+= LOCALEDIR=${PREFIX}/${PKGLOCALEDIR}/locale | |
21 | MAKE_ENV+= GREP=${GREP:Q} | 20 | MAKE_ENV+= GREP=${GREP:Q} | |
22 | 21 | |||
23 | .if defined(GKRELLM_DEBUG) | 22 | .if defined(GKRELLM_DEBUG) | |
24 | MAKE_ENV+= DEBUG=yes | 23 | MAKE_ENV+= DEBUG=yes | |
25 | .endif | 24 | .endif | |
26 | 25 | |||
27 | GKRELLM_VERSION= 2.2.10 | 26 | GKRELLM_VERSION= 2.2.10 | |
28 | GKRELLM_DISTBASE= gkrellm-${GKRELLM_VERSION} | 27 | GKRELLM_DISTBASE= gkrellm-${GKRELLM_VERSION} |
@@ -1,17 +1,29 @@ | @@ -1,17 +1,29 @@ | |||
1 | # $NetBSD: buildlink3.mk,v 1.52 2020/08/17 20:17:43 leot Exp $ | 1 | # $NetBSD: buildlink3.mk,v 1.53 2021/11/27 05:37:59 pho Exp $ | |
2 | 2 | |||
3 | BUILDLINK_TREE+= gkrellm | 3 | BUILDLINK_TREE+= gkrellm | |
4 | 4 | |||
5 | .if !defined(GKRELLM_BUILDLINK3_MK) | 5 | .if !defined(GKRELLM_BUILDLINK3_MK) | |
6 | GKRELLM_BUILDLINK3_MK:= | 6 | GKRELLM_BUILDLINK3_MK:= | |
7 | 7 | |||
8 | BUILDLINK_API_DEPENDS.gkrellm+= gkrellm>=2.1.22 | 8 | BUILDLINK_API_DEPENDS.gkrellm+= gkrellm>=2.1.22 | |
9 | BUILDLINK_ABI_DEPENDS.gkrellm+= gkrellm>=2.2.10nb43 | 9 | BUILDLINK_ABI_DEPENDS.gkrellm+= gkrellm>=2.2.10nb43 | |
10 | BUILDLINK_PKGSRCDIR.gkrellm?= ../../sysutils/gkrellm | 10 | BUILDLINK_PKGSRCDIR.gkrellm?= ../../sysutils/gkrellm | |
11 | 11 | |||
12 | .include "../../devel/glib2/buildlink3.mk" | 12 | .include "../../devel/glib2/buildlink3.mk" | |
13 | .include "../../security/openssl/buildlink3.mk" | |||
14 | .include "../../x11/gtk2/buildlink3.mk" | 13 | .include "../../x11/gtk2/buildlink3.mk" | |
14 | .include "../../x11/libSM/buildlink3.mk" | |||
15 | ||||
16 | pkgbase := gkrellm | |||
17 | .include "../../mk/pkg-build-options.mk" | |||
18 | ||||
19 | .if ${PKG_BUILD_OPTIONS.gkrellm:Mopenssl} | |||
20 | .include "../../security/openssl/buildlink3.mk" | |||
21 | .endif | |||
22 | ||||
23 | .if ${PKG_BUILD_OPTIONS.gkrellm:Mgnutls} | |||
24 | .include "../../security/gnutls/buildlink3.mk" | |||
25 | .endif | |||
26 | ||||
15 | .endif # GKRELLM_BUILDLINK3_MK | 27 | .endif # GKRELLM_BUILDLINK3_MK | |
16 | 28 | |||
17 | BUILDLINK_TREE+= -gkrellm | 29 | BUILDLINK_TREE+= -gkrellm |
@@ -1,19 +1,20 @@ | @@ -1,19 +1,20 @@ | |||
1 | # $NetBSD: options.mk,v 1.1 2005/09/08 13:19:56 cube Exp $ | 1 | # $NetBSD: options.mk,v 1.2 2021/11/27 05:37:59 pho Exp $ | |
2 | 2 | |||
3 | PKG_OPTIONS_VAR= PKG_OPTIONS.${GKRELLM_PKGBASE} | 3 | PKG_OPTIONS_VAR= PKG_OPTIONS.gkrellm | |
4 | PKG_OPTIONS_REQUIRED_GROUPS= tls | 4 | PKG_OPTIONS_OPTIONAL_GROUPS= tls | |
5 | PKG_OPTIONS_GROUP.tls= gnutls ssl | 5 | PKG_OPTIONS_GROUP.tls= gnutls openssl | |
6 | 6 | |||
7 | PKG_SUGGESTED_OPTIONS= ssl | 7 | PKG_OPTIONS_LEGACY_OPTS+= ssl:openssl | |
8 | PKG_SUGGESTED_OPTIONS= openssl | |||
8 | 9 | |||
9 | .include "../../mk/bsd.options.mk" | 10 | .include "../../mk/bsd.options.mk" | |
10 | 11 | |||
11 | .if !empty(PKG_OPTIONS:Mssl) | 12 | .if !empty(PKG_OPTIONS:Mopenssl) | |
12 | .include "../../security/openssl/buildlink3.mk" | 13 | .include "../../security/openssl/buildlink3.mk" | |
13 | GKRELLM_SSL_TYPE= openssl | 14 | GKRELLM_SSL_TYPE= openssl | |
14 | .endif | 15 | .endif | |
15 | 16 | |||
16 | .if !empty(PKG_OPTIONS:Mgnutls) | 17 | .if !empty(PKG_OPTIONS:Mgnutls) | |
17 | .include "../../security/gnutls/buildlink3.mk" | 18 | .include "../../security/gnutls/buildlink3.mk" | |
18 | GKRELLM_SSL_TYPE= gnutls | 19 | GKRELLM_SSL_TYPE= gnutls | |
19 | .endif | 20 | .endif |