Opened 11 years ago

Last modified 15 months ago

#39424 new enhancement

Ports that use qmake: consider using the new qmake portgroup — at Version 35

Reported by: cooljeanius (Eric Gallager) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: portgroup Cc: blair@…, cal@…, css@…, ctreleaven@…, dports@…, easieste@…, ecronin@…, macports@…, mcalhoun@…, michaelld@…, mk@…, mnick@…, nicos@…, ocroquette@…, phw@…, pixilla@…, rowue@…, rudloff@…, ryandesign@…, sami.laine@…, snc@…, khindenburg@…
Port: arora clustalx GLC_lib goldendict k3dsurf kdiff3 OpenCSG openscad psi-otr qgit QMK-Groundstation qt-assistant qt4-creator-mac qtc-qtsharp qtpfsgui qtscriptgenerator qtsharp QupZilla qwt qxmpp sqlitedbrowser tea tiled

Description (last modified by michaelld (Michael Dickens))

(Made as a separate ticket per comment:ticket:38280:22) There is now a qmake portgroup as of r106912. Here is a list of ports that use qmake somewhere in their respective portfiles:

[moved to attachment:qmake_search.txt because it’s pointless and makes ticket modification emails enormous. —larryv]

List of ports that still need to be fixed:

  • arora (strasweb.fr:rudloff, openmaintainer)
  • clustalx (snc, openmaintainer)
  • GLC_lib (lilalinux.net:macports)
  • k3dsurf (nomaintainer, purely based on qt3)
  • psi-otr (rowue)
  • qgit (free.fr:ocroquette, openmaintainer)
  • QMK-Groundstation (lilalinux.net:macports)
  • qscintilla (michaelld, openmaintainer)
  • qt-assistant (michaelld, openmaintainer)
  • qt4-creator-mac (sicherha.de:macports, openmaintainer)
  • qt4-mac-mysql55-plugin (michaelld, pixilla, openmaintainer)
  • qt4-mac-postgresql91-plugin (michaelld, pixilla, openmaintainer)
  • qt4-mac-sqlite2-plugin (michaelld, pixilla, openmaintainer)
  • qtc-qtsharp (nomaintainer)
  • qtpfsgui (gmail.com:clubjuggler, openmaintainer)
  • qtscriptgenerator (michaelld, openmaintainer)
  • qtsharp (nomaintainer)
  • qwt (michaelld, phw, openmaintainer)
  • qxmpp (nomaintainer)
  • sqlitedbrowser (phw, openmaintainer)
  • texstudio (nomaintainer)
  • x2goclient (cal, openmaintainer)

List of completed/irelevant ports:

Maintainers of these ports, consider whether you could use the qmake portgroup for your respective ports.

Change History (36)

comment:1 Changed 11 years ago by ctreleaven (Craig Treleaven)

I've browsed the linked tickets, but I didn't notice a reason that this is a separate port group. Why not just add these bits to the qt4 port group (maybe version 2.0)?

Also, I noticed the line:

    configure.pre_args-delete       --prefix=${prefix}

I'm pretty sure mythtv-core.25 needs that argument. Is it harmful to other ports?

comment:2 in reply to:  1 Changed 11 years ago by cooljeanius (Eric Gallager)

Replying to ctreleaven@…:

I've browsed the linked tickets, but I didn't notice a reason that this is a separate port group. Why not just add these bits to the qt4 port group (maybe version 2.0)?

I wanted to just include this in the qt4 portgroup, too, but apparently not all ports that use the qt4 portgroup use qmake, so that's why there's now a separate qmake portgroup. See comment:ticket:38272:1 for reference.

Last edited 11 years ago by cooljeanius (Eric Gallager) (previous) (diff)

comment:3 Changed 11 years ago by NicosPavlov

The portgroup is not meant for all ports using qmake. For example, several ports use qmake within standard autotools, thus compiling by

configure
make

These ports are not concerned by this portgroup, as they define variables during the configuration phase. However, several other ports must be compiled through

qmake
make

The portgroup aims at setting variables for these ports.

Typically, in case the configuration phase is made with qmake, the --prefix argument is not used, and PREFIX is, hence the changes of variables in the portgroup. Due to these differences, it did not appear suitable to include these in the qt4 portgroup which is more general for ports depending on qt4.

comment:4 Changed 11 years ago by mkae (Marko Käning)

Done for abtransfers in r106975

comment:5 Changed 11 years ago by mkae (Marko Käning)

I think gwenhywfar4 should be taken out of the above list, since it uses configure!

comment:6 in reply to:  5 Changed 11 years ago by cooljeanius (Eric Gallager)

Replying to mk@…:

I think gwenhywfar4 should be taken out of the above list, since it uses configure!

I'd edit it to do so, except people without commit access can't edit the OP's of tickets (even if they're their own tickets), so I can't (yet)...

comment:7 Changed 11 years ago by ctreleaven (Craig Treleaven)

