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

Jameica und Hibiscus Abhängigkeit im build definieren und bauen #677

Merged
merged 27 commits into from
Feb 23, 2025

Conversation

tobidope
Copy link
Member

Da der Umbau auf Maven wohl doch ein zu großer erster Schritt ist, habe ich den ant build erweitert. Mit dem target build-dependencies werden jameica und hibiscus heruntergeladen und gebaut, so wie es der jverein build erwartet. Dies funktioniert lokal wie im Github Action Kontext identisch. Somit kann das Aufsetzen des lokalen builds vereinfacht werden und man baut gegen gleiche Versionen wie in Github. In einem zweiten Schritt kann renovate aktiviert werden, so dass neue Versionen erkannt und ein pr erstellt wird.
Die workflows sind auch wesentlich vereinfacht, da der cache nun abhängig von build.properties und build.xml ist. Solange sich dort nichts verändert, haben sich auch keine externen Abhängigkeiten verändert. Es werden nur die Elemente gecached, die auch im ant build im classpath referenziert werden.

@tobidope tobidope requested review from tolot27 and dippeal February 16, 2025 15:20
@tolot27 tolot27 added the patch label Feb 18, 2025
Copy link
Member

@tolot27 tolot27 left a comment

Choose a reason for hiding this comment

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

Step "restore-cache" nach reusable-build.yml auslagern.

@tobidope tobidope force-pushed the feature/antGetsDependencies branch from ae7e329 to 3b54e0d Compare February 18, 2025 23:15
@tobidope tobidope requested a review from tolot27 February 18, 2025 23:19
@tobidope tobidope self-assigned this Feb 18, 2025
@tolot27
Copy link
Member

tolot27 commented Feb 19, 2025

Es ist auf jeden Fall schon mal cool, dass die Cache Usage mit diesem PR mehr als halbiert wird (siehe https://github.com/openjverein/jverein/actions/caches). Dadurch laufen die Actions auch schneller.

@tobidope tobidope requested a review from tolot27 February 20, 2025 18:09
Copy link
Member

@tolot27 tolot27 left a comment

Choose a reason for hiding this comment

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

Sehr schön aufgeräumt sieht es nun aus. Vielen Dank und entschuldige, dass ich da bisl nervig war.

tolot27
tolot27 previously approved these changes Feb 21, 2025
@tobidope tobidope requested a review from lenilsas February 21, 2025 11:26
@lenilsas
Copy link

Ich kenne mich bisher mit den Actions nur wenig aus und überblicke daher nicht alle folgen. Wenn sich sonst niemand findet kann ich es schon aprooven. @dippeal ich glaube du kennst dich besser damit aus und ich fände gut wenn du es reviewen könntest.

@tobidope
Copy link
Member Author

Die Version von Jameica war noch alt. Die habe ich jetzt auf die aktuellste angepasst.

tolot27
tolot27 previously approved these changes Feb 21, 2025
@dippeal
Copy link
Member

dippeal commented Feb 23, 2025

Kannst du mal bitte skizzieren oder erklären, wo der renovate ins Spiel kommt bei dem PR?
Irgendwie sehe ich noch nicht wo der Cache befüllt wird. In https://github.com/openjverein/jverein/pull/677/files#diff-e5e50a1d786c73146f839e2f5406dc6b8442431260d86f9098448a8c2a96af7cR13-R24 holst du die Daten, aber wo werden sie aktualisiert wenn der hash sich geändert hat?
Ansonsten ganz schön.

@tobidope
Copy link
Member Author

Kannst du mal bitte skizzieren oder erklären, wo der renovate ins Spiel kommt bei dem PR? Irgendwie sehe ich noch nicht wo der Cache befüllt wird. In https://github.com/openjverein/jverein/pull/677/files#diff-e5e50a1d786c73146f839e2f5406dc6b8442431260d86f9098448a8c2a96af7cR13-R24 holst du die Daten, aber wo werden sie aktualisiert wenn der hash sich geändert hat? Ansonsten ganz schön.

Mit diesem pr existiert eine Versionsnummer, die von renovate verwendet werden kann. Bis jetzt nehmen wir einfach was gerade aktuell ist. Renovate muss noch kommen, siehe #699 . Der cache wird durch die cache action befüllt. Die schaut, ob sie einen miss hatte und aktualisiert im Nachgang den cache mit dem cache key. Der key generiert sich aus den build.properties und build.xml Dateien. Wenn diese sich geändert haben, ist nicht mehr sicher, dass wir die gleichen Abhängigkeiten haben.

@tobidope
Copy link
Member Author

@dippeal und @tolot27 könntet ihr euer Review abgeben? Aus meiner Sicht ist nichts mehr offen.

@tolot27 tolot27 merged commit 2bf4c15 into openjverein:master Feb 23, 2025
1 check passed
@tobidope tobidope deleted the feature/antGetsDependencies branch February 23, 2025 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants