| Summary: | sci-geosciences/gpxsee-13.16 - WARNING: Project ERROR: Unknown module(s) in QT: openglwidgets | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
| Component: | Current packages | Assignee: | Andrey Grozin <grozin> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | asturm, grozin, sci |
| Priority: | Normal | Keywords: | PullRequest |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| See Also: |
https://github.com/gentoo/gentoo/pull/39913 https://bugs.gentoo.org/show_bug.cgi?id=948004 |
||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
emerge-info.txt
emerge-history.txt environment etc.clang.tar.xz etc.portage.tar.xz qlist-info.txt.xz sci-geosciences:gpxsee-13.16:20240227-022606.log temp.tar.xz |
||
|
Description
Toralf Förster
2024-02-27 09:06:32 UTC
Created attachment 886089 [details]
emerge-info.txt
Created attachment 886090 [details]
emerge-history.txt
Created attachment 886091 [details]
environment
Created attachment 886092 [details]
etc.clang.tar.xz
Created attachment 886093 [details]
etc.portage.tar.xz
Created attachment 886094 [details]
qlist-info.txt.xz
Created attachment 886095 [details]
sci-geosciences:gpxsee-13.16:20240227-022606.log
Created attachment 886096 [details]
temp.tar.xz
*** Bug 946586 has been marked as a duplicate of this bug. *** Andrey, seriously please improve your package dependency game and git messages. Just look at the diff in my PR. (In reply to Andreas Sturmlechner from comment #10) > Just look at the diff in my PR. Why do you think qtserialport should be in DEPEND but not RDEPEND? I thought that gpxsee can get the realtime gps position from an external gps device (like garmin etrex) connected to the computer via a serial port. If so, a way to read/write data from/to a serial port is needed at runtime. I never used this functionality myself, so, I may be mistaken. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=501914d6f1c26befc8b3ce4888a0f0d60c76013a commit 501914d6f1c26befc8b3ce4888a0f0d60c76013a Author: Andreas Sturmlechner <asturm@gentoo.org> AuthorDate: 2025-01-01 11:47:39 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2025-01-01 16:47:45 +0000 sci-geosciences/gpxsee: Drop IUSE qt5 (make Qt6 non-optional), fix deps - Fix entirely missing dev-qt/qtbase:6 USEdeps - Add missing slot op on dev-qt/qtbase:6 (GuiPrivate dependency in .pro file) - Add missing BDEPEND=dev-qt/qttools:6[linguist] USEdep - Run eqmake in src_configure(), not src_compile() dev-qt/qtserialport:6 DEPEND appears to be upstream compensating for a dependency not having defined its link interface properly. Closes: https://bugs.gentoo.org/925596 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org> sci-geosciences/gpxsee/gpxsee-13.33-r1.ebuild | 51 +++++++++++++++++++++++++++ 1 file changed, 51 insertions(+) (In reply to Andrey Grozin from comment #11) > (In reply to Andreas Sturmlechner from comment #10) > > Just look at the diff in my PR. > Why do you think qtserialport should be in DEPEND but not RDEPEND? There is no linking nor including of headers of qtserialport in gpxsee sources. Here's the upstream commit introducing it: https://github.com/tumic0/GPXSee/commit/e4d42453a0c9090b7e1b055f30a594c0777ba5b7 > The serialport module is a dependency of the nmea positioning plugin. Without > being explicitly specified, the Qt deploy tools skip the nmea positioning plugin > when creating the application package. That nmea positioning plugin is provided by: > # equery u dev-qt/qtpositioning > [...] > - - nmea : Build plugin for parsing NMEA (National Marine Electronics Association) sentences *that plugin* has dependencies: > * dependency graph for dev-qt/qtpositioning-6.8.1[nmea] > `-- dev-qt/qtbase-6.8.1[network] > `-- dev-qt/qtserialport-6.8.1 Whatever they are describing in that commit is not related to real package dependencies, but a workaround for whatever deployment tool they have to use - if you ask me, completely bogus in the build system (dev-qt/qtserialport could also be removed from gpxsee.pro file and consequently from DEPEND after that). However, your ebuild currently depends on dev-qt/qtpositioning:6, not dev-qt/qtpositioning[nmea]. Maybe you want to find out if this would better be an unconditional RDEPEND USEdep (I don't think it does anything at build time, but I haven't checked) or optfeature (depends on graceful handling of lack thereof at runtime). |