Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#50321 closed enhancement (fixed)

gpsbabel: install an .app, figure out Qt dependencies

Reported by: mojca (Mojca Miklavec) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: ryandesign (Ryan Carsten Schmidt), florian@…, nerdling (Jeremy Lavergne), mdholling@…, m.schwarz@…, f.macports@…, arsenm2@…, enpe@…, mp@…, MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Port: gpsbabel

Description

gpsbabel comes with a gui, see http://www.gpsbabel.org/screenshots.html

However there is something weird about our configuration. qmake pull in some undeclared dependencies on Qt5 ports, but on the other hand it doesn't really install anything (see also #47759) and it doesn't seem to build an app. There is an xcodeproject and a bunch of other stuff in the sources that I didn't try to fully explore yet. In Makefile.in there's a target mac-gui for example.

I believe that dependencies should be resolved and a gui should be provided, either as a variant or as a subport (maybe subport gpsbabel-gui would be better?).

Change History (5)

comment:1 Changed 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Cc: mcalhoun@… added

Cc Me!

comment:2 Changed 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Resolution: fixed
Status: newclosed

This should be fixed in r144799.

comment:3 Changed 8 years ago by mojca (Mojca Miklavec)

> port installed gpsbabel
The following ports are currently installed:
  gpsbabel @1.5.3_1+qt4 (active)
> port contents gpsbabel
Port gpsbabel contains:
  /opt/local/bin/gpsbabel
  /opt/local/share/doc/gpsbabel/AUTHORS
  /opt/local/share/doc/gpsbabel/COPYING
  /opt/local/share/doc/gpsbabel/README.contrib
  /opt/local/share/doc/gpsbabel/README.igc
  /opt/local/share/doc/gpsbabel/README.mapconverter
  /opt/local/share/doc/gpsbabel/README.xmapwpt
> otool -L /opt/local/bin/gpsbabel
/opt/local/bin/gpsbabel:
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 159.1.0)
	/opt/local/libexec/qt4/Library/Frameworks/QtCore.framework/Versions/4/QtCore (compatibility version 4.8.0, current version 4.8.7)
	/System/Library/Frameworks/IOKit.framework/Versions/A/IOKit (compatibility version 1.0.0, current version 275.0.0)
	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 635.21.0)
	/usr/lib/libstdc++.6.dylib (compatibility version 7.0.0, current version 52.0.0)
Last edited 8 years ago by mojca (Mojca Miklavec) (previous) (diff)

comment:4 Changed 8 years ago by mojca (Mojca Miklavec)

I get an app if I use +gui, but why do you need dependency on Qt4 or Qt5 when there is no app?

The following is also slightly debatable, but I'll try to figure out if I can come up with a better proposal:

if { ${os.major} < ${qt5_min_tested_version} } {
    default_variants-append +qt4
} ...

Qt5 is somewhat problematic on libstdc++ only, so maybe this if should check what the default stdlib is set to as well.

Last edited 8 years ago by mojca (Mojca Miklavec) (previous) (diff)

comment:5 in reply to:  4 Changed 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Replying to mojca@…:

I get an app if I use +gui, but why do you need dependency on Qt4 or Qt5 when there is no app?

As noted previously, the non-gui program uses it.
Why exactly, I could not say.
Qt does have other features (file IO, networking, etc.).

The following is also slightly debatable, but I'll try to figure out if I can come up with a better proposal:

if { ${os.major} < ${qt5_min_tested_version} } {
    default_variants-append +qt4
} ...

Qt5 is somewhat problematic on libstdc++ only, so maybe this if should check what the default stdlib is set to as well.

Whatever others decide is fine.
I do not use gpsbabel and was just trying to lend a hand on the Qt side.
If there is a better solution, that would be great.
I am hoping that Octave will eventually support Qt5.
If that day should ever come, whatever is decided here might prove to be a useful model.

Last edited 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) (previous) (diff)
Note: See TracTickets for help on using tickets.