planetsplitter: Cannot open file

Poste doch mal die gesamte Ausgabe von QMapShack. Also Befehle und Fehlermeldungen.

Tatsächlich. Der erste Teil fehlt komplett.

Wenn ich mir den Code ansehe kann das nur passieren wenn `sourceFiles` leer bleibt. Gefüllt wird diese Liste beim Start

void CRoutinoDatabaseBuilder::slotStart()
{
    pushStart->setDisabled(true);

    sourceFiles.clear();
    for(const QListWidgetItem * item : listWidget->findItems("*", Qt::MatchWildcard))
    {
        sourceFiles << item->text();
    }

Das über `findItems` zu machen ist vielleicht nicht der beste Weg. Gut möglich das deine Qt Version etwas anderes als erwartete zurückgibt. Ich würde das heute eher so schreiben:

    const int N = listWidget->count();
    for(int n = 0; n < N; n++)
    {
        sourceFiles << listWidget->item(n)->text();
    }

Ist deine Qt Version zufällig 5.12? Die scheint recht buggy zu sein.

Qt hat in der letzten Zeit öfters mit seiner API Stabilität gebrochen. Also gut möglich das wir den Code ändern müssen.

Die Liste ist in der Regel nicht leer, weil QMapShack ja überprüft, ob alle nötigen Vorgaben da sind. Und wenn wir anfangen Fehlermeldungen für interne Bibliotheksprobleme zu implementieren wüsste ich nicht wo man da aufhören sollte.

Kannst Du mal den for-loop austauschen und testen ob es daran liegt? Wenn ja bitte ein Ticket aufmachen.

Danke schon mal für den PR. Ich würde Dich aber bitten den üblichen Prozess zu folgen. Ja ich weiß ist bei so kleinen Sachen nervig, aber erstens gibt es sonst ein schlechtes Beispiel und zweitens fehlt es dann an Dokumentation, so dass andere auch verstehen worum es ging.

Also:

1. Ticket erstellen. Bitte alles ausfüllen. Erklärende Texte ersetzen

2. Branch erstellen mit der Ticketnummer. Also QMS-XXX

3. Commits sollten auch immer die Ticketnummer erwähnen. Einfach mal mit `git log` ansehen wie es gemacht wurde.

4. Der PR sollte auch die Ticketnummer im Titel erwähnen und natürlich im Text das Ticket referenzieren. Man findet so alles viel besser.

Alles Kleinigkeiten die eigentlich nicht viel Zeit kosten, dafür aber helfen den Überblick zu behalten

Zum PR selber:

1. Der ist natürlich schon `user facing`. Und braucht einen Eintrag in die changelog.txt. Bitte ordentlich einsortieren.

2. Die fehlenden includes dürfen gerne mit. Ich dachte eigentlich wir hätten die schon alle.

3. Es gibt noch eine zweite Stelle im Code wo der Loop ähnlich gemacht wird. Der VRT Builder. Kannst Du den bitte auch ändern.

Ach ja und auch beim PR keinen Abschnitt löschen. Vor allem nicht so einen wichtigen wie den `smoke test` Gehe mal davon aus dass noch mehr Leute die PRs beobachten und ausprobieren. Die haben keine Lust Rätsel zu raten. Machen aber eine sehr wichtige Arbeit indem sie den Code auf anderen Systemen testen.

Nimm dir am Besten ein Beispiel an dem anderen PR der gerade offen ist. Da ist das Ticket sehr gut geschrieben. Und natürlich habe ich auch beim PR die nötige Sorgfalt walten lassen. Die Änderung selber ist ein ähnlicher Pipidax wie die hier

Gerade gemerged. Vielen Dank für die Arbeit