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

Kotlin 1.4 #363

Merged
merged 4 commits into from
Nov 4, 2020
Merged

Kotlin 1.4 #363

merged 4 commits into from
Nov 4, 2020

Conversation

fkorotkov
Copy link
Contributor

@fkorotkov fkorotkov commented Aug 28, 2020

Just bumped all the version. Seems there were not much changes on the compiler side since 1.3.x beside adding new language features and removing deprecating stdlib methods. Should be pretty painless update.

Fixes #295

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@fkorotkov
Copy link
Contributor Author

@googlebot I signed it!

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@restingbull
Copy link
Collaborator

Brief pause on this -- I'm seeing some oddities where using 1.3 is getting 1.4 libraries without changing the default version.

@fkorotkov
Copy link
Contributor Author

@restingbull if oddities are hard to investigate, alternatively we can keep the 1.3 default and just allow 1.4 language version with a custom toolchain.

@rockwotj
Copy link
Contributor

@restingbull if oddities are hard to investigate, alternatively we can keep the 1.3 default and just allow 1.4 language version with a custom toolchain.

I'd really love to move forward with 1.4 - can we do this and document the 1.4 custom toolchain? I think a separate change could make it the default...

/cc @restingbull @cgruber @fkorotkov

@cgruber
Copy link
Collaborator

cgruber commented Oct 9, 2020

If I understand comments from @restingbull - this can't work as-is, as there are some issues with hand-crafted classpaths in the builder that need fixing. I believe he's going to take a crack at it shortly, but we've been on-deadline internally, and haven't had the time to take care of this.

We absolutely do need 1.4 support ourselves, and soon, so this will get done.

@keith keith mentioned this pull request Nov 3, 2020
@cgruber cgruber merged commit 730d9ef into bazelbuild:master Nov 4, 2020
@cgruber
Copy link
Collaborator

cgruber commented Nov 4, 2020

I've merged this, as despite us having some theoretical misgivings that we're missing something, we're just not moving forward fast enough. So this way the community is freed up to detect it.

@rockwotj
Copy link
Contributor

rockwotj commented Nov 4, 2020

Thanks @cgruber should we wait for a release or just point something to master?

@cgruber
Copy link
Collaborator

cgruber commented Nov 4, 2020

You can point to master if you want, but I plan to release in a day or two. There are one or two last little fixes i would like to get in together, but if they take more than tomorrow I'll roll a release on Friday. Nothing ever stops you from pointing to a specific commit though.

That said, please remember that if you point at master, and not from a release binary, you'll need to do the extra developer setup, so you can build all the tooling itself. The release package will have all pre-built binaries.

@rockwotj
Copy link
Contributor

@cgruber I finally got around to testing this with my company's codebase and 1.4 works beautifully. We're planning a release tomorrow using master.

Is there a reason https://github.com/bazelbuild/rules_kotlin/releases/tag/legacy-1.4.0-rc4 didn't include this PR? (see here for example)

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.

Validate rules against 1.4.x kotlinc.
5 participants