Page MenuHomeFreeBSD
Differential D14699 Diff 44396 net-p2p/transmission-qt/Makefile

Changeset View

Standalone View

View Options

net-p2p/transmission-qt/Makefile

  • This file was added.
PropertyOld ValueNew Value
svn:eol-stylenullnative
\ No newline at end of property
svn:keywordsnullFreeBSD=%H
\ No newline at end of property
svn:mime-typenulltext/plain
\ No newline at end of property
# $FreeBSD$
PKGNAMESUFFIX= -${FLAVOR}
LIB_DEPENDS+= libfontconfig.so:x11-fonts/fontconfig \
libfreetype.so:print/freetype2
FLAVORS= qt4 qt5
FLAVOR?= ${FLAVORS:[1]}
qt4_CONFLICTS_INSTALL= transmission-qt5
qt5_CONFLICTS_INSTALL= transmission-qt4
USES= gettext-runtime:run desktop-file-utils
.if ${FLAVOR} == qt4
USES+= gettext-runtime:run
matUnsubmitted
Done
Inline Actions

This should happen later. See Chapter 15. Order of Variables in Port Makefiles.

mat: This should happen later. See [[ https://www.freebsd.org/doc/en/books/porters-handbook/porting…
USE_QT4= moc_build qmake_build rcc_build uic_build corelib dbus gui network xml
CMAKE_OFF= ENABLE_CLI ENABLE_DAEMON ENABLE_GTK ENABLE_NLS ENABLE_UTILS
tcbernerUnsubmitted
Done
Inline Actions

^ you should probably set the shared USES lines in the master port, with += and only add the qt-specific here.

tcberner: ^ you should probably set the shared USES lines in the master port, with += and only add the qt…
tcbernerUnsubmitted
Done
Inline Actions

^ you should add
CMAKE_OFF+=USE_QT5 for the qt4 one

tcberner: ^ you should add CMAKE_OFF+=USE_QT5 for the qt4 one
.endif
matUnsubmitted
Done
Inline Actions

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

mat: Wrong place in the Makefile. See [[ https://www.freebsd.org/doc/en/books/porters…
.if ${FLAVOR} == qt5
USES+= gettext-runtime:run
matUnsubmitted
Done
Inline Actions

Both USES lines are the same, extract them from the .if.

mat: Both USES lines are the same, extract them from the .if.
USE_QT5= buildtools_build linguisttools_build qmake_build \
core dbus gui network widgets
CAKE_OFF= ENABLE_CLI ENABLE_DAEMON ENABLE_NLS ENABLE_GTK \
tcbernerUnsubmitted
Done
Inline Actions

you need to add

CMAKE_ON= #empty (to not use the default in the masterport)
tcberner: you need to add ``` CMAKE_ON= #empty (to not use the default in the masterport) ```
tcbernerUnsubmitted
Done
Inline Actions

make that

CMAKE_ON=ENABLE_QT

othwerise you'll get -D:BOOL=TRUE added .... that needs to be fixed in cmake.mk :D

tcberner: make that ``` CMAKE_ON=ENABLE_QT ``` othwerise you'll get `-D:BOOL=TRUE` added .... that…
ENABLE_UTILS
CMAKE_ON= USE_QT5
.endif
matUnsubmitted
Done
Inline Actions

Still wrong. up, up, with the flavors.

Could you please read Chapter 15. Order of Variables in Port Makefiles, and apply its content. It would go so much faster than me having to knit pick every lines.

mat: Still wrong. up, up, with the flavors. Could you please read [[ https://www.freebsd.
MASTERDIR= ${.CURDIR}/../transmission
DESCR= ${.CURDIR}/pkg-descr
tcbernerUnsubmitted
Done
Inline Actions

^typo

tcberner: ^typo
PLIST= ${.CURDIR}/pkg-plist
SLAVEPORT= qt
.include "${MASTERDIR}/Makefile"
matUnsubmitted
Done
Inline Actions

This looks bogus.

mat: This looks bogus.
rigolettoAuthorUnsubmitted
Done
Inline Actions

qt[4,5] versions can not be installed together because several files are called 'transmission-qt' and would conflict.

rigoletto: qt[4,5] versions can not be installed together because several files are called 'transmission…
matUnsubmitted
Done
Inline Actions

Yes, good, I had understood that. The line is still wrong.

mat: Yes, good, I had understood that. The line is still wrong.
matUnsubmitted
Done
Inline Actions

This is still wrong.

mat: This is still wrong.
matUnsubmitted
Done
Inline Actions

This should happen earlier.

mat: This should happen earlier.
tcbernerUnsubmitted
Done
Inline Actions

Conflicts Documentation :)

tcberner: [[https://www.freebsd.org/doc/en/books/porters-handbook/conflicts.html|Conflicts…
tcbernerUnsubmitted
Done
Inline Actions

^ this still looks wrong.
I would probably just set something like

qt4_CONFLICTS_INSTALL= transmission-qt5
qt5_CONFLICTS_INSTALL= transmission-qt4

you could also do it a bit more magically/confusing in one line with something like

${FLAVOR}_CONFLICTS_INSTALL=transmission-${FLAVORS:N${FLAVOR}}

:P

tcberner: ^ this still looks wrong. I would probably just set something like ``` qt4_CONFLICTS_INSTALL=…
tcbernerUnsubmitted
Done
Inline Actions

Also, as a side-note: the master port is overriding whatever you set directly as CONFLICTS_INSTALL :)

you can check the values you need by doing for example

/usr/ports # make -VPKGBASE FLAVOR=qt4 -C net-p2p/transmission-qt
transmission-qt4
/usr/ports # make -VPKGBASE FLAVOR=qt5 -C net-p2p/transmission-qt
transmission-qt5

In the qt4-flavor you therefore want to conflict with transmission-qt5 and vice versa.

tcberner: Also, as a side-note: the master port is overriding whatever you set directly as…
matUnsubmitted
Done
Inline Actions

That is extremely fragile, and extremely obfuscated, please expand it in the two lines it ends up doing.

mat: That is extremely fragile, and extremely obfuscated, please expand it in the two lines it ends…
matUnsubmitted
Done
Inline Actions

Wrong place in the Makefile, they are flavors helpers, they should happen in the flavors block. See Chapter 15. Order of Variables in Port Makefiles.

mat: Wrong place in the Makefile, they are flavors helpers, they should happen in the flavors block.