On Mon, Jan 01, 2018 at 11:21:32AM -0500, Brian Callahan wrote:
>
> On 01/01/18 11:08, Anton Lindqvist wrote:
> > Hi,
> > Update pick to 2.0.1 from maintainer, the changelog can be found here[1]
> > and the only change is:
> >
> > - Typing a query fast could yield incorrect results.
> >
> > Comments? OK?
> >
> > [1] https://github.com/calleerlandsson/pick/releases/tag/v2.0.1
> >
> > Index: Makefile
> > ===================================================================
> > RCS file: /cvs/ports/sysutils/pick/Makefile,v
> > retrieving revision 1.7
> > diff -u -p -r1.7 Makefile
> > --- Makefile 7 Nov 2017 14:16:54 -0000 1.7
> > +++ Makefile 1 Jan 2018 16:07:36 -0000
> > @@ -2,7 +2,7 @@
> > COMMENT = fuzzy select anything
> > -V = 2.0.0
> > +V = 2.0.1
> > DISTNAME = pick-${V}
> > CATEGORIES = sysutils
> > Index: distinfo
> > ===================================================================
> > RCS file: /cvs/ports/sysutils/pick/distinfo,v
> > retrieving revision 1.6
> > diff -u -p -r1.6 distinfo
> > --- distinfo 7 Nov 2017 14:16:54 -0000 1.6
> > +++ distinfo 1 Jan 2018 16:07:36 -0000
> > @@ -1,2 +1,2 @@
> > -SHA256 (pick-2.0.0.tar.gz) = DocUG5zKfDHU13yHp8BYLjFvQPkHZETHxuh6eR8a6As=
> > -SIZE (pick-2.0.0.tar.gz) = 107860
> > +SHA256 (pick-2.0.1.tar.gz) = Sllrj0CjFrxOLA2OiEKBDXp7adRkpBDk7ipldOAWKeM=
> > +SIZE (pick-2.0.1.tar.gz) = 108010
> >
>
> Two thoughts:
> 1. SEPARATE_BUILD=Yes can be set
> 2. There's a -Werror floating around the build
>
> If you don't want to patch ${WRKSRC}/Makefile.in to remove the -Werror, this
> will do:
> pre-configure:
> sed -i 's,-Werror,,g' ${WRKSRC}/Makefile.in
Thanks, both thoughts addressed. I went with your proposal and added a
pre-configure target. But, I will make sure that -Werror is removed in
the next upstream release.
Index: Makefile
===================================================================
RCS file: /cvs/ports/sysutils/pick/Makefile,v
retrieving revision 1.7
diff -u -p -r1.7 Makefile
--- Makefile 7 Nov 2017 14:16:54 -0000 1.7
+++ Makefile 1 Jan 2018 19:00:38 -0000
@@ -2,7 +2,7 @@
COMMENT = fuzzy select anything
-V = 2.0.0
+V = 2.0.1
DISTNAME = pick-${V}
CATEGORIES = sysutils
@@ -19,6 +19,10 @@ WANTLIB += c curses
MASTER_SITES = https://github.com/calleerlandsson/pick/releases/download/v${V}/
+SEPARATE_BUILD = Yes
CONFIGURE_STYLE = gnu
+
+pre-configure:
+ sed -i 's,-Werror,,g' ${WRKSRC}/Makefile.in
.include <bsd.port.mk>
Index: distinfo
===================================================================
RCS file: /cvs/ports/sysutils/pick/distinfo,v
retrieving revision 1.6
diff -u -p -r1.6 distinfo
--- distinfo 7 Nov 2017 14:16:54 -0000 1.6
+++ distinfo 1 Jan 2018 19:00:38 -0000
@@ -1,2 +1,2 @@
-SHA256 (pick-2.0.0.tar.gz) = DocUG5zKfDHU13yHp8BYLjFvQPkHZETHxuh6eR8a6As=
-SIZE (pick-2.0.0.tar.gz) = 107860
+SHA256 (pick-2.0.1.tar.gz) = Sllrj0CjFrxOLA2OiEKBDXp7adRkpBDk7ipldOAWKeM=
+SIZE (pick-2.0.1.tar.gz) = 108010