Skip to content

Conversation

@SethTisue
Copy link
Member

@SethTisue SethTisue commented Jul 19, 2022

@SethTisue
Copy link
Member Author

SethTisue commented Jul 20, 2022

FAILURES (UNEXPECTED): scala-collection-contrib

https://scala-ci.typesafe.com/job/scala-2.13.x-jdk11-integrate-community-build/3818/artifact/logs/scala-collection-contrib-build.log

a series of "incompatible type in overriding" compilation errors, the first of which is

[scala-collection-contrib] [error] /home/jenkins/workspace/scala-2.13.x-jdk11-integrate-community-build/target-0.9.17/project-builds/scala-collection-contrib-10f20e94f4f64087cad7a7582bf16efe24375298/src/main/scala/scala/collection/MultiDict.scala:11:7: incompatible type in overriding
[scala-collection-contrib] [error] def addString(b: StringBuilder, start: String, sep: String, end: String): b.type (defined in trait IterableOnceOps)
[scala-collection-contrib] [error]   with override def addString(sb: StringBuilder, start: String, sep: String, end: String): StringBuilder (defined in trait MultiDictOps);
[scala-collection-contrib] [error]  found   : (sb: StringBuilder, start: String, sep: String, end: String): StringBuilder
[scala-collection-contrib] [error]  required: (b: StringBuilder, start: String, sep: String, end: String): b.type
[scala-collection-contrib] [error] trait MultiDict[K, V]
[scala-collection-contrib] [error]       ^

regression? progression? or just a change downstream needs to adapt to?

do I need to bisect?

@SethTisue
Copy link
Member Author

oh, it's obviously this StringBuilder PR: scala/scala#10047

@som-snytt offhand, I guess the PR is fine and we just need to change the code in scala-collection-contrib? wdyt?

@SethTisue
Copy link
Member Author

I'm going to merge this PR regardless — it's only one red repo, so let's book the overall progress

@SethTisue SethTisue merged commit 72a46f9 into scala:2.13.x Jul 20, 2022
@SethTisue SethTisue deleted the new-scala-2.13-sha-20220719 branch July 20, 2022 16:57
@som-snytt
Copy link

There is probably more to the story.

It was a cliffhanger!

SethTisue added a commit to SethTisue/scala-collection-contrib that referenced this pull request Jul 20, 2022
namely scala/scala#10047

this came up at scala/community-build#1579

not for merge until 2.13.9 is actually released; in the meantime
we can use the branch in the community build
@SethTisue
Copy link
Member Author

The one-line change to scala-collection-contrib is scala/scala-collection-contrib#185

To make the repo green again here, I pushed 046e880

SethTisue added a commit to SethTisue/scala-collection-contrib that referenced this pull request Oct 13, 2022
includes a small tweak necessary in order to adapt to
scala/scala#10047

this originally came up at
scala/community-build#1579
SethTisue added a commit to SethTisue/scala-collection-contrib that referenced this pull request Oct 27, 2022
includes a small tweak necessary in order to adapt to
scala/scala#10047

this originally came up at
scala/community-build#1579
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants