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, Nov 27, 7:52 AM
Unknown Object (File)
Sat, Nov 22, 8:55 PM
Unknown Object (File)
Sat, Nov 22, 9:26 AM
Unknown Object (File)
Fri, Nov 21, 9:19 AM
Unknown Object (File)
Fri, Nov 21, 3:01 AM
Unknown Object (File)
Wed, Nov 19, 5:45 AM
Unknown Object (File)
Tue, Nov 18, 6:46 AM
Unknown Object (File)
Mon, Nov 17, 6:22 AM
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.