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 #142

Merged
merged 1 commit into from
Nov 17, 2015

Conversation

muuki88
Copy link
Contributor

@muuki88 muuki88 commented Dec 16, 2014

Merry Christmas 🎄

Seems like the 0644 chmod before the copy destroys the copy attempt of sbt.
However using the jdeb implementation works just fine (seem so).

@adriaanm adriaanm self-assigned this Dec 23, 2014
@adriaanm adriaanm added this to the 2.11.6 milestone Dec 23, 2014
@adriaanm
Copy link
Contributor

Thanks again, @muuki88, and happy holidays! We'll use this to roll 2.11.6 and 2.12.0-M1. I unfortunately ran out of time for 2.11.5 -- our Jenkins caught fire, so I've been swamped with that.

@adriaanm adriaanm force-pushed the 2.11.x branch 3 times, most recently from b8f2d81 to 2035c46 Compare December 24, 2014 18:05
@adriaanm
Copy link
Contributor

adriaanm commented Jan 2, 2015

Testing on our new release CI: http://scala-ci.typesafe.com/job/scala-release-2.11.x/8/console

@muuki88
Copy link
Contributor Author

muuki88 commented Jan 2, 2015

Ahh... windows needs some love as well

windows:packageBin from windows:packageBin (C:\jenkins\workspace\scala-release-2.11.x-windows\build.sbt:24

http://scala-ci.typesafe.com/job/scala-release-2.11.x-windows/5/console

@SethTisue
Copy link
Member

@adriaanm does this need some attention?

@adriaanm
Copy link
Contributor

It would be good to upgrade if/when possible.

@muuki88
Copy link
Contributor Author

muuki88 commented Jul 27, 2015

I can upgrade the versions ( unfortunately in two weeks ) if this helps.

@SethTisue
Copy link
Member

@muuki88 ping! still interested?

@SethTisue
Copy link
Member

@muuki88 does this result in any user-visible changes that we should mention in the release notes?

@muuki88
Copy link
Contributor Author

muuki88 commented Nov 17, 2015

Nothing I can think of. Startscript, permissions stay the same.

@SethTisue SethTisue changed the title FIX #141 Upgrading to native-packager 1.0.0-M3 and autoplugins FIX #141 Upgrading to native-packager 1.0.4 and autoplugins Nov 17, 2015
SethTisue added a commit that referenced this pull request Nov 17, 2015
FIX #141 Upgrading to native-packager 1.0.4 and autoplugins
@SethTisue SethTisue merged commit 6b3dd49 into scala:2.11.x Nov 17, 2015
@SethTisue
Copy link
Member

Thanks Muuki!

@SethTisue
Copy link
Member

oh, I shouldn't have merged this, it breaks on Windows: https://scala-ci.typesafe.com/job/scala-2.11.x-release-package-windows/295/console . I should have inquired more closely about Muuki's earlier comment about it.

@muuki88
Copy link
Contributor Author

muuki88 commented Nov 19, 2015

I'll try to have a look this or next week. Have to setup a windows machine :D

@adriaanm
Copy link
Contributor

In the mean time, could we revert to get the nightlies back to green?

@adriaanm
Copy link
Contributor

Thanks for looking into this, @muuki88!

SethTisue added a commit that referenced this pull request Nov 19, 2015
This reverts commit 1da1582
because it doesn't work on Windows; see
#142 for discussion.
@SethTisue
Copy link
Member

reverted in 07c4270

@SethTisue SethTisue removed their assignment Nov 19, 2015
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.

3 participants