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

danger-kotlin cannot compile on Xcode 15+ #286

Closed
wants to merge 2 commits into from

Conversation

mrbass21
Copy link

@mrbass21 mrbass21 commented Oct 16, 2024

danger-kotlin will not compile on native mac using Xcode 15+. This seems to be due to Kotlin. Kotlin seems to have resolved this in version 1.9.10.

I don't know Kotlin tooling that well, so please let me know if I've missed anything in this PR that needs to be addressed. I was able to run the gradlew commands in the make file and all of them build successfully.

Thanks!

@Vacxe
Copy link
Member

Vacxe commented Oct 16, 2024

@mrbass21 Hey mate, thanks for the contribution!

However, that does not make sense to upgrade kotlin to that version because the latest is higher (check on kotlin website), I'll be happy to merge this PR if you can update it.

@mrbass21
Copy link
Author

mrbass21 commented Oct 16, 2024

Just letting you know I'm working on this. I'm not an Android or Kotlin developer and the builds break when Kotlin is updated. I'm seeing if I can fix them.

Edit: I found that I could get to 2.0.10 without any build issues. Going to 2.0.20 causes the library to have compilation issues. I will keep working at it, but if 2.0.10 is acceptable to get compiles working under Xcode 15+, and someone else (who actually knows the tooling better) can update to the latest Kotlin, feel free to accept as is.

Thanks!

Edit 2: Was able to get it working with the latest version of Kotlin, but it required updating JDK to 17, and upgrading Gradle to 8, and changing the source for shadowjar to com.gradleup. If those changes push this out of scope, this can go in as is.

Getting the project to compile with Kotlin 2.0.20+ was entirely solved by @DavidBrunow, should this work be used.

Will check for feedback. Thank you!

@mrbass21 mrbass21 force-pushed the xcode15KotlinUpdate branch from 58f4b53 to 72eb681 Compare October 16, 2024 15:26
@snowe2010
Copy link

@mrbass21 I think you're a Kotlin dev now. You got through a major Gradle and JVM upgrade in one PR. 😄

@mrbass21
Copy link
Author

@mrbass21 I think you're a Kotlin dev now. You got through a major Gradle and JVM upgrade in one PR. 😄

Not qualified yet! This one only has a Kotlin bump. We'll have to delay the celebration 😝

@Vacxe
Copy link
Member

Vacxe commented Oct 18, 2024

Gradle and Kotlin versions has been updated #251 with pipeline fixing. Could you please check with the latest master branch please?

@mrbass21
Copy link
Author

Gradle and Kotlin versions has been updated #251 with pipeline fixing. Could you please check with the latest master branch please?

Was able to build from main with Xcode 16 and macOS 15.0.1.

@Vacxe Vacxe closed this Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants