On 2019 Feb 08 (Fri) at 20:34:00 +0100 (+0100), Ingo Feinerer wrote:
:On Fri, Nov 23, 2018 at 05:41:09PM +0100, Ingo Feinerer wrote:
:> Brian Callahan wrote:
:> > Joel Carnat wrote:
:> > > On a journey to find a statusbar for cwm, I found that x11/lemonbar
:> > > didn't support XFT. I used sources from https://github.com/krypt-n/bar
:> > > to compile on OpenBSD 6.4/amd64 and it seems to work. So far, it
:> > > compiles, lemonbar starts and renders text using xft:Sans:size=11.
:> 
:> Thanks for investigating and preparing a patch!
:> 
:> > This would additionally need a REVISION bump in the port Makefile.
:> > But this seems rather large and invasive.
:> 
:> I agree that this patch is quite invasive.
:> 
:> > That's not a "no" in and of itself, but would it make more sense to
:> > send this upstream
:> 
:> Since there is a issue (https://github.com/LemonBoy/bar/issues/188) from
:> Aug 5, 2016 with no progress I do not think that this has good chances.
:> But who knows ...
:> 
:> > or perhaps switch to that forked repo with this already included?
:> 
:> I think that is the way to go. The few commits in LemonBoy/bar in the
:> last two years are mainly of cosmetic nature (see GitHub history), so
:> the forked repo has the original and new functionality.
:
:A few days later ...
:
:Here is a diff that switches lemonbar to the krypt-n/bar fork supporting
:XFT. A tarball is attached for two patches (I wanted to avoid `cvs add`
:on the patches directory). The inline diff just shows the changes to the
:Makefile and distinfo.
:
:At the same time add pledge() before the main processing loop. It is
:nice to see that "stdio" is enough at this point (after initialization
:which needed at least "rpath" is my tests; likely for loading some
:fonts).
:
:Alternatively we could create a new port for the fork (e.g.,
:lemonbar-xft). But since both LemonBoy/bar and krypt-n/bar have not seen
:a lot of changes recently we won't miss anything important if we just
:switch.
:

I tried running it, but it kept puking on my existing scripts.

Also, this branch is quite a few commits behind the existing one.

I'd *much* rather we not change the upstream.  Either a new port or the
big invasive patch.  Considering how inactive both projects are,
carrying a patch isn't completely terrible :/.


:So OK for the switch?
:
:Best regards,
:Ingo
:
:Index: Makefile
:===================================================================
:RCS file: /cvs/ports/x11/lemonbar/Makefile,v
:retrieving revision 1.5
:diff -u -p -r1.5 Makefile
:--- Makefile   21 Nov 2017 06:08:37 -0000      1.5
:+++ Makefile   8 Feb 2019 18:52:54 -0000
:@@ -2,13 +2,14 @@
: 
: COMMENT =     featherweight lemon-scented bar
: 
:-VERSION =     1.3
:+VERSION =     1.1_20180416
:+EPOCH =               0
: 
:-PKGNAME =     lemonbar-${VERSION}
:+DISTNAME =    lemonbar-${VERSION}
: 
:-GH_ACCOUNT =  LemonBoy
:+GH_ACCOUNT =  krypt-n
: GH_PROJECT =  bar
:-GH_TAGNAME =  v${VERSION}
:+GH_COMMIT =   00a3fe6f3485203896156480e86014136749a221
: 
: CATEGORIES =  x11
: 
:@@ -17,7 +18,9 @@ MAINTAINER =         Ingo Feinerer <feinerer@lo
: # MIT
: PERMIT_PACKAGE_CDROM =        Yes
: 
:-WANTLIB += c xcb xcb-randr xcb-xinerama
:+# uses pledge()
:+WANTLIB += X11 X11-xcb Xft c fontconfig freetype xcb xcb-randr
:+WANTLIB += xcb-xinerama z
: 
: COMPILER =            base-clang ports-gcc
: COMPILER_LANGS =      c
:@@ -31,9 +34,6 @@ NO_TEST =            Yes
: CFLAGS +=             -I${X11BASE}/include
: 
: ALL_TARGET =          all doc
:-
:-pre-configure:
:-      sed -i '/^CFLAGS/s/-Os //' ${WRKSRC}/Makefile
: 
: do-install:
:       ${INSTALL_DATA_DIR} ${PREFIX}/share/doc/lemonbar/
:Index: distinfo
:===================================================================
:RCS file: /cvs/ports/x11/lemonbar/distinfo,v
:retrieving revision 1.2
:diff -u -p -r1.2 distinfo
:--- distinfo   21 Nov 2017 06:08:37 -0000      1.2
:+++ distinfo   8 Feb 2019 18:52:54 -0000
:@@ -1,2 +1,2 @@
:-SHA256 (bar-1.3.tar.gz) = 6ueExeN1stCxITi+GpkCvrS17rgH81EvMPFLoj3ao30=
:-SIZE (bar-1.3.tar.gz) = 14930
:+SHA256 (lemonbar-1.1_20180416-00a3fe6f.tar.gz) = 
wNz3vlI37dHYiA/Zg0WIPDRDsxl6BFd6nYKuGzjDA94=
:+SIZE (lemonbar-1.1_20180416-00a3fe6f.tar.gz) = 16715



-- 
Q:  How many heterosexual males does it take to screw in a light bulb
    in San Francisco?
A:  Both of them.

Reply via email to