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

FIX #141 Upgrading to native-packager 1.0.4 and autoplugins #154

Merged
merged 1 commit into from
Dec 15, 2015

Conversation

muuki88
Copy link
Contributor

@muuki88 muuki88 commented Nov 22, 2015

Followup to #142

SethTisue added a commit that referenced this pull request Dec 15, 2015
FIX #141 Upgrading to native-packager 1.0.4 and autoplugins
@SethTisue SethTisue merged commit cf25344 into scala:2.11.x Dec 15, 2015
@SethTisue
Copy link
Member

and let's see how the next nightly does...

@SethTisue
Copy link
Member

log after the change (https://scala-ci.typesafe.com/job/scala-2.11.x-release-package-windows/323/console) looks similar to the log before (https://scala-ci.typesafe.com/job/scala-2.11.x-release-package-windows/322/console)

but at http://www.scala-lang.org/files/archive/nightly/2.11.x/ I'm not seeing any nightlies yet for 7559aed, the most recent is a0dabe3. I'll check back tomorrow

@SethTisue
Copy link
Member

no nightlies might just be because https://scala-ci.typesafe.com/job/scala-2.11.x-release-smoketest/263/ got wedged for some reason. I aborted it; let's see what happens tonight.

@SethTisue
Copy link
Member

tracking this on the scala-dev ticket now. seems likely this PR is responsible, but maybe it's a coincidence). I'll look into it.

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.

2 participants