compile problem #220

mrdudz · 2022-03-18T15:12:29Z

Ich habe grade mal das repo gecloned und versucht zu kompilieren. Als erstes stolpert es über:

In Datei, eingebunden von ././ui_cartridge_window.h:26,
                 von ../../src/cartridge_window.cpp:21:
../../src/widget_file_browse.h:29:18: schwerwiegender Fehler: quazip5/quazip.h: Datei oder Verzeichnis nicht gefunden
   29 |         #include "quazip5/quazip.h"
      |                  ^~~~~~~~~~~~~~~~~~
Kompilierung beendet.

offenbar heisst das paket hier auch anders, und zwar "QuaZip-Qt5". pkgconfig sagt dazu:

pkgconf --cflags quazip1-qt5
-I/usr/include/QuaZip-Qt5-1.2 -I/usr/include/QuaZip-Qt5-1.2/quazip -DQT_CORE_LIB -I/usr/include/qt5/QtCore -I/usr/include/qt5

edit: nachdem ich in src.pro die zeilen für den zip support auskommentiert habe kompiliert es durch:

equals(QT_MAJOR_VERSION, 5) {

#    DEFINES += ZIP_SUPPORT

    win32 {
        PKGCONFIG += quazip
    } else {
#        LIBS += -lquazip5
    }

    message("QT5 is active")
}

vllt wäre es sinnvoll das irgendwie optional zu machen, wenn das paket tatsächlich auf manchen systemen (hier gentoo) anders heisst

The text was updated successfully, but these errors were encountered:

Zirias · 2023-06-30T19:12:16Z

Oh, das ist interessant, siehe 785b176

Vielleicht sollte man generell auf PKGCONFIG gehen? @ThKattanek kannst du mal schauen, ob es funktioniert, wenn man quazip1-qt5 auch auf Linux als Paketnamen nutzt (statt direkt LIBS zu setzen)?

This might fix ThKattanek#220

Zirias · 2023-06-30T19:22:40Z

Ok, sollte man nicht, auf Ubuntu tut das nicht. Sorry for the noise ....

mrdudz · 2023-06-30T19:34:22Z

Äh natürlich - man muss es halt installieren. Jedes Linux kann pkgconfig. Und jedes ernst zu nehmende Buildsystem benutzt das :)

Zirias · 2023-06-30T19:36:56Z

Sorry, aber DAS ist nicht das Problem hier. Das Problem hier ist, dass zumindest das Ubuntu-Paket libquazip5-dev offenbar gar kein .pc-file installiert, zumindest keines mit dem namen quazip oder quazip1-qt5.

Zirias · 2023-06-30T19:39:38Z

Ergänzend: Auf FreeBSD gibt es quazip1-qt5 (siehe verlinkter commit, das ist als patch schon länger im FreeBSD port drin), und auf deinem Gentoo offenbar auch, aber auf Ubuntu (und vermutlich anderern Linux dists?) wohl leider nicht -- damit weiß ich gerade nicht, wie man das Problem je portable lösen sollte...

mrdudz · 2023-06-30T19:40:09Z

Das klingt allerdings sehr seltsam

Zirias · 2023-06-30T19:48:07Z

Ja! Leider ...

FreeBSD kann ich im qmake .pro file checken, weil es halt ein anderes OS ist (bin hier aktiv, weil ich den FreeBSD port von emu64 maintaine) ... aber Linux ist halt Linux und wenn verschiedene Distributionen unterschiedliche Dinge in ihren Paketen ausliefern wird es eklig.

Da wäre es wohl nötig, verschiedene Ansätze, Quazip zu finden, durchzuprobieren. Ich weiß nicht ob qmake das überhaupt kann, könnte sein, dass man dieses issue nur lösen kann wenn man auf ein anderes Build-System (z.B. cmake) wechselt -> sehr viel Aufwand :(

mrdudz · 2023-06-30T19:51:49Z

neeee. Da ist einfach das Paket kaputt, imho. Wäre nicht das erste mal :)

Zirias · 2023-06-30T19:54:16Z

