Hi all and especially Stuart,
On Tue, May 07, 2019 at 11:47:35PM +0100, Stuart Henderson wrote:
>
> Please mention the flavour in pkg/DESCR (also it's a good idea to include
> productivity/Makefile in your diff to add the line to build it ..
> makes it harder to forget :)
Thanks for the pointer. I've now added a blurb. I've also added a line
to the productivity/Makefile.
> Please use the standard "# uses pledge()" line next to WANTLIB for this.
> (Maybe it's worth retiring this now that pledge is not so much in a state
> of flux, but if we do that it would be a sweep across the tree rather
> than piecemeal).
OK, thanks, I didn't know we were "tracking" this explicitly this way.
Many thanks!
Tim
Here's the diff.
Index: productivity/Makefile
===================================================================
RCS file: /cvs/ports/productivity/Makefile,v
retrieving revision 1.84
diff -u -p -r1.84 Makefile
--- productivity/Makefile 2 May 2019 15:54:06 -0000 1.84
+++ productivity/Makefile 8 May 2019 02:50:38 -0000
@@ -28,6 +28,7 @@
SUBDIR += libkgapi
SUBDIR += librcps
SUBDIR += mcds
+ SUBDIR += mcds,no_gpgme
SUBDIR += osmo
SUBDIR += p5-Business-Tax-VAT-Validation
SUBDIR += projectlibre
Index: productivity/mcds/Makefile
===================================================================
RCS file: /cvs/ports/productivity/mcds/Makefile,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 Makefile
--- productivity/mcds/Makefile 2 May 2019 15:52:58 -0000 1.1.1.1
+++ productivity/mcds/Makefile 8 May 2019 02:50:38 -0000
@@ -2,7 +2,7 @@
COMMENT = tty-based carddav search tool
-V = 0.9
+V = 1.2
DISTNAME = mcds-${V}
CATEGORIES = productivity
@@ -11,6 +11,7 @@ MAINTAINER = Timothy Brown <tbrown@free
# GPLv3+
PERMIT_PACKAGE_CDROM = Yes
+# uses pledge()
WANTLIB = c curl iconv intl xml2
MASTER_SITES = https://github.com/t-brown/mcds/releases/download/v${V}/
@@ -22,5 +23,15 @@ LIB_DEPENDS = devel/gettext \
CONFIGURE_STYLE = gnu
NO_TEST = Yes
+
+FLAVORS = no_gpgme
+FLAVOR ?=
+
+.if ${FLAVOR:L:Mno_gpgme}
+CONFIGURE_ARGS += --disable-gpgme
+.else
+WANTLIB += assuan gpg-error gpgme
+LIB_DEPENDS += security/gpgme
+.endif
.include <bsd.port.mk>
Index: productivity/mcds/distinfo
===================================================================
RCS file: /cvs/ports/productivity/mcds/distinfo,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 distinfo
--- productivity/mcds/distinfo 2 May 2019 15:52:58 -0000 1.1.1.1
+++ productivity/mcds/distinfo 8 May 2019 02:50:38 -0000
@@ -1,2 +1,2 @@
-SHA256 (mcds-0.9.tar.gz) = p+H8Q94kiHDDo/pV570uCXZki5YnyC41tUDx8HgARKc=
-SIZE (mcds-0.9.tar.gz) = 194620
+SHA256 (mcds-1.2.tar.gz) = XvOuh5IgjdWF1BPE/mowwCv8BKY4l/nvmnzZEK4Pbx4=
+SIZE (mcds-1.2.tar.gz) = 201027
Index: productivity/mcds/pkg/DESCR
===================================================================
RCS file: /cvs/ports/productivity/mcds/pkg/DESCR,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 DESCR
--- productivity/mcds/pkg/DESCR 2 May 2019 15:52:58 -0000 1.1.1.1
+++ productivity/mcds/pkg/DESCR 8 May 2019 02:50:38 -0000
@@ -1,2 +1,8 @@
Mcds is a command line tool primarily used as a search query plugin for mutt
-to query a carddav server.
+to query a CardDav server.
+
+Flavors:
+ no_gpgme - build with-out GPGME support. By default it will build
+ with GPGME support for encrypted password files, so you
+ do not have to store your password in cleartext in your
+ .netrc file.