On Fri, Mar 05, 2021 at 08:34:08AM +0100, Guy Godfroy wrote:
> Hello,
> 
> Here is the new tarball with requested changes.
> 
> Guy
> 
> Le 05/03/2021 à 00:17, Solene Rapenne a écrit :
> > On Thu, 4 Mar 2021 21:51:30 +0100
> > Guy Godfroy <[email protected]>:
> > 
> >> Hello,
> >>
> >> xmenu is a contextual menu tool for x11. I used pmenu port proposal I 
> >> saw today (developped by the same guy), very few changes. Thanks to 
> >> Jerome for that.
> >>
> >> xmenu and pmenu have in common to take their data from stdin following 
> >> the same syntax. xmenu was created a bit before pmenu, which explains 
> >> maybe some oddity in the config.mk that required a patch.
> >>
> >> Works on my amd64.
> >>
> >> Guy Godfroy
> >>
> > 
> > 
> > 
> > For consistency the space before the = sign should be put here
> > LIB_DEPENDS= graphics/imlib2
> > 
> > like x11/pmenu, I don't know if it's important but WANTLIB in
> > automatically generated order would look like
> > 
> > WANTLIB += Imlib2 X11 Xft Xinerama c fontconfig
> > 
> > With theses changes, ok solene@
> > 

This could use the same treatment as pmenu. You can kill the patch and
use:

MAKE_FLAGS =    PREFIX=${PREFIX} \
                FREETYPEINC=${X11BASE}/include/freetype2 \
                MANPREFIX=${PREFIX}/man

-- 

Tracey Emery

Attachment: xmenu.tgz
Description: Binary data

Reply via email to