Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

close #27 appimage #30

Merged
merged 47 commits into from
Nov 6, 2019
Merged

close #27 appimage #30

merged 47 commits into from
Nov 6, 2019

Conversation

Huluvu424242
Copy link
Owner

@Huluvu424242 Huluvu424242 commented Nov 6, 2019

PR zur Erstellung eines AppImages für Linux Systeme, siehe issue#27

Der Feature Branch wurde vom feature issue#18 abgebrancht (Was nicht üblich ist. Es wurde sich aber dafür entschieden, da gedacht wurde, dass einige Anpassungen aus der #18 benötigt werden. Die Abhängigkeit war beim Storyschnitt noch nicht erkennbar und ist möglicherweise später auch nicht mehr gegeben. Die Entwicklung auf #18 wird demnächst wieder aufgenommen evtl. aber gestaltet sich die Lösung viel einfacher als zunächst gedacht - mal sehen ob das Problem der Automatic Module umgangen werden kann. Einige Abhängigkeiten lassen sich einfach nicht als Modul bekommen).

Durch den Merge sind nun bereits einige #18 er Funktionalitäten in den master gewandert :(

Ich versuche selbst zu kommentieren wenn ich #18 er Anpassungen finde, die evtl. obsolet werden.

FunThomas424242 (Thomas Schubert) added 30 commits October 27, 2019 16:44
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
…xed)

Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>

Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
FunThomas424242 (Thomas Schubert) added 17 commits November 6, 2019 18:00
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
Signed-off-by: FunThomas424242 (Thomas Schubert) <funthomas424242@gmail.com>
@Huluvu424242 Huluvu424242 merged commit 92babae into master Nov 6, 2019
Copy link
Owner Author

@Huluvu424242 Huluvu424242 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die Anpassungen für #18 werde ich wohl drinne lassen müssen bis #18 umgesetzt ist. Das ist recht blöd.
Der Review wäre besser vor dem Merge passiert ;) so wie eigentlich üblich :(

@@ -2,9 +2,13 @@ language: java


jdk:
- openjdk11
- openjdk13
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mein lokaler Rechner lies sich nach einem Update einfach nicht mehr dazu herab mir ein volles jdk11 zur Verfügung zu stellen. Seither habe ich nur ein jdk13 und von 8 bis 11 jre's.

@@ -0,0 +1,4 @@
#!/bin/bash
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das ist eher für #18 gedacht

@@ -0,0 +1,20 @@
#!/bin/bash
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das war für #18 gedacht. Ist jetzt eher obsolet. Ausgelagert in ein extra Projekt:
https://github.com/PIUGroup/informally-jmod-hub

@@ -7,7 +7,7 @@
<groupId>com.github.funthomas424242</groupId>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vorbereitung für #18

@@ -454,12 +469,76 @@
</build>

<profiles>
<profile>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das wäre ein schönes Feature gewesen um zu prüfen, dass der Build mit dem lokalen JDK übereinstimmt. Es tut aber nicht :(

@@ -0,0 +1,11 @@
```shell
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obsoletes #18

@@ -0,0 +1,21 @@
#!/bin/bash
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obsoletes #18

@@ -0,0 +1,21 @@
#!/bin/bash
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obsoletes #18

@@ -0,0 +1,37 @@
/*-
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das wäre brauchbar als Vorbereitung für #18. Da muss man aber sicher wissen was man tut und es sollte vollständig und korrekt sein. -> Kann ich aktuell nicht sicherstellen.

@Huluvu424242 Huluvu424242 self-assigned this Nov 13, 2019
@Huluvu424242 Huluvu424242 added the enhancement New feature or request label Nov 13, 2019
@Huluvu424242 Huluvu424242 added this to the 0.0.2 milestone Nov 13, 2019
@Huluvu424242 Huluvu424242 linked an issue Mar 4, 2020 that may be closed by this pull request
@Huluvu424242 Huluvu424242 modified the milestones: 0.0.2, 0.2.0 Apr 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Erstellung AppImage Version für Linux Systeme
1 participant