On 2016-04-18 Florian Stinglmayr <flor...@n0la.org> wrote: > On Mon, Apr 18, 2016 at 08:32:46PM +0200, Christopher Zimmermann > wrote: > > I have a finished herbstluftwm port upgrade, too. > > You can remove the BUILD_DEPENDS and the CC=egcc in MAKE_FLAGS > > since the gcc4 module takes care of them. Just use += for > > assignments to LIB_DEPENDS and RUN_DEPENDS so you don't overwrite > > the assignments made by the gcc4 module. > > Thanks for the suggestions, it's fixed in a new diff below.
OK chrisz@ > It's weird btw. Their entire code base is basically C with a few C++11 > things mixed in there. They are rewriting the whole thing in proper C++, but as a separate project. I guess they wanted to use the expressions they learned there... > > I could remove intl, m and pthread from WANTLIB, after adding > > devel/gettext to MODULES. > > > > naddy@ wishes to remove the devel/gettext module [1] and thus I > cleaned up herbstluftwm while I am at it. > > > a fixed RUN_DEPENDS is overkill. You may mention dmenu in a > > MESSAGE, if you want to. I don't think it is necessary. > > > > You are right, it isn't. It is covered well enough in herbstluftwm's > FAQ on their homepage. > > Regards, > Florian > > [1] https://marc.info/?l=openbsd-ports&m=145917613914938 > > Index: Makefile > =================================================================== > RCS file: /cvs/ports/x11/herbstluftwm/Makefile,v > retrieving revision 1.4 > diff -u -p -u -r1.4 Makefile > --- Makefile 25 Aug 2015 13:18:28 -0000 1.4 > +++ Makefile 18 Apr 2016 20:22:58 -0000 > @@ -1,29 +1,33 @@ > # $OpenBSD: Makefile,v 1.4 2015/08/25 13:18:28 sthen Exp $ > > COMMENT = manual tiling window manager > -DISTNAME = herbstluftwm-0.6.2 > +DISTNAME = herbstluftwm-0.7.0 > CATEGORIES = x11 > -REVISION = 1 > > HOMEPAGE = http://herbstluftwm.org/ > > +MAINTAINER = Florian Stinglmayr <flor...@n0la.org> > + > # BSD > PERMIT_PACKAGE_CDROM = Yes > > -WANTLIB += X11 Xext Xinerama c glib-2.0 pthread > +WANTLIB += X11 Xext Xinerama c estdc++ glib-2.0 intl m pthread > > MASTER_SITES = http://herbstluftwm.org/tarballs/ > > -MODULES = devel/gettext > +MODULES = gcc4 > + > +MODGCC4_LANGS= c++ > +MODGCC4_ARCHS= * > > -LIB_DEPENDS = devel/glib2 > +LIB_DEPENDS += devel/glib2 > > -RUN_DEPENDS = devel/desktop-file-utils \ > +RUN_DEPENDS += devel/desktop-file-utils \ > shells/bash \ > x11/dzen2,-gadgets > > USE_GMAKE = Yes > -MAKE_FLAGS = CC="${CC}" LDFLAGS= VERBOSE= COLOR=0 > +MAKE_FLAGS = LDFLAGS= VERBOSE= COLOR=0 > > BASEDIR = ${PREFIX}/share/examples/herbstluftwm > FAKE_FLAGS = SYSCONFDIR="${BASEDIR}" \ > @@ -33,9 +37,5 @@ FAKE_FLAGS = SYSCONFDIR="${BASEDIR}" \ > XSESSIONSDIR="${PREFIX}/share/applications" > > NO_TEST = Yes > - > -post-patch: > - perl -pi -e "s,^#!/bin/bash,#!${LOCALBASE}/bin/bash," \ > - `grep -Rl /bin/bash ${WRKSRC}/*` > > .include <bsd.port.mk> > Index: distinfo > =================================================================== > RCS file: /cvs/ports/x11/herbstluftwm/distinfo,v > retrieving revision 1.2 > diff -u -p -u -r1.2 distinfo > --- distinfo 5 Apr 2014 05:42:07 -0000 1.2 > +++ distinfo 18 Apr 2016 20:22:58 -0000 > @@ -1,2 +1,2 @@ > -SHA256 (herbstluftwm-0.6.2.tar.gz) = > qJ9cOXrufxqjs53VO8Z+KCZ58C8cC/PDPEGmCOIX8Kw= -SIZE > (herbstluftwm-0.6.2.tar.gz) = 224191 +SHA256 > (herbstluftwm-0.7.0.tar.gz) = > fi/8MLeRFAcjCl7rYeBEzqeFV5mRB9NyvC28PYLQric= +SIZE > (herbstluftwm-0.7.0.tar.gz) = 247469 Index: pkg/PLIST > =================================================================== > RCS file: /cvs/ports/x11/herbstluftwm/pkg/PLIST,v retrieving revision > 1.2 diff -u -p -u -r1.2 PLIST > --- pkg/PLIST 5 Apr 2014 05:42:07 -0000 1.2 > +++ pkg/PLIST 18 Apr 2016 20:22:58 -0000 > @@ -1,4 +1,5 @@ > @comment $OpenBSD: PLIST,v 1.2 2014/04/05 05:42:07 bcallah Exp $ > +@bin bin/dmenu_run_hlwm > @bin bin/herbstclient > @bin bin/herbstluftwm > @man man/man1/herbstclient.1 > @@ -7,9 +8,9 @@ > share/applications/herbstluftwm.desktop > share/doc/herbstluftwm/ > share/doc/herbstluftwm/BUGS > +share/doc/herbstluftwm/INSTALL > share/doc/herbstluftwm/LICENSE > share/doc/herbstluftwm/NEWS > -share/doc/herbstluftwm/README > share/doc/herbstluftwm/herbstclient.html > share/doc/herbstluftwm/herbstluftwm-tutorial.html > share/doc/herbstluftwm/herbstluftwm.html > @@ -29,9 +30,12 @@ share/examples/herbstluftwm/keychain.sh > share/examples/herbstluftwm/lasttag.sh > share/examples/herbstluftwm/layout.sh > share/examples/herbstluftwm/loadstate.sh > +share/examples/herbstluftwm/maximize.sh > share/examples/herbstluftwm/q3terminal.sh > share/examples/herbstluftwm/savestate.sh > share/examples/herbstluftwm/scratchpad.sh > +share/examples/herbstluftwm/toggledualhead.sh > +share/examples/herbstluftwm/windowmenu.sh > share/examples/herbstluftwm/wselect.sh > share/examples/herbstluftwm/xdg/ > share/examples/herbstluftwm/xdg/herbstluftwm/ > -- http://gmerlin.de OpenPGP: http://gmerlin.de/christopher.pub 2779 7F73 44FD 0736 B67A C410 69EC 7922 34B4 2566
pgp6GK8XnIhyd.pgp
Description: OpenPGP digital signature