Dungeon-CampusMinden / Dungeon

The "Dungeon" is a tool to gamify classroom content and integrate it into a 2D Rogue-Like role-playing game.
MIT License
15 stars 36 forks source link

Werden die jarConverter-Shellskripte/ bat-Files noch gebraucht? #1500

Open tgrothe opened 4 months ago

tgrothe commented 4 months ago

diese seltsamen jarConverter-shellskripte/bat-files sollten wir auch mal entfernen (nicht hier!), das ist overkill.

edit: ich sehe grad, die dienen gar nicht zum erzeugen des starter.jar, sondern zum zusammenpacken der dsl-skripte. da entstehen bei mir zwei fragen: (a) brauchen wir das überhaupt (noch)? (@AMatutat?) und (b) kann man das nicht über ein gradle-target lösen? (@tgrothe bitte mach mal ein neues ticket für diese geschichte auf. danke.)

Originally posted by @cagix in https://github.com/Dungeon-CampusMinden/Dungeon/issues/1491#issuecomment-2076498547

tgrothe commented 4 months ago

@AMatutat Es wäre schön, wenn du etwas dazu schreibst, ich kenne die jarConverter gar nicht...

AMatutat commented 4 months ago

https://github.com/Dungeon-CampusMinden/Dungeon/blob/master/dungeon/doc/misc/jar_converter.md

Ist halt zum einfachen Zusammepacken von DSL-Files. Ein Non-Programmer kennt Gradle nicht.

tgrothe commented 4 months ago

https://github.com/Dungeon-CampusMinden/Dungeon/blob/master/dungeon/doc/misc/jar_converter.md

Ist halt zum einfachen Zusammepacken von DSL-Files. Ein Non-Programmer kennt Gradle nicht.

Ach so, ok, übersehen, dass es dazu auch eine Dokumentation gibt. :)

AMatutat commented 4 months ago

Im Kern ist das noch viel zu kompliziert, ne richtige GUI wäre schöner, aber best we have

tgrothe commented 4 months ago

Idee: Anstatt das Script manuell anzupassen, könnte man auch eine zusätzliche gradle settings/properties Datei manuell anpassen.

cagix commented 4 months ago

https://github.com/Dungeon-CampusMinden/Dungeon/blob/master/dungeon/doc/misc/jar_converter.md

Ist halt zum einfachen Zusammepacken von DSL-Files. Ein Non-Programmer kennt Gradle nicht.

@AMatutat Die Frage, die für mich dahinter steht: Brauchen wir das (noch)? Wird es diesen Use-Case (noch) geben? Wir haben aktuell die Möglichkeit, beim Start des Dungeons ein DSL-File auszuwählen. Das könnte man ohne Probleme nochmal ergänzen, so dass man auch mehrere DSL-Files auswählen kann. Damit brauchen wir aus meiner Sicht wohl nicht mehr die Möglichkeit, DSL-Files zusammenzupacken in ein JAR.

Im Kern ist das noch viel zu kompliziert, ne richtige GUI wäre schöner, aber best we have

Könnte man ja mal bauen: Einen einfachen Auswahldialog mit ein paar Buttons für Generieren und Hinzufügen/Entfernen.

Idee: Anstatt das Script manuell anzupassen, könnte man auch eine zusätzliche gradle settings/properties Datei manuell anpassen.

@tgrothe Das ist ja gar nicht das Problem hier bzw. würde auch nicht helfen, weil die Notwendigkeit für Änderungen einfach nur von A nach B verschoben wird.

AMatutat commented 4 months ago

@AMatutat Die Frage, die für mich dahinter steht: Brauchen wir das (noch)? Wird es diesen Use-Case (noch) geben? Wir haben aktuell die Möglichkeit, beim Start des Dungeons ein DSL-File auszuwählen. Das könnte man ohne Probleme nochmal ergänzen, so dass man auch mehrere DSL-Files auswählen kann. Damit brauchen wir aus meiner Sicht wohl nicht mehr die Möglichkeit, DSL-Files zusammenzupacken in ein JAR.

Naja, du eine Jar ist angenehmer Verteilt als mehrere DSL-Files.

Aber praktisch gesprochen: Aktuell muss eine DSL-File eh immer vollständig sein und Inhalte können nicht über mehrere DSL-Files verteilt werden => Man hat eh immer nur ein DSL-File pro Szenario (oder wie du es nenne willst).

Frag @bcg7 ob sie das benutzt, sie ist aktuell unsere einzige Kundin.

cagix commented 4 months ago

@AMatutat Die Frage, die für mich dahinter steht: Brauchen wir das (noch)? Wird es diesen Use-Case (noch) geben? Wir haben aktuell die Möglichkeit, beim Start des Dungeons ein DSL-File auszuwählen. Das könnte man ohne Probleme nochmal ergänzen, so dass man auch mehrere DSL-Files auswählen kann. Damit brauchen wir aus meiner Sicht wohl nicht mehr die Möglichkeit, DSL-Files zusammenzupacken in ein JAR.

Naja, du eine Jar ist angenehmer Verteilt als mehrere DSL-Files.

Aber praktisch gesprochen: Aktuell muss eine DSL-File eh immer vollständig sein und Inhalte können nicht über mehrere DSL-Files verteilt werden => Man hat eh immer nur ein DSL-File pro Szenario (oder wie du es nenne willst).

Frag @bcg7 ob sie das benutzt, sie ist aktuell unsere einzige Kundin.

@AMatutat D.h. verstehe ich das korrekt, dass Dungeon aktuell in der Lage wäre, statt einer oder mehrerer .dng-Datei(en) auch eine .jar-Datei zu "fressen"?

Wenn ich an Lehrende/Kunden wie @bcg7 denke: Das Nutzen einer JAR mit verschiedenen DNG-Dateien drin bedeutet einen zusätzlichen Arbeitsschritt, insbesondere muss hierzu noch extra eine Skriptgeschichte verstanden und angepasst und ausgeführt werden. Am Ende bin ich mir nicht sicher, ob das wirklich (noch[^1]) einen Mehrwert hat.

Vielleicht sollte man das Bauen der JAR dann in ein kleines GUI-basiertes Mini-Tool auslagern und/oder mit einem Plugin für VSC/Idea verbinden, welches dann nicht nur Syntax-Highlighting und Autocompletion macht (vgl. https://github.com/Dungeon-CampusMinden/Dungeon/discussions/1395), sondern auch den Export eines JARs für die DNGs anbietet.

Die aktuelle Skriptlösung wird vermutlich niemand außerhalb der Informatik anfassen wollen, und selbst @bcg7 wird dazu vermutlich keine Muße haben (und auch keinen Mehrwert sehen). Aber ich frage sie mal.

[^1]: Früher hatten wir mit dem Packen der DNG-Files in ein JAR ja noch andere Ziele verfolgt, beispielsweise Absicherung gegen Schummeln und so. Aber davon sind wir ja weit weg.

AMatutat commented 4 months ago

@AMatutat D.h. verstehe ich das korrekt, dass Dungeon aktuell in der Lage wäre, statt einer oder mehrerer .dng-Datei(en) auch eine .jar-Datei zu "fressen"?

Das Dungeon ja, ob der File-Explorer das kann, k.a.

tgrothe commented 4 months ago

Wenn mit "File-Explorer" der JFileChooser gemeint ist, dann ist das kein Problem.