Skip to content

Bump version from 0.1-SNAPSHOT to 0.1.1-SNAPSHOT #1888

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

Merged
merged 1 commit into from
Jan 8, 2017

Conversation

smarter
Copy link
Member

@smarter smarter commented Jan 8, 2017

This is useful for two reasons:

  • All published Scala versions are of the form a.b.c and some tooling
    expect that, like sbt CrossVersion API.
  • Using 0.1.1 instead of 0.1.0 means that we match the version number of
    dotty-sbt-bridge, this is simpler and means that in the future sbt
    could automatically choose the correct version of dotty-sbt-bridge so
    that the user does not need to specify scalaCompilerBridgeSource in
    his build.sbt

Note: it's awful that we have hardcoded paths to jars and that I had to
change them, but I won't fix that now.

Review by @felixmulder

This is useful for two reasons:
- All published Scala versions are of the form a.b.c and some tooling
  expect that, like sbt CrossVersion API.
- Using 0.1.1 instead of 0.1.0 means that we match the version number of
  dotty-sbt-bridge, this is simpler and means that in the future sbt
  could automatically choose the correct version of dotty-sbt-bridge so
  that the user does not need to specify scalaCompilerBridgeSource in
  his build.sbt

Note: it's awful that we have hardcoded paths to jars and that I had to
change them, but I won't fix that now.
@felixmulder
Copy link
Contributor

Do we have another option when it comes to the jars? I'd love to get rid of it. Martin doesn't like them either :)

Will have a closer look in the morning! Cheers,
Felix

@smarter
Copy link
Member Author

smarter commented Jan 8, 2017

Get them from sbt, it's the only sane thing to do, and what I plan to do on my sbt-based bootstrapping branch (I need to do it, since the jars are different depending on whether I'm running dotty-compiled-by-scala2 or dotty-bootstrapped)

@smarter
Copy link
Member Author

smarter commented Jan 8, 2017

Once this PR is in I'll make a PR to sbt to have automatic scalaCompilerBridgeSource support for dotty, hopefully this can make it into 0.13.14

"0.1.1-" + VersionUtil.commitDate + "-" + VersionUtil.gitHash + "-NIGHTLY"
else
"0.1.1-SNAPSHOT"
},
Copy link
Contributor

Choose a reason for hiding this comment

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

When publishing nightlies we run NIGHTLYBUILD=yes sbt ... and thus L419 is used as the value of version. What was the reason for removing this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Because it now uses the global version set above.

Copy link
Contributor

Choose a reason for hiding this comment

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

You mean baseVersion? If so, won't the resulting jars from snapshot publish drop SNAPSHOT from the artefact name? Also, perhaps more importantly - how do you publish nightlies with the hash and commit date?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Previously this was done differently in dotty-sbt-bridge because of its different versioning scheme, now we can use the same version setting as dotty for dotty-sbt-bridge

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah I see, it was previously duplicated. Cool, then LGTM!

@smarter smarter merged commit add9a03 into scala:master Jan 8, 2017
smarter added a commit to scala/scala3-example-project that referenced this pull request Jan 8, 2017
@allanrenucci allanrenucci deleted the bump-version-011 branch December 14, 2017 16:58
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.

2 participants