Opened 10 years ago

Closed 6 years ago

#47047 closed submission (fixed)

new (meta)port submission: qt5-mac-mysql-plugins

Reported by: RJVB (René Bertin) Owned by:
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: michaelld (Michael Dickens), pixilla (Bradley Giesbrecht), chrstphrchvz (Christopher Chavez)
Port: qt5-mac-mysql-plugins

Description

Attached a portfile for a new metaport, qt5-mac-mysql-plugins, which provides the mutually conflicting subports for Qt 5.

This is an adapted copy of qt4-mac-mysql55-plugin which also provides the other mysql plugin variants, for Qt 4 . Contrary to that portfile, I've opted *not* to use one of the variants as the main port, but rather a stub main (meta) port, an approach I find a bit cleaner.

I'm also trying a new (for me) way of defining the port version. Since it depends on the installed (and even the active) Qt port, I obtain the active version through the qmake executable that's an inevitable part of every Qt install. This should remove the need to bump the port version every time Qt is updated, and should even allow users to reinstall (using e.g. port -n upgrade --force) the installed subport after activating a different Qt 5 version (regardless of whether that's qt5-mac or qt5-mac-devel). I hope I haven't overlooked potential issues with this approach.

Attachments (1)

Portfile (3.5 KB) - added by RJVB (René Bertin) 10 years ago.

Download all attachments as: .zip

Change History (15)

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

Replying to rjvbertin@…:

I'm also trying a new (for me) way of defining the port version. Since it depends on the installed (and even the active) Qt port, I obtain the active version through the qmake executable that's an inevitable part of every Qt install.

This won’t work. This portfile will trip up portindex on all systems that do not have qmake installed.

Creating port index in /Users/larryv/Projects/MacPorts/git-svn/trunk/dports
Failed to parse file aqua/qt5-mac-mysql-plugins/Portfile: couldn't execute "/opt/local/bin/qmake": no such file or directory

comment:2 Changed 10 years ago by RJVB (René Bertin)

Doh, that's the only use case I didn't foresee :)

What about something like

if {[file exists ${qt_qmake_cmd}]} {
    version    [exec ${qt_qmake_cmd} -query QT_VERSION]
} else {
    version    5.4.1
}

Kinda removes part of the charm of my clever lil idea, we'll still have to bump that fallback version each time Qt is updated, but the other advantages should remain intact for users with Qt 5 installed...

Last edited 10 years ago by RJVB (René Bertin) (previous) (diff)

comment:3 Changed 10 years ago by mkae (Marko Käning)

Cc: mk@… added

Cc Me!

Changed 10 years ago by RJVB (René Bertin)

Attachment: Portfile added

comment:4 Changed 10 years ago by RJVB (René Bertin)

This version supports Qt 5.3.2 on OS X 10.6.8 .

comment:5 Changed 10 years ago by RJVB (René Bertin)

https://github.com/RJVB/mp-port-repository/tree/master/aqua/qt5-mac-mysql-plugins

This version improves the update from 2 days ago: the source is now treated as if it were "external" code (= not part of Qt). Notably, the installed mkspecs files are used instead of the bundled ones (rather important on OS X 10.6.8)

comment:6 Changed 10 years ago by ctreleaven (Craig Treleaven)

Cc: ctreleaven@… added

Cc Me!

comment:7 Changed 8 years ago by mkae (Marko Käning)

Cc: mkae removed
Owner: changed from macports-tickets@… to mkae
Status: newassigned

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

What's the status of this, at the moment, René?

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

Cc: ctreleaven removed

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

Owner: mkae deleted

comment:11 Changed 6 years ago by chrstphrchvz (Christopher Chavez)

I think this has since been superseded by qt5-mysql-plugin. Can this be closed?

Last edited 6 years ago by chrstphrchvz (Christopher Chavez) (previous) (diff)

comment:12 Changed 6 years ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added

comment:13 Changed 6 years ago by RJVB (René Bertin)

Largely...

comment:14 Changed 6 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.