mythtv-core.25 also uses ./configure and would not benefit from this portgroup.

(And yes, tickets can be assigned to us...but we can't close them or do anything more than comment on them!)

comment:8 Changed 11 years ago by larryv (Lawrence Velázquez)

Description: modified (diff)
Port: gwenhywfar4 mythtv-core.25 removed

Removed gwenhywfar4 and mythtv-core.25.

comment:9 in reply to:  description Changed 11 years ago by larryv (Lawrence Velázquez)

Replying to egall@…:

Here is a list of ports that use qmake somewhere in their respective portfiles:

By the way, please use better judgment in the future when pasting a ton of output from logs or stdout into ticket descriptions or comments. Where by “use better judgment” I mean “don’t do it”.

comment:10 Changed 11 years ago by NicosPavlov

Description: modified (diff)

qtiplot done in r106995.
Suppressing djview as it uses configure.

comment:11 Changed 11 years ago by raphael-st (Raphael Straub)

Cc: raphael@… removed
Description: modified (diff)

libQGLViewer done in r107011.

comment:12 Changed 11 years ago by mkae (Marko Käning)

Description: modified (diff)

modified abtransfers' Portfile in r107026

Changed 11 years ago by larryv (Lawrence Velázquez)

Attachment: qmake_search.txt added

original search output, removed from ticket description

comment:13 Changed 11 years ago by larryv (Lawrence Velázquez)

Description: modified (diff)

comment:14 Changed 11 years ago by NicosPavlov

Description: modified (diff)

I passed through the listed ports, and tried to clean up the irrelevant ones, i.e. the ones for which the quake portgroup would be of no use.

  • Ports using standard autotools (eg ./configure): bitdht, cmake, doxygen, qca, qfsm, Vidalia
  • Ports using cmake: gnuradio
  • Ports using python: py-pyqt4, py26-pyqt3
  • Ports without configure phase: xxdiff

Additionally, I noted that some ports employ some conditional code to take into account the case of qt3, which is not considered in the portgroup for now.

I also made a note beside ports which have no maintainer.

comment:15 Changed 11 years ago by neverpanic (Clemens Lang)

What's the rationale behind setting configure.* in a pre-configure block in the PortGroup? This makes it difficult to change the defaults given in the PortGroup.

comment:16 in reply to:  15 Changed 11 years ago by cooljeanius (Eric Gallager)

Replying to cal@…:

What's the rationale behind setting configure.* in a pre-configure block in the PortGroup? This makes it difficult to change the defaults given in the PortGroup.

I think nicos did that in comment:ticket:38280:20...

comment:17 Changed 11 years ago by NicosPavlov

Sorry I did not answer right away.

If I did put those in a pre-configure block, it is because otherwise, the flags were not set right if the portfile was for example setting configure.blacklist, as flags such as QMAKE_CC must be set accordingly, and after having blacklisted compilers. This is also why all flags are with -delete or -append. However, I fully get the point, and if there is another more convenient way of ensuring that some commands are followed that I am not aware of, it surely could improve the portgroup.

comment:18 Changed 11 years ago by easye

Description: modified (diff)

Addressed for finance/bitcoin in r108515.

comment:19 Changed 11 years ago by macports@…

Fixed for qt4-creator-mac in r107796.

comment:20 Changed 11 years ago by sharky@…

Cc: sharky@… removed

Cc Me!

comment:21 Changed 11 years ago by mojca (Mojca Miklavec)

Port: abtransfers bitcoin bitdht cmake djview doxygen gnuradio libQGLViewer py-pyqt4 py26-pyqt3 qca qfsm qtiplot Vidalia xxdiff removed

Removed abtransfers bitcoin bitdht cmake djview doxygen gnuradio libQGLViewer py-pyqt4 py26-pyqt3 qca qfsm qtiplot Vidalia xxdiff.

comment:22 Changed 11 years ago by mf2k (Frank Schima)

Cc: mf2k@… added; macsforever2000@… removed

comment:23 Changed 11 years ago by mf2k (Frank Schima)

Description: modified (diff)
Version: 2.1.3

comment:24 Changed 10 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:25 Changed 10 years ago by mf2k (Frank Schima)

OpenCSG switched in r120659.

comment:26 Changed 10 years ago by mf2k (Frank Schima)

Description: modified (diff)

openscad r125699

comment:27 Changed 10 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:28 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

QupZilla done in r125940

comment:29 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

tiled done in r125941

comment:30 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)

tea done in r125942

comment:31 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)

goldendict done in r125945

comment:32 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:33 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Description: modified (diff)

kdiff3 done in r127115

comment:34 Changed 10 years ago by mf2k (Frank Schima)

Cc: mf2k@… removed

comment:35 Changed 9 years ago by michaelld (Michael Dickens)

Description: modified (diff)
Port: qwtplot3d removed

qwtplot3d done in r138217.

Note: See TracTickets for help on using tickets.