ThKattanek / emu64

C64 Emulator
https://www.thorsten-kattanek.de/index.php/projekte/emu64
GNU General Public License v2.0
37 stars 5 forks source link

several cross-platform and packaging fixes #187

Closed Zirias closed 4 years ago

Zirias commented 4 years ago
Zirias commented 4 years ago

Just noticed the "lrelease" option doesn't work yet as intended, I'll have a look at that and try to fix it...

ThKattanek commented 4 years ago

thaks, for the nice job. welcome to the emu64 contributors. ok, if you fixed that, I'll take the pull request. my english is so bad, google help me. ;)

Zirias commented 4 years ago

Na, ich kann auch deutsch schreiben :) bin es bei opensource eben gewohnt dass alles englisch ist, wegen potentiell internationaler Entwicklergruppe.

Ich schau dann so bald wie möglich mal ob ich das mit lrelease in den Griff kriege und aktualisiere dann den PR. Dann könnte es vllt recht bald n offiziellen FreeBSD Port geben.

ThKattanek commented 4 years ago

Momentan ist die Anzahl der Entwickler sehr überschaubar. ;) Deutsch wäre mir momentan lieber, englisch lesen + verstehen kann ich recht gut aber mich ausdrücken ist ein Krampf ;). Super wenn du das gefixt hast übernehme ich das gerne im Masterbranch. Ich hatte da auch so meine Probleme mit lrelease automatisiert in der .pro Datei. Hatte dann die jetzige Lösung erst mal für mich gefunden. Viel Erfolg, ein offizieller FreeBSD Port ist natürlich Super.

Zirias commented 4 years ago

So, die "lrelease-Nuss" ist geknackt, leider mit einem hässlichen Workaround für einen Qt-Bug: https://bugreports.qt.io/browse/QTBUG-77398

Dazu noch zwei Kleinigkeiten, siehe commit message. Ich hoffe das passt jetzt so, in dem Zustand wäre das Projekt schön einfach zu paketieren :)

ThKattanek commented 4 years ago

Das hört sich gut an. Ich habe das jetzt schnell mal hier bei mir getestet. 2 Sachen die mir aufgefallen sind. PREFIX wird gesetzt wenn nicht Win32. Soweit so gut. Ich hatte das auskommentiert da ich den PREFIX per qmake setze (bsp. qmake .. PREFIX="/usr/local") so das ich das Install Verzeichnis mir beliebig aussuchen kann. Ich fand keine Lösung wo ich PREFIX mit /usr/local in der pro setze, wenn es nicht per qmake übergeben wurde. Evtl möchte ja jemand nicht nach /usr/local installieren, das würde jetzt unter linux nicht funktionieren. Das wäre schön wenn du da auch noch eine Lösung hast.

Merkwürdigerweise funktioniert make install bei nicht mehr mit den languages. Das Verzeichnis wird erstellt aber keine Dateien werden hinein kopiert, auch die Grafiken nicht. Werde mir das morgen noch mal anschauen, leider heute wenig Zeit gehabt.

Zirias commented 4 years ago

Punkt 1: das ist so peinlich, dass ich den commit per force push korrigiert habe. Zu meiner Verteidigung, ich nutze normalerweise kein qmake :) Selbstverständlich muss $prefix immer irgendwie von außen angegeben werden können, da gibt es gar keine Diskussion. Allein schon wegen Paketierung für Linux (da ist prefix normalerweise /usr)... Ist jetzt gefixt!

Punkt 2: Das ist seltsam, "works for me" ;) Aber dem kann man sicher auch noch auf den Grund gehen. Plan B wäre jederzeit, auf das integrierte lrelease zu verzichten, wie schon gesagt. Ich denke aber immer: "muss gehen!" :)

Zirias commented 4 years ago

Gibt's noch Probleme mit den Änderungen? Wie gesagt, das Installationsproblem kann ich hier leider nicht nachvollziehen...

ThKattanek commented 4 years ago

Sorry, war erkältet und auf Arbeit waren Überstunden angesagt. Werde das jetzt übernehmen und dann bei mir schauen was ich noch anpassen muss.