[2.x] feat: repositories_force support by bitloi · Pull Request #8761 · sbt/sbt

bitloi · 2026-02-18T04:46:31Z

Summary

Adds support for forcing dependency resolution through proxy repositories by placing a repositories_force file under the sbt global base (e.g. ~/.sbt/repositories_force). When this file exists, sbt sets overrideBuildResolvers := true so resolution uses the proxy repos (e.g. from sbtopts) instead of the build’s resolvers.

Useful in locked-down or corporate environments where all artifact access must go through a proxy.

Changes

Read repositories_force under sbt.global.base (or $SBT_OPTS / sbtopts) and enable override when present.
Add scripted test dependency-management/force-proxy-repos-file to cover the behavior.

Closes #1870

…ixes sbt#1870) - Add Classpaths.shouldOverrideBuildResolvers: true when launcher flag, SBT_OVERRIDE_BUILD_REPOS env, or sbt.override.build.repos property, or when ~/.sbt/repositories_force (or sbt.global.base/repositories_force) exists - Use global base from sbt.global.base or default ~/.sbt for file check - Handle SecurityException on file existence check - overrideBuildResolvers now uses shouldOverrideBuildResolvers

…sbt#1870) - dependency-management/force-proxy-repos-file: with global/repositories_force present (and -Dsbt.global.base=<testDir>/global from scripted), check task asserts overrideBuildResolvers is true

eed3si9n

I think this needs to be implemented in sbt/launcher because the resolution of sbt artifact itself needs to be overridden if you're behind the firewall - https://github.com/search?q=repo%3Asbt%2Flauncher%20overrideRepos&type=code

…erride Avoid enabling override when only the sentinel file exists (e.g. scripted test dirs), which broke dependency-management tests that use custom resolvers (ivyless-publish-maven-http, pom-advanced, transitive-excludes). Add global/repositories to force-proxy-repos-file test.

Prevents force-proxy-repos-file's repositories_force/repositories from leaking into later tests in the same batch (e.g. ivyless-publish-maven-http).

Use find -exec instead of rm $(find ...) so rm is only invoked when there are matches; quoted paths and suppress stderr for robustness.

The sbt launcher does not support a 'repository-override' label in the [ivy] section, causing 'Invalid label(s): repository-override' errors during scripted tests. This label needs to be implemented in sbt/launcher first before it can be used here. The Scala implementation in Defaults.scala that checks for repositories_force file existence at runtime remains valid and functional.

Always remove leftover repositories_force and repositories files from the previous test BEFORE copying the new test content. This prevents force proxy repos from leaking between batched tests while preserving the boot/ directory containing compiler core libraries. If the current test provides these files (e.g. force-proxy-repos-file), IO.copyDirectory restores them after the cleanup.

bitloi · 2026-02-19T00:25:38Z

@eed3si9n Would you mind reviewing the changes again?

eed3si9n · 2026-02-19T06:01:42Z

I just released launcher 1.6.0 with your change. I think you just need to update the launcher and update https://github.com/sbt/sbt/blob/develop/launch/src/main/input_resources/sbt/sbt.boot.properties.

…erties - Bump launcherVersion to 1.6.0 (launcher-interface, launcher) - Add repository-override to [ivy] in default boot properties for force-proxy-repos support (repositories_force file)

bitloi · 2026-02-19T09:04:42Z

I just released launcher 1.6.0 with your change. I think you just need to update the launcher and update https://github.com/sbt/sbt/blob/develop/launch/src/main/input_resources/sbt/sbt.boot.properties.

@eed3si9n I've just updated them. Please check again.

main/src/main/scala/sbt/Defaults.scala

…efaults The launcher (1.6.0+) sets override from repository-override and passes it via isOverrideRepositories(). No need to duplicate the file check in sbt.

...c/repo-override-test/force-proxy-repos-file/force-proxy-repos-file/global/repositories_force

Launcher uses repository-override file as the repo list when present, so we only need global/repositories_force (with content); remove global/repositories.

When scripted uses RunFromSourceBased (e.g. CI), isOverrideRepositories() was hardcoded to false. Check sbt.global.base/repositories_force so force-proxy-repos-file test passes.

eed3si9n · 2026-02-19T19:14:28Z

One thing I forgot to tell you is that scripted by default runs in parallel, so testing the launcher requires a different configuration. Luckily we have a skeleton setup already in https://github.com/sbt/sbt/tree/develop/sbt-app/src/repo-override-test.

