Skip to content

[1.x] Fix missing project directory#8583

Merged
eed3si9n merged 2 commits intosbt:1.12.xfrom
azdrojowa123:missing-project-directory
Jan 20, 2026
Merged

[1.x] Fix missing project directory#8583
eed3si9n merged 2 commits intosbt:1.12.xfrom
azdrojowa123:missing-project-directory

Conversation

@azdrojowa123
Copy link
Copy Markdown
Contributor

Fixes #8570

$ copy-file temp/build.sbt build.sbt

# Execute early command with addPluginSbtFile and reload
> early(addPluginSbtFile=temp/extra.sbt); reload No newline at end of file
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't find any way to pass additional command line arguments when using scripted tests to have the test with the same steps as in #8570, but I have little experience with scripted tests
However, these steps also trigger the same exception. Locally, without my fix, this test fails with the same exception as in #8570, and with my fix, it passes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Interestingly project/aggregate is failing, which I'm guessing is because we batch multiple scripted tests into one sbt session. If the manual test works for you, I'm ok without the scripted test.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It seems so :/ It might be useful to run tests in a separate session.
Then, I'll remove the test for now.

- PR sbt#8583. It seems that multiple tests can run in a single sbt session, so adding an extra sbt file causes other tests to fail because the sbt file is not present for them.
@eed3si9n eed3si9n changed the title Fix missing project directory [1.x] Fix missing project directory Jan 20, 2026
@eed3si9n eed3si9n merged commit 16cbaef into sbt:1.12.x Jan 20, 2026
10 checks passed
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.

2 participants