ho-dev / HattrickOrganizer

Assistant for Hattrick online football manager
https://ho-dev.github.io/HattrickOrganizer/
GNU Lesser General Public License v3.0
195 stars 78 forks source link

[BUG] Building stable release of HO 8.1 failed #2116

Closed wsbrenk closed 3 weeks ago

wsbrenk commented 1 month ago

Describe the bug A clear and concise description of what the bug is.

Task :buildSrc:compileKotlin w: file:///home/runner/work/HattrickOrganizer/HattrickOrganizer/buildSrc/src/main/kotlin/poeditor/PullRunnable.kt:59:50 Unnecessary non-null assertion (!!) on a non-null receiver of type File? w: file:///home/runner/work/HattrickOrganizer/HattrickOrganizer/buildSrc/src/main/kotlin/poeditor/PullTask.kt:51:50 Type mismatch: inferred type is String? but Any was expected w: file:///home/runner/work/HattrickOrganizer/HattrickOrganizer/buildSrc/src/main/kotlin/poeditor/PullTask.kt:52:40 Type mismatch: inferred type is List? but (Mutable)Iterable<String?> was expected

Task :buildSrc:compileJava NO-SOURCE Task :buildSrc:compileGroovy NO-SOURCE Task :buildSrc:classes UP-TO-DATE

Task :buildSrc:jar :jar: No valid plugin descriptors were found in META-INF/gradle-plugins

FAILURE: Build failed with an exception.

wsbrenk commented 1 month ago

@tychobrailleur Again a poeditor issue!? what do you think about bringing the language translation files back to the HO repo?

wsbrenk commented 1 month ago

https://github.com/kordamp/markdown-gradle-plugin/issues/34 seems to be the reason of this failure

wsbrenk commented 1 month ago

@tychobrailleur it looks like kotlin supports md to html conversion. it might be the easiest solution to create again our own gradle plugin?

tychobrailleur commented 1 month ago

@tychobrailleur it looks like kotlin supports md to html conversion. it might be the easiest solution to create again our own gradle plugin?

Yeah, as a stop-gap measure, it sounds good. Let me look into this.

tychobrailleur commented 1 month ago

It seems @simonpoole already did the heavy lift for us; we just have to wait for the 0.2.5 release of the hard fork! ;-)

simonpoole commented 1 month ago

Just as a clarification: I hard forked the dependency causing the issue (remark-java) here https://github.com/simonpoole/remark-java, no heavy lifting required (I did update a couple of dependencies and moved the build to gradle). Both the docs and a couple of other things need clean up, but outside of that it works..

The other thing is that I use it in a version of the markdown gradle plugin that I forked a long time ago which has a couple of extra features but which is not synced with the original. Currently I would not recommend using my fork, as there are a number of security issues with the version of flexmark that I'm using (that are however not relevant for my use case) and likely it would probably be better to merge my changes back in to upstream now that it is showing life again.

tl;dr you should get the maintainer of the gradle plugin to use my fork of remark-java if you want quick resolution.

tychobrailleur commented 1 month ago

tl;dr you should get the maintainer of the gradle plugin to use my fork of remark-java if you want quick resolution.

Awesome, thanks for taking the time to clarify the state of your fork, really appreciate! Will see how to get this addressed upstream.

tychobrailleur commented 4 weeks ago

This job completed successfully. I'll cherry-pick the change on master.

tychobrailleur commented 4 weeks ago

This job completed successfully. I'll cherry-pick the change on master.

This is now done, and the job completed successfully.