step 1: uncomment repo override test

sbt/.github/workflows/ci.yml

Line 163 in b634e1b

# ./sbt -v "repoOverrideTest:scripted dependency-management/*"

step 2: replace the existing override scripted test

You can move your new scripted test into https://github.com/sbt/sbt/tree/develop/sbt-app/src/repo-override-test, and feel free to delete the existing one in there.

steps 3: fix the configuration

sbt/build.sbt

Lines 1042 to 1058 in b634e1b

    
           ) ++ inConfig(Scripted.RepoOverrideTest)( 
        
             Seq( 
        
               scriptedLaunchOpts := List( 
        
                 "-Xmx1500M", 
        
                 "-Xms512M", 
        
                 "-server", 
        
                 "-Dsbt.override.build.repos=true", 
        
                 s"""-Dsbt.repository.config=${scriptedSource.value / "repo.config"}""" 
        
               ) ::: 
        
                 (sys.props.get("sbt.ivy.home") match { 
        
                   case Some(home) => List(s"-Dsbt.ivy.home=$home") 
        
                   case _          => Nil 
        
                 }), 
        
               scripted := scriptedTask(true).evaluated, 
        
               scriptedUnpublished := scriptedTask(true).evaluated, 
        
               scriptedSource := (sbtProj / sourceDirectory).value / "repo-override-test" 
        
             )

The repoOverride specific configuration is done here. Feel free to change whatever you have to there.

- Uncomment repoOverrideTest in CI job 4: run force-proxy-repos-file/* with launcher - Replace default-resolvers with force-proxy-repos-file under repo-override-test - Remove -Dsbt.override.build.repos and -Dsbt.repository.config from RepoOverrideTest so the test asserts launcher/repositories_force behavior - Delete force-proxy-repos-file from dependency-management (parallel scripted)

bitloi · 2026-02-19T22:08:08Z

One thing I forgot to tell you is that scripted by default runs in parallel, so testing the launcher requires a different configuration. Luckily we have a skeleton setup already in https://github.com/sbt/sbt/tree/develop/sbt-app/src/repo-override-test.

step 1: uncomment repo override test

sbt/.github/workflows/ci.yml

Line 163 in b634e1b

# ./sbt -v "repoOverrideTest:scripted dependency-management/*"

step 2: replace the existing override scripted test

You can move your new scripted test into https://github.com/sbt/sbt/tree/develop/sbt-app/src/repo-override-test, and feel free to delete the existing one in there.

steps 3: fix the configuration

sbt/build.sbt

Lines 1042 to 1058 in b634e1b

) ++ inConfig(Scripted.RepoOverrideTest)(

Seq(

scriptedLaunchOpts := List(

"-Xmx1500M",

"-Xms512M",

"-server",

"-Dsbt.override.build.repos=true",

s"""-Dsbt.repository.config=${scriptedSource.value / "repo.config"}"""

) :::

(sys.props.get("sbt.ivy.home") match {

case Some(home) => List(s"-Dsbt.ivy.home=$home")

case _ => Nil

}),

scripted := scriptedTask(true).evaluated,

scriptedUnpublished := scriptedTask(true).evaluated,

scriptedSource := (sbtProj / sourceDirectory).value / "repo-override-test"

)

The repoOverride specific configuration is done here. Feel free to change whatever you have to there.

I've fixed it. Would you mind reviewing it again?

The launcher 1.6.0 bool() default for override-build-repos uses repository-override file existence, but only when the label is absent. The explicit 'override-build-repos: false' line was preventing this auto-detection. sbt.override.build.repos is still checked by shouldOverrideBuildResolvers in Defaults.scala.

bitloi · 2026-02-20T02:43:56Z

@eed3si9n Do you have any other feedback?

eed3si9n

Thanks!

bitloi added 4 commits February 18, 2026 03:08

test: scripted test for force proxy repos via repositories_force file (…

22be517

…sbt#1870) - dependency-management/force-proxy-repos-file: with global/repositories_force present (and -Dsbt.global.base=<testDir>/global from scripted), check task asserts overrideBuildResolvers is true

style: scalafmt Defaults.scala

9d9cb37

chore: empty commit to rerun CI

3f33daf

eed3si9n requested changes Feb 18, 2026

View reviewed changes

bitloi mentioned this pull request Feb 18, 2026

Support repositories_force file for launcher boot resolution sbt/launcher#140

Merged

bitloi requested a review from eed3si9n February 18, 2026 09:56

bitloi added 4 commits February 18, 2026 11:42

Fix batch scripted: clear tempTestDir/global when test has no global/

6644717

Prevents force-proxy-repos-file's repositories_force/repositories from leaking into later tests in the same batch (e.g. ivyless-publish-maven-http).

Fix 'rm: missing operand' in CI cleanup when no SNAPSHOT dirs exist

a8bc520

Use find -exec instead of rm $(find ...) so rm is only invoked when there are matches; quoted paths and suppress stderr for robustness.

Add repository-override to default sbt.boot.properties

23a03cb

bitloi force-pushed the feature/1870-force-proxy-repos branch from b44dec1 to 8662e77 Compare February 18, 2026 23:49

bitloi force-pushed the feature/1870-force-proxy-repos branch from 8662e77 to cd60198 Compare February 18, 2026 23:51

Update launcher to 1.6.0 and add repository-override to sbt.boot.prop…

f0da871

…erties - Bump launcherVersion to 1.6.0 (launcher-interface, launcher) - Add repository-override to [ivy] in default boot properties for force-proxy-repos support (repositories_force file)

eed3si9n reviewed Feb 19, 2026

View reviewed changes

main/src/main/scala/sbt/Defaults.scala Show resolved Hide resolved

Rely on launcher for override; remove repositories_force check from D…

031b1b2

…efaults The launcher (1.6.0+) sets override from repository-override and passes it via isOverrideRepositories(). No need to duplicate the file check in sbt.

bitloi requested a review from eed3si9n February 19, 2026 16:46

eed3si9n reviewed Feb 19, 2026

View reviewed changes

...c/repo-override-test/force-proxy-repos-file/force-proxy-repos-file/global/repositories_force Show resolved Hide resolved

bitloi added 2 commits February 19, 2026 17:58

force-proxy-repos-file: use only repositories_force as repo config

7aa062d

Launcher uses repository-override file as the repo list when present, so we only need global/repositories_force (with content); remove global/repositories.

ScriptedLauncher: honor repositories_force when run-from-source

5ae50b6

When scripted uses RunFromSourceBased (e.g. CI), isOverrideRepositories() was hardcoded to false. Check sbt.global.base/repositories_force so force-proxy-repos-file test passes.

bitloi added 2 commits February 19, 2026 23:31

chore: re-trigger CI to verify lm-coursier/caller flake

bde8613

eed3si9n approved these changes Feb 20, 2026

View reviewed changes

eed3si9n changed the title ~~feat/Force proxy repos via file~~ [2.x] feat: repositories_force support Feb 20, 2026

eed3si9n merged commit c045c72 into sbt:develop Feb 20, 2026
15 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[2.x] feat: repositories_force support#8761

[2.x] feat: repositories_force support#8761
eed3si9n merged 17 commits intosbt:developfrom
bitloi:feature/1870-force-proxy-repos

bitloi commented Feb 18, 2026 •

edited

Loading

Uh oh!

eed3si9n left a comment

Uh oh!

bitloi commented Feb 19, 2026

Uh oh!

eed3si9n commented Feb 19, 2026

Uh oh!

bitloi commented Feb 19, 2026

Uh oh!

Uh oh!

Uh oh!

eed3si9n commented Feb 19, 2026

Uh oh!

bitloi commented Feb 19, 2026

step 1: uncomment repo override test

step 2: replace the existing override scripted test

steps 3: fix the configuration

Uh oh!

bitloi commented Feb 20, 2026

Uh oh!

eed3si9n left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

bitloi commented Feb 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

eed3si9n left a comment

Choose a reason for hiding this comment

Uh oh!

bitloi commented Feb 19, 2026

Uh oh!

eed3si9n commented Feb 19, 2026

Uh oh!

bitloi commented Feb 19, 2026

Uh oh!

Uh oh!

Uh oh!

eed3si9n commented Feb 19, 2026

step 1: uncomment repo override test

step 2: replace the existing override scripted test

steps 3: fix the configuration

Uh oh!

bitloi commented Feb 19, 2026

step 1: uncomment repo override test

step 2: replace the existing override scripted test

steps 3: fix the configuration

Uh oh!

bitloi commented Feb 20, 2026

Uh oh!

eed3si9n left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

bitloi commented Feb 18, 2026 •

edited

Loading