Page MenuHomeFreeBSD

net-p2p/transmission-*: update to version 2.94
ClosedPublic

Authored by rigoletto on Mar 14 2018, 10:54 PM.

Details

Summary

PR: 226710

  • major rework
  • update to version 2.94
  • convert Qt[4,5] to FLAVORS
  • take maintainership

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
net-p2p/transmission-qt/Makefile
36 ↗(On Diff #43292)

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

rigoletto marked an inline comment as done.

Done.

net-p2p/transmission-qt/Makefile
36–37 ↗(On Diff #43323)

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.

rigoletto marked an inline comment as done.

Done.

net-p2p/transmission-qt/Makefile
28–29 ↗(On Diff #43338)

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 added a reviewer: portmgr.

Accept, flavor-wise.

Could you add some MOVED entries too?

net-p2p/transmission-qt4|net-p2p/transmission-qt@qt4|...

Could you update the diff making you the maintainer of the whole transmission-suite?

mfg Tobias

I just did a quick update to version 2.94, and changed the maintainer as previously concerted.

I need some time to try to re-work the entire thing aiming to make it more simple to maintain. It is too complicated right now.

Thanks! :)

Minor fix.

Btw, btransmission.cachefly.net gone.

rigoletto retitled this revision from net-p2p/transmission-qt[4,5]: switching to FLAVORS. to net-p2p/transmission-*: update to verson 1.94.Jun 24 2018, 1:43 AM
rigoletto edited the summary of this revision. (Show Details)
rigoletto edited the summary of this revision. (Show Details)
rigoletto retitled this revision from net-p2p/transmission-*: update to verson 1.94 to net-p2p/transmission-*: update to verson 2.94.Jun 24 2018, 6:56 AM

Saving state.

Nothing build, WORKDIR is always empty, the distfile does not extract (and IDK why). :-)

Saving state.

Current situation:

www/transmission-web does not need any LIB_DEPENDS, USES etc. and the only way to arrange it would be:

1- create a separated port
2- add a conditional to disable those for this slave port before LIB_DEPENDS, like was done on the old version and (similar) to the previous FLAVORized version.

net-p2p/transmission-qt:

For some unknown reason "PKGNAMESUFFIX= -${FLAVOR}" is broken (it is ignored). If I set -qt4 or -qt5 just for testing it works.

I need instructions!

Thanks.

In D14699#339060, @lbdm_privacychain.ch wrote:

Saving state.

Current situation:

www/transmission-web does not need any LIB_DEPENDS, USES etc. and the only way to arrange it would be:

1- create a separated port
2- add a conditional to disable those for this slave port before LIB_DEPENDS, like was done on the old version and (similar) to the previous FLAVORized version.

Use something like in the master-port Makefile

. if ${SLAVEPORT:M....}
LIB_DEPENDS=...
.endif

if the libdpeends are used by everything but the web one, you can also use something like

.if ${SLAVEPORT:Nweb}
LIB_DEPENDS=...
.endif

net-p2p/transmission-qt:

For some unknown reason "PKGNAMESUFFIX= -${FLAVOR}" is broken (it is ignored). If I set -qt4 or -qt5 just for testing it works.

This seems to work here.

[00:00:30] [08] [00:00:00] Building net-p2p/transmission-qt@qt5 | transmission-qt5-2.94
[00:00:33] [05] [00:00:00] Building net-p2p/transmission-qt@qt4 | transmission-qt4-2.94

I need instructions!

Thanks.

net-p2p/transmission-daemon/Makefile
21 ↗(On Diff #44449)

^ that should be daemon

net-p2p/transmission-qt/Makefile
18 ↗(On Diff #44449)

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

25 ↗(On Diff #44449)

you need to add

CMAKE_ON= #empty (to not use the default in the masterport)

transmission-qt@qt4 seems to need a patch to include time.h

net-p2p/transmission-qt/Makefile
25 ↗(On Diff #44449)

make that

CMAKE_ON=ENABLE_QT

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

net-p2p/transmission-qt/Makefile
31 ↗(On Diff #44449)

^typo

net-p2p/transmission-cli/Makefile
6 ↗(On Diff #44449)

I would define

SLAVEPORT?= cli
PKGNAMESUFFIX?= -${SLAVEPORT}

here too -- that way you can remove the PKGNAMESUFFIX definition in the slave ports. (the -qt-one will need some work then :) )

28 ↗(On Diff #44449)

^ you can probably do something like the following -- if you define

SLAVES= cli daemon gtk qt 
CMAKE_ON= ENABLE_${SLAVEPORT:tu}
CMAKE_OFF= ${SLAVES:N${SLAVEPORT}:tu:S/^/ENABLE_/}

and get rid of most of the CMAKE_* in the slave ports.

Saving state again.

Evrything build but net-p2p/transmission and net-p2p/transmission-qt@qt4.

PolarSSL OPTION gone, that is not in ports anymore.

Thanks!

Qt4 version is still failing, but that I leave for tomorrow.

Thanks!

Updating to use the new Qt USES. Also

  • need to fix LICENSE for most ports
  • fix Qt:4 build. Seem related with NLS.

transmission-qti@qt4:

Fail on 10.4 if NLS is ON, but build if NLS is OFF. On 11.2 fail on both situations.

rigoletto retitled this revision from net-p2p/transmission-*: update to verson 2.94 to net-p2p/transmission-*: update to version 2.94.Jun 29 2018, 12:31 AM

The distinfo file was not included in the previous update, for unknown reason.

All builds fixed but can't get it right with the translation files of -gtk and -qt versions. Adding %%NLS%% did not help.

Thanks!

net-p2p/transmission-cli/Makefile
58 ↗(On Diff #44670)

NLS_CMAKE_BOOL=ENABLE_NLS

68 ↗(On Diff #44670)

^ This seems broken.

  • the line continuation does not make sense
  • Configure args won't get understood by cmake
73 ↗(On Diff #44670)

^ you need to find the CMAKE option here too :)

net-p2p/transmission-cli/Makefile
42 ↗(On Diff #44670)

^is this still needed with cmake? -- if ${LOCALBASE}/lib is required you can add USES=localbase, but maybe try it without the whole line once

net-p2p/transmission-cli/files/patch-qt_Application.h
7 ↗(On Diff #44670)

^ shouldn't this rather be

#include <time.h>
net-p2p/transmission-qt/Makefile
19 ↗(On Diff #44670)

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

rigoletto marked 4 inline comments as done.

Done.

net-p2p/transmission-cli/Makefile
42 ↗(On Diff #44670)

Unfortunately yes, otherwise fail. :/

76 ↗(On Diff #44880)

Apparently not needed anymore. I built it in here and tested. Seem to be working.

net-p2p/transmission-cli/files/patch-qt_Application.h
7 ↗(On Diff #44880)

Should this be up-streamed or this is a FreeBSD specific?

net-p2p/transmission-cli/Makefile
42 ↗(On Diff #44670)

Then that is a bug in FindDHT.cmake of transmission, respectively how it is used :) -- it should either return the full librarypath, or add the library-dir to the linkdirectories for the targets that use it.

net-p2p/transmission-cli/files/patch-qt_Application.h
7 ↗(On Diff #44880)

Yeah, that sounds very upstreamable

rigoletto marked 2 inline comments as done.

Small advance on removing LDFLAGS.

New errors:

cli/CMakeFiles/transmission-cli.dir/cli.c.o: In function `tr_strlratio':
cli.c:(.text+0x8ed): undefined reference to `libintl_gettext'
cc: error: linker command failed with exit code 1 (use -v to see invocation)

Thanks!

This revision was not accepted when it landed; it landed in state Needs Review.Jul 15 2018, 5:01 PM
This revision was automatically updated to reflect the committed changes.