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

Update Kotlin to 1.5.30 #588

Merged
merged 5 commits into from
Nov 22, 2021

Conversation

Bencodes
Copy link
Collaborator

@Bencodes Bencodes commented Sep 18, 2021

Tracking the latest Kotlin 1.5.30 release and removing the deprecated plugin encoding configuration CLI usage.

@Bencodes Bencodes force-pushed the update-kotlin-to-1.5.30 branch 2 times, most recently from b453c3f to 2b8d609 Compare September 18, 2021 20:44
fun list(): List<String> = args.toList()

fun base64Encode(
fun flagRepeated(
Copy link
Collaborator

Choose a reason for hiding this comment

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

The undocumented format changed? Oh, surprise....

This will need to be pulled into the versioned repos. Lmk if you'd prefer to pass this off to me, as I haven't finished writing up how to make the versioned toolchains work.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure you can take over.

Have an ETA on when the versioned toolchains will be ready? People won't be able to update Kotlin and likely Compose until this is merged.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Bencodes any idea if this works for 1.6-M1? https://github.com/JetBrains/kotlin/releases/tag/v1.6.0-M1 If we can stay on top of the feature previews we can hopefully get 1.6 support out in a timely manner.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jeffzoch In theory yes but I haven't tried yet. We are waiting on some important fixes in 1.6.0 so i'll be looking at supporting that soon.

@nkoroste
Copy link
Collaborator

@Bencodes @restingbull What do you guys think the status of this PR? are we planning to shoot for 1.6 or get this merged in the meantime so that we can start using 1.5.30?

@Kernald
Copy link
Contributor

Kernald commented Oct 4, 2021

Having this before 1.6 would be quite nice given that Compose now depends on Kotlin 1.5.30.

@restingbull
Copy link
Collaborator

restingbull commented Oct 5, 2021 via email

Rename variable for clarity.

Signed-off-by: restingbull@cabaretmechanique.com <restingbull@cabaretmechanique.com>
@restingbull restingbull added this to the Candidate 1.5 milestone Nov 19, 2021
Signed-off-by: restingbull@cabaretmechanique.com <restingbull@cabaretmechanique.com>
@restingbull restingbull merged commit 0be47a2 into bazelbuild:master Nov 22, 2021
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.

Reduced classpath doesn't correctly manage transitive dependencies for kapt
5 participants