Skip to content

Conversation

eatkins
Copy link
Contributor

@eatkins eatkins commented Oct 13, 2020

When project/target is a symbolic link, sbt 1.4.0 crashes on startup
because Files.createDirectories will throw a FileAlreadyExistsException.
The fix is to first check if the target directory exists before trying
to create it.

When project/target is a symbolic link, sbt 1.4.0 crashes on startup
because Files.createDirectories will throw a FileAlreadyExistsException.
The fix is to first check if the target directory exists before trying
to create it.
@sideeffffect
Copy link
Contributor

Awesome, thank you!
Btw there could be cases where not only project/target is a symlink, but also project/project/target or project/project/project/target (or how deep the rabbit hole goes). Does this fix address those cases too?

@sideeffffect
Copy link
Contributor

When could we expect this fix to land in a release (1.4.1 I presume)?

@eatkins
Copy link
Contributor Author

eatkins commented Oct 13, 2020

Yes, it would be in 1.4.1 but when that release will land is a question for @eed3si9n

@sideeffffect
Copy link
Contributor

The build seems to have failed because of Internet connectivity issues on Travis side. Otherwise it should be OK.

Copy link
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

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

LGTM Thanks for the fix

@eatkins eatkins merged commit 8e4ec16 into sbt:develop Oct 14, 2020
@eatkins eatkins deleted the symlink-target branch October 14, 2020 03:02
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