"Kaputt" ist Ansichtssache ... Bis vor kurzem kam OpenSSL in FreeBSD base ohne .pc files. Und das funktionierte gut, weil kaum ein Consumer von OpenSSL je erwartet hat, es mit pkg-config konfigurieren zu können ;)

Es gibt nunmal keinen verbindlichen Standard für C/C++ library pakete. pkg-config ist ein "quasi-standard", es wäre schön, wenn es immer funktionieren würde.

mrdudz · 2023-06-30T19:58:12Z

"Kaputt" ist Ansichtssache

daher "imho" :) In dem Fall liegt es einfach daran, dass dieses Paket ansich schon halbwegs exotisch ist, denke ich. Das wird halt von quasi nix benutzt, da fällt das nicht auf.

Zirias · 2023-06-30T20:11:55Z

Hm, da ist wohl was dran, auf https://www.freshports.org/archivers/quazip/ finde ich genau 8 FreeBSD ports (inkl. emu64), die Quazip nutzen.

Nur ändert das halt leider auch nichts. Wenn sowohl der FreeBSD als auch der Gentoo port ein .pc file für quazip1-qt5 liefern dann vermute ich mal, dass das "richtig" ist (diese Ports sind ja immer so nahe wie möglich an "upstream"). Auf Ubuntu (und darauf läuft auch der neue CI build hier) gibt's das halt trotzdem nicht :(

Wie gesagt, die "übliche" Lösung in solchen Fällen ist ein Build-System, das verschiedene Möglichkeiten, die benötigte Lib zu finden, "durchprobiert". Ich weiß nicht, ob qmake sowas kann, ich fürchte nicht ....

mrdudz · 2023-06-30T20:16:47Z

Vllt einfach mal einen Bug melden? Ich kann mir nicht vorstellen, dass die das mit Absicht weg werfen, wenn es im upstream drin ist. Das wäre schon ziemlich gaga. Aber vllt ist es auch garnicht drin?

Zirias · 2023-07-01T05:29:47Z

Ja, wenn man es genau nimmt liegt der Fehler offenbar beim Ubuntu package.

Upstream (Template): https://github.com/stachenov/quazip/blob/master/quazip/quazip.pc.cmakein

Resultat auf FreeBSD:

$ cat /usr/local/libdata/pkgconfig/quazip1-qt5.pc
prefix=/usr/local
exec_prefix=${prefix}
libdir=${prefix}/lib
includedir=${prefix}/include

Name: QuaZip-Qt5
Description: Minizip wrapper library for Qt 5.x
Version: 1.4
Libs: -lquazip1-qt5
Cflags: -I${includedir}/QuaZip-Qt5-1.4 -I${includedir}/QuaZip-Qt5-1.4/quazip
Requires: zlib, Qt5Core

Ubuntu package-list: https://packages.ubuntu.com/focal/amd64/libquazip5-dev/filelist

Ok, Schrott aus alten Flippern. Korrekt wäre jetzt, auch auf Linux pkg-config quazip1-qt5 zu verwenden. Geht aber erstmal nicht, weil dann der CI Build kaputt ist ...

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias · 2023-07-01T08:38:06Z

So. "Schön" ist es nicht, aber ich denke ich hab es gefixt ;)

ThKattanek · 2023-07-01T10:04:13Z

Under Kubuntu funktionierts, leider will er jetzt nicht mehr unter MXE builden.

fatal error: quazip.h: No such file or directory

Aber das schaue ich mir gleich mal an.

Zirias · 2023-07-01T10:06:36Z

Ah interessant. dein MXE Tree scheint eine uralte Version von quazip zu haben, das pkg-config File hieß soweit ich das nachvollziehen konnte bis vor 4 Jahren mal quazip (und seither quazip1-qt5). Vielleicht kann man das in MXE einfach mal upgraden? Dann würde der komplette Sonderfall für Windows vermutlich wegfallen.

Zirias · 2023-07-01T10:11:43Z

Alternativ: das #ifdef __WIN32 wieder rein und NUR auf Windows eben <quazip/quazip.h> includen statt <quazip.h>. Upgraden wäre aber vermutlich sinnvoller, denn wenn es jetzt jemand mit einem aktuellen MXE, MSYS2 oder Cygwin bauen will geht es vermutlich auch schon schief ....

