Skip to content

Deliver the shaded ivy.xml only where Ivy publishes - #147

Merged
tgodzik merged 3 commits into
sbt:mainfrom
kitbellew:147
Oct 26, 2026
Merged

tgodzik merged 3 commits into
sbt:mainfrom
kitbellew:147

Conversation

@kitbellew

Copy link
Copy Markdown
Contributor

sbt 2.1 has no Ivy publishing of its own: deliverLocal says an Ivy-based plugin is needed and ivyModule is unit. Loading IvyRepublish also stopped the plugin loading at all on sbt 2.1, which removed Ivy from the plugin classpath.

Now the plugin republishes only where IvyRepublish loads. Elsewhere, as on sbt 2.1, it rewrites the ivy.xml that publishLocal wrote, and its checksums, and CI runs the sbt 2 scripted tests on sbt 2.1 too.

kitbellew and others added 3 commits October 12, 2026 13:37
project-matrix-shading expected shading a Scala.js row to fail on every
sbt 2, since the platform was not part of the CrossVersion. From sbt
2.0.7 (sbt/sbt#9621) and 2.1 (sbt/sbt#9620) it is, so the row shades,
and the test failed there.

Now the test expects the failure only before 2.0.7.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
sbt 2.1 has no Ivy publishing of its own: deliverLocal says an
Ivy-based plugin is needed and ivyModule is unit. Loading
IvyRepublish also stopped the plugin loading at all on sbt 2.1, which
removed Ivy from the plugin classpath.

Now the plugin republishes only where IvyRepublish loads. Elsewhere,
as on sbt 2.1, it rewrites the ivy.xml that publishLocal wrote, and
its checksums, and CI runs the sbt 2 scripted tests on sbt 2.1 too.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@kitbellew

Copy link
Copy Markdown
Contributor Author

@alexarchambault @tgodzik tried updating sbt in scalameta to 2.1.0-RC1 (which dropped explicit ivy support), and this is what claude proposed as a fix.

@tgodzik tgodzik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not an expert here, but the PR seems to make sense. @alexarchambault any opinion?

@tgodzik
tgodzik merged commit 6db6ff8 into sbt:main Oct 26, 2026
3 checks passed
@kitbellew
kitbellew deleted the 147 branch October 26, 2026 14:37
@kitbellew

Copy link
Copy Markdown
Contributor Author

@tgodzik thank you. any thought on releasing?

@tgodzik

tgodzik commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Let's see if it works https://github.com/coursier/sbt-shading/actions/runs/41475339364 I know @alexarchambault was changing things around publishing

@tgodzik

tgodzik commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Looks like we can publish here anymore, looks more complicated. @alexarchambault how can we release this? Or should we fork the repo under scalameta?

@alexarchambault

Copy link
Copy Markdown
Contributor

@tgodzik Feel free to fork it and publish it from another org. I don't use it anymore in my projects, and I'd rather let others maintain it and publish it.

@kitbellew

Copy link
Copy Markdown
Contributor Author

Looks like we can publish here anymore, looks more complicated. @alexarchambault how can we release this? Or should we fork the repo under scalameta?

@tgodzik who can help with bringing it into the scalameta umbrella and setting up publishing/secrets/etc? sbt 2.0.x has some issues which 2.1.x presumably fixes, so this release will come in handy.

@tgodzik

tgodzik commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

We discussed it and I think we can bring it to sbt org. @alexarchambault it's probably easiest to transfer the repo to a different org, would you be able to do that?

@tgodzik

tgodzik commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

transfer the repo to a different org,

Meaning sbt obviously. I will take care of everything there.

@alexarchambault

Copy link
Copy Markdown
Contributor

@tgodzik I don't have the rights to create a repository in the sbt org (needed to transfer the repo there apparently), but I gave you admin rights to the repo here, so that you can proceed

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