Re: UPDATE: herbstluftwm to 0.7.0
Florian Stinglmayr writes: >> OK chrisz@ > > Ping? Any other comments or oks? > Committed, thanks. -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
Re: UPDATE: herbstluftwm to 0.7.0
> OK chrisz@ Ping? Any other comments or oks?
Re: UPDATE: herbstluftwm to 0.7.0
On 2016-04-18 Florian Stinglmayr 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 - 1.4 > +++ Makefile 18 Apr 2016 20:22:58 - > @@ -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 > + > # 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 > 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 - 1.2 > +++ distinfo 18 Apr 2016 20:22:58 - > @@ -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 - 1.2 > +++ pkg/PLIST 18 Apr 2016 20:22:58 - > @@ -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
Re: UPDATE: herbstluftwm to 0.7.0
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. It's weird btw. Their entire code base is basically C with a few C++11 things mixed in 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 --- Makefile25 Aug 2015 13:18:28 - 1.4 +++ Makefile18 Apr 2016 20:22:58 - @@ -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 + # 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 Index: distinfo === RCS file: /cvs/ports/x11/herbstluftwm/distinfo,v retrieving revision 1.2 diff -u -p -u -r1.2 distinfo --- distinfo5 Apr 2014 05:42:07 - 1.2 +++ distinfo18 Apr 2016 20:22:58 - @@ -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 - 1.2 +++ pkg/PLIST 18 Apr 2016 20:22:58 - @@ -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/
Re: UPDATE: herbstluftwm to 0.7.0
On 2016-04-18 Florian Stinglmayr wrote: > Hi list, > > below is an update for herbstluftwm to 0.7.0 which was released > earlier this year. Some new features are: Hi, 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. I could remove intl, m and pthread from WANTLIB, after adding devel/gettext to MODULES. > The port runs well for me but I have one question: dmenu_run_hlwm > requires x11/dmenu (so RUN_DEPENDS) obviously, but the default > herbstluftwm does not mention or use dmenu_run_hlwm. How should I > handle this: Either with a fixed RUN_DEPENDS on dmenu_run_hlwm or > just with a note that suggests to install dmenu if you wish to use > dmenu_run_hlwm? a fixed RUN_DEPENDS is overkill. You may mention dmenu in a MESSAGE, if you want to. I don't think it is necessary. > Also I am gladly taking maintainer for this. great! thanks. > Regards, > Florian > > 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 - 1.4 > +++ Makefile 18 Apr 2016 16:45:43 - > @@ -1,20 +1,26 @@ > # $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 > + > # 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= * > + > +BUILD_DEPENDS= lang/gcc/4.9,-c++>=4.9 > > LIB_DEPENDS =devel/glib2 > > @@ -23,7 +29,7 @@ RUN_DEPENDS = devel/desktop-file-utils \ > x11/dzen2,-gadgets > > USE_GMAKE = Yes > -MAKE_FLAGS = CC="${CC}" LDFLAGS= VERBOSE= COLOR=0 > +MAKE_FLAGS = CC="egcc" LDFLAGS= VERBOSE= COLOR=0 > > BASEDIR =${PREFIX}/share/examples/herbstluftwm > FAKE_FLAGS = SYSCONFDIR="${BASEDIR}" \ > @@ -33,9 +39,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 > 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 - 1.2 > +++ distinfo 18 Apr 2016 16:45:43 - > @@ -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 - 1.2 > +++ pkg/PLIST 18 Apr 2016 16:45:43 - > @@ -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 pgpija6K7jKxE.pgp Description: OpenPGP digital signature
UPDATE: herbstluftwm to 0.7.0
Hi list, below is an update for herbstluftwm to 0.7.0 which was released earlier this year. Some new features are: * EWMH request _NET_WM_MOVERESIZE is handled better now. At least for me this fixed weird drawing issues with firefox if the frame is resized. * New command: close_and_remove * Tags are now accessible through their index * Native support for dmenu (through dmenu_run_hlwm) * Post patch is no longer needed, hardcoded paths have been replaced by calls to /usr/bin/env. The port runs well for me but I have one question: dmenu_run_hlwm requires x11/dmenu (so RUN_DEPENDS) obviously, but the default herbstluftwm does not mention or use dmenu_run_hlwm. How should I handle this: Either with a fixed RUN_DEPENDS on dmenu_run_hlwm or just with a note that suggests to install dmenu if you wish to use dmenu_run_hlwm? Also I am gladly taking maintainer for this. Regards, Florian Index: Makefile === RCS file: /cvs/ports/x11/herbstluftwm/Makefile,v retrieving revision 1.4 diff -u -p -u -r1.4 Makefile --- Makefile25 Aug 2015 13:18:28 - 1.4 +++ Makefile18 Apr 2016 16:45:43 - @@ -1,20 +1,26 @@ # $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 + # 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= * + +BUILD_DEPENDS= lang/gcc/4.9,-c++>=4.9 LIB_DEPENDS = devel/glib2 @@ -23,7 +29,7 @@ RUN_DEPENDS = devel/desktop-file-utils \ x11/dzen2,-gadgets USE_GMAKE =Yes -MAKE_FLAGS = CC="${CC}" LDFLAGS= VERBOSE= COLOR=0 +MAKE_FLAGS = CC="egcc" LDFLAGS= VERBOSE= COLOR=0 BASEDIR = ${PREFIX}/share/examples/herbstluftwm FAKE_FLAGS = SYSCONFDIR="${BASEDIR}" \ @@ -33,9 +39,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 Index: distinfo === RCS file: /cvs/ports/x11/herbstluftwm/distinfo,v retrieving revision 1.2 diff -u -p -u -r1.2 distinfo --- distinfo5 Apr 2014 05:42:07 - 1.2 +++ distinfo18 Apr 2016 16:45:43 - @@ -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 - 1.2 +++ pkg/PLIST 18 Apr 2016 16:45:43 - @@ -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/