Page MenuHomeFreeBSD

New port: audio/abgate-lv2: Noise gate LV2 plugin
ClosedPublic

Authored by yuri on Nov 7 2017, 11:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 16, 10:50 PM
Unknown Object (File)
Thu, Oct 16, 10:50 PM
Unknown Object (File)
Thu, Oct 16, 10:50 PM
Unknown Object (File)
Thu, Oct 16, 10:50 PM
Unknown Object (File)
Thu, Oct 16, 10:50 PM
Unknown Object (File)
Thu, Oct 16, 10:38 PM
Unknown Object (File)
Fri, Sep 26, 11:00 AM
Unknown Object (File)
Thu, Sep 25, 2:51 PM
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 12551
Build 12826: arc lint + arc unit

Event Timeline

audio/abgate-lv2/Makefile
20

Change PORTNAME to aBGate and remove the GH_PROJECT line.

Changed PORTNAME to mixed case.

yuri marked an inline comment as done.Nov 8 2017, 5:15 PM

Maybe also rename the directory to abGate-lv2 -- I personally like it if it is matching, but I think there are others that are very opposed to capital letters :)

Maybe also rename the directory to abGate-lv2 -- I personally like it if it is matching, but I think there are others that are very opposed to capital letters :)

I also would like directory to be abGate, but I named one port's directory qTox once, and I remember getting unhappy e-mails about this.
I will ask for extra-opinions on ports@.

Maybe also rename the directory to abGate-lv2 -- I personally like it if it is matching, but I think there are others that are very opposed to capital letters :)

Kurt Jaeger opined that he is for lowercase, let's just keep it lowercase?

audio/abgate-lv2/Makefile
32

you should not use ${LOCALBASE}/llib/qt4/bin/qmake -- qmake should work. [QT_SELECT=qt4 is in both CONFIGURE_ENV and MAKE_ENV].

If you need the path to qmake, you should ask bsd.qt.mk for it, and use ${QMAKE}, or ${QT_BINDIR}/qmake.
Which seems more robust than to just assume it' in ${LOCALBASE}/lib/qt4/bin/

yuri marked an inline comment as done.Nov 11 2017, 4:34 PM
This revision is now accepted and ready to land.Nov 11 2017, 5:48 PM
This revision was automatically updated to reflect the committed changes.