Skip to content

Conversation

@SethTisue
Copy link
Member

I have verified in advance that this fixes the bootstrap, which was
failing since we ditched scala-xml from this repo

@scala-jenkins scala-jenkins added this to the 2.13.0-M4 milestone Apr 26, 2018
I have verified in advance that this fixes the bootstrap, which was
failing since we ditched scala-xml from this repo
@SethTisue SethTisue force-pushed the use-xml-free-partest branch from 4352100 to 286c0ab Compare April 26, 2018 10:27
@SethTisue
Copy link
Member Author

SethTisue commented Apr 26, 2018

log of this succeeding with partest 1.2.0-M2, which should be identical to 1.2.0: https://travis-ci.org/scala/scala/jobs/371467201

regardless, I'll wait for PR validation to succeed before merging, just to be sure that I didn't botch the 1.2.0 publishing somehow

EDIT: I'll need to wait for the partest artifacts to reach Maven Central first...

@SethTisue
Copy link
Member Author

/rebuild

@SethTisue SethTisue merged commit 91a7d72 into scala:2.13.x Apr 26, 2018
@SethTisue SethTisue deleted the use-xml-free-partest branch April 26, 2018 11:08
@SethTisue
Copy link
Member Author

https://travis-ci.org/scala/scala/jobs/371503760 was green and the new nightly it published is 2.13.0-pre-91a7d72

@lrytz
Copy link
Member

lrytz commented Apr 26, 2018

Small fallout: https://travis-ci.org/scala/scala-dist-smoketest/builds/371518382

[error] /home/travis/build/scala/scala-dist-smoketest/src/main/scala/scaladist/XmlTest.scala:5: To compile XML syntax, the scala.xml package must be on the classpath.
[error] Please see https://github.com/scala/scala-xml for details.
[error]     <hallo>mai mai</hallo> match {
[error]     ^
[error] one error found

@SethTisue
Copy link
Member Author

thanks... I'll take care of scala-dist and scala-dist-smoketest. I'm also working on updating the 2.13 community build for this.

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