Opened 11 years ago
Last modified 15 months ago
#39424 new enhancement
Ports that use qmake: consider using the new qmake portgroup — at Version 52
Reported by: | cooljeanius (Eric Gallager) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | portgroup | Cc: | blair (Blair Zajac), neverpanic (Clemens Lang), cssdev, ctreleaven (Craig Treleaven), drkp (Dan Ports), easye, ecronin (Eric Cronin), macports@…, MarcusCalhoun-Lopez (Marcus Calhoun-Lopez), michaelld (Michael Dickens), mkae (Marko Käning), mnick@…, NicosPavlov, ocroquette (Olivier Croquette), phw@…, pixilla (Bradley Giesbrecht), rowue@…, rudloff@…, ryandesign (Ryan Carsten Schmidt), sami.laine@…, nerdling (Jeremy Lavergne), kurthindenburg (Kurt Hindenburg) |
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 kencu (Ken))
(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:
- k3dsurf (nomaintainer) - purely based on qt3. upstream inactive since 2005.
- psi-otr (rowue)
- qgit (nomaintainer)
- qt-assistant (michaelld, openmaintainer)
- qtc-qtsharp (nomaintainer) - Upstream homepage dead
- qtscriptgenerator (michaelld, openmaintainer)
- qtsharp (nomaintainer) - Upstream homepage dead
- qwt (michaelld, openmaintainer)
- qxmpp (nomaintainer)
- sqlitedbrowser (nomaintainer)
List of completed/irrelevant ports:
abtransfers[r106975, see comment:4 and r107026, see comment:12]arora(nomaintainer)bitcoin[r108515, see comment:18]bitdht[no benefit, see comment:14]clustalx(snc, openmaintainer)cmake[no benefit, see comment:14]djview[no benefit, see comment:10]doxygen[no benefit, see comment:14]gnuradio[no benefit, see comment:14]goldendict (nomaintainer)[r125945, see comment:31]gwenhywfar4[no benefit, see comment:5]kdiff3 (nomaintainer)[127115, see comment:33]libQGLViewer[r107011, see comment:11]mythtv-core.25[no benefit, see comment:7]OpenCSG[r120659, see comment:25]openscad (mf2k, openmaintainer)[r125699, see comment:26]py-pyqt4[no benefit, see comment:14]py26-pyqt3[no benefit, see comment:14]qca[no benefit, see comment:14]qfsm[no benefit, see comment:14]qscintilla(michaelld, openmaintainer) b9de20c6qt4-creator-mac(michaelld, openmaintainer)qtpfsgui(gmail.com:clubjuggler, openmaintainer) - Not in Macportsqtiplot[r106995, see comment:10]QMK-Groundstation(lilalinux.net:macports) [r140991]QupZilla (ryandesign, openmaintainer)[r125940, see comment:28]tea (nomaintainer)[r125942, see comment:30]texstudio(48565da617e90ab46887be37c9b8b551284c9c05/macports-ports, see comment:43)tiled (nomaintainer)[r125941, see comment:29]Vidalia[no benefit, see comment:14]xxdiff[no benefit, see comment:14]x2goclient(2b292f41894577ca90da9fa6d8dc0c5989977933/macports-ports)
Maintainers of these ports, consider whether you could use the qmake portgroup for your respective ports.
Change History (53)
comment:1 follow-up: 2 Changed 11 years ago by ctreleaven (Craig Treleaven)
comment:2 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.
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:5 follow-up: 6 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 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 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 follow-up: 16 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 Changed 11 years ago by cooljeanius (Eric Gallager)
Replying to cal@…:
What's the rationale behind setting
configure.*
in apre-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: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: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: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: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.
comment:36 Changed 9 years ago by michaelld (Michael Dickens)
QMK-Groundstation has a patch for this in #48226.
comment:37 Changed 9 years ago by michaelld (Michael Dickens)
Description: | modified (diff) |
---|---|
Port: | qt3 qt3-mac qt4-mac qt4-mac-mysql55-plugin qt4-mac-postgresql91-plugin qt4-mac-sqlite2-plugin qt4-x11 removed |
comment:38 Changed 9 years ago by michaelld (Michael Dickens)
Description: | modified (diff) |
---|
QMK-Groundstation done in r140991.
comment:39 Changed 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
qtpfsgui fixed in r149176.
comment:40 Changed 8 years ago by mojca (Mojca Miklavec)
Description: | modified (diff) |
---|---|
Port: | qscintilla removed |
qscintilla fixed in b9de20c6
comment:41 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
comment:42 Changed 6 years ago by yan12125 (Chih-Hsuan Yen)
Cc: | yan12125 added |
---|
comment:43 Changed 6 years ago by yan12125 (Chih-Hsuan Yen)
comment:44 Changed 6 years ago by yan12125 (Chih-Hsuan Yen)
Description: | modified (diff) |
---|---|
Port: | texstudio removed |
comment:45 Changed 6 years ago by mf2k (Frank Schima)
Description: | modified (diff) |
---|
comment:46 Changed 6 years ago by mf2k (Frank Schima)
Description: | modified (diff) |
---|
comment:47 Changed 6 years ago by mf2k (Frank Schima)
Description: | modified (diff) |
---|
comment:48 follow-up: 49 Changed 6 years ago by Mihai Moldovan <ionic@…>
comment:49 Changed 6 years ago by Ionic (Mihai Moldovan)
Description: | modified (diff) |
---|---|
Port: | x2goclient removed |
comment:50 Changed 5 years ago by yan12125 (Chih-Hsuan Yen)
Cc: | yan12125 removed |
---|
comment:51 Changed 5 years ago by kencu (Ken)
comment:52 Changed 5 years ago by kencu (Ken)
Description: | modified (diff) |
---|
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:
I'm pretty sure mythtv-core.25 needs that argument. Is it harmful to other ports?