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

upgrade to sbt-native-packager 1.0.6 and sbt 0.13.9 #156

Merged
merged 2 commits into from
Jan 15, 2016

Conversation

SethTisue
Copy link
Member

we recently upgraded to 1.0.4, but that caused a regression
so the change hasn't really been fully tested yet. I committed
a fix for the regression (#155), so as long as we're going to do
a new round of testing, let's test the latest latest

@SethTisue
Copy link
Member Author

let's not merge this until we see green builds at https://scala-ci.typesafe.com/job/scala-2.11.x-release-main/ again (after merging #155)

@SethTisue SethTisue self-assigned this Jan 12, 2016
we recently upgraded to 1.0.4, but that caused a regression so the
change never received full testing yet.  the regression has bene
fixed, so as long as we're going to do a new round of testing, let's
test the latest latest
@SethTisue SethTisue force-pushed the sbt-native-packager-1.0.6 branch from 8aabf95 to 71aba62 Compare January 13, 2016 22:11
just on general dogfooding principle. delicious gravy!
@SethTisue SethTisue changed the title upgrade to sbt-native-packager 1.0.6 upgrade to sbt-native-packager 1.0.6 and sbt 0.13.9 Jan 13, 2016
@SethTisue
Copy link
Member Author

the coast seems clear

SethTisue added a commit that referenced this pull request Jan 15, 2016
upgrade to sbt-native-packager 1.0.6 and sbt 0.13.9
@SethTisue SethTisue merged commit d40dd6f into scala:2.11.x Jan 15, 2016
@SethTisue SethTisue deleted the sbt-native-packager-1.0.6 branch January 15, 2016 19:17
SethTisue added a commit to SethTisue/scala-dist that referenced this pull request Mar 3, 2016
the upgrade turned out to be way too problematic.  even after multiple
PRs addressing regressions, new ones continue to turn up; see
scala/scala-dev#92 for details on the latest
regressions. for past history (including details on regressions), see
these PRs in this repo: scala#159, scala#157, scala#156, scala#155, scala#154, scala#142, plus issue

so what's next after this?

- we could maybe still consider upgrading for 2.11.9, but someone
  would need to thoroughly QA it on all platforms and assure us there
  are no regressions

- or we could restrict the upgrade to 2.12.x and hope for
  partially crowdsourced QA so that regressions would be caught
  during the milestone and release candidate phases.

I lean towards leaving 2.11.x frozen at 0.6.4, at least unless the
upgrade brings concrete benefits to end users (no one has listed
any, to my knowledge).  if this is mainly just dogfooding, then
2.12.x is a better context for that.
SethTisue added a commit to SethTisue/scala-dist that referenced this pull request Mar 3, 2016
this restores the 2.11.7 status quo for the 2.11.8 release.

the upgrade turned out to be way too problematic.  even after multiple
PRs addressing regressions, new ones continue to turn up; see
scala/scala-dev#92 for details on the latest
regressions. for past history (including details on regressions), see
these PRs in this repo: scala#159, scala#157, scala#156, scala#155, scala#154, scala#142, plus issue

so what's next after this?

- we could maybe still consider upgrading for 2.11.9, but someone
  would need to thoroughly QA it on all platforms and assure us there
  are no regressions

- or we could restrict the upgrade to 2.12.x and hope for
  partially crowdsourced QA so that regressions would be caught
  during the milestone and release candidate phases.

I lean towards leaving 2.11.x frozen at 0.6.4, at least unless the
upgrade brings concrete benefits to end users (no one has listed
any, to my knowledge).  if this is mainly just dogfooding, then
2.12.x is a better context for that.
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.

1 participant