ThKattanek · 2023-07-01T10:37:28Z

MXE hatte ich vor dem Release komplett neu geclont und neu gebuildet. Und ich sehe schon du hast ja schon Lösungsansätze. Aber ich schaue mal in MXE warum das jetzt schief geht.

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias · 2023-07-01T10:41:14Z

Bitte nochmal testen ... das ist jetzt ein WEITERER workaround der hoffentlich mit dem alten quazip.pc auch funktioniert.

Zirias · 2023-07-01T10:42:35Z

Hm. Ne, jetzt isses kaputt.

ThKattanek · 2023-07-01T10:45:01Z

Auch gerade gemerkt, selber Fehler.

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

ThKattanek · 2023-07-01T10:54:33Z

Parallel schon mal bei mir mit MXE getestet, läuft jetzt wieder.

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias · 2023-07-01T10:56:27Z

Jetzt tut's. Keine OS-Checks mehr, stattdessen "durchprobieren". quazip1-qt5 sollte da sein, dann passt alles. Wenn stattdessen quazip da ist, wird das genommen, da müssen aber die include-paths manuell gesetzt werden. Letzter Fallback (gar kein .pc File gefunden): Raten. Naja.

ThKattanek · 2023-07-01T10:58:21Z

MXE verwendet quazip.pc

Zirias · 2023-07-01T10:59:07Z

Ja. Und das stammt aus alten Quazip-Versionen und ist teilweise "kaputt" (setzt nicht die richtigen include-paths). Aber der neue Workaround scheint damit jetzt klarzukommen.

ThKattanek · 2023-07-01T11:01:53Z

Ja sieht erstmal sehr gut. Muss ich mal beobachten ob sich da bei MXE was ändert, und die irgendwann auf die neue Version umsteigen. Aber so, denke ich mal, kann man gut weiter arbeiten.

Zirias · 2023-07-01T11:03:40Z

Dürfte jetzt eigentlich kein Problem mehr sein, weil die neue Version zuerst probiert wird ;)

ThKattanek · 2023-07-01T11:04:14Z

Sehr schön. Na dann übernehme ich mal dein PR.

Zirias added a commit to Zirias/emu64 that referenced this issue Jun 30, 2023

Fix quazip dependency on Linux

8948cd3

This might fix ThKattanek#220

Zirias added a commit to Zirias/emu64 that referenced this issue Jul 1, 2023

Quazip: Use pkg-config if available

4e29cde

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias mentioned this issue Jul 1, 2023

Quazip: Use pkg-config if available #248

Merged

Zirias added a commit to Zirias/emu64 that referenced this issue Jul 1, 2023

Quazip: Use pkg-config if available

aa1da14

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias added a commit to Zirias/emu64 that referenced this issue Jul 1, 2023

Quazip: Use pkg-config if available

6eda634

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias added a commit to Zirias/emu64 that referenced this issue Jul 1, 2023

Quazip: Use pkg-config if available

33c142b

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias added a commit to Zirias/emu64 that referenced this issue Jul 1, 2023

Quazip: Use pkg-config if available

3601b0c

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

Zirias added a commit to Zirias/emu64 that referenced this issue Jul 1, 2023

Quazip: Use pkg-config if available

cce549b

Otherwise, guess libs and include dirs (best effort). Fixes: ThKattanek#220

ThKattanek closed this as completed in #248 Jul 1, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

compile problem #220

compile problem #220

mrdudz commented Mar 18, 2022 •

edited

Loading

Zirias commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023 •

edited

Loading

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jul 1, 2023 •

edited

Loading

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

compile problem #220

compile problem #220

Comments

mrdudz commented Mar 18, 2022 • edited Loading

Zirias commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023 • edited Loading

Zirias commented Jun 30, 2023

mrdudz commented Jun 30, 2023

Zirias commented Jul 1, 2023 • edited Loading

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

Zirias commented Jul 1, 2023

ThKattanek commented Jul 1, 2023

mrdudz commented Mar 18, 2022 •

edited

Loading

mrdudz commented Jun 30, 2023 •

edited

Loading

Zirias commented Jul 1, 2023 •

edited

Loading