Skip to content

Fix #2299: Fix import into IDEs #2316

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 27, 2017
Merged

Conversation

smarter
Copy link
Member

@smarter smarter commented Apr 27, 2017

Both IntelliJ and Eclipse fail horribly when trying to import a project
using unmanagedSourceDirectories containing files (and not
directories), but this is necessary for our build since we only want to
compile a subset of the backend folder. We workaround this by
linking (or copying on OS when symbolic links do not work) the files we
want to compile in sbt managedSources.

Make sure to run sbt compile at least once before running sbt eclipse or importing the project in IntelliJ.

@smarter smarter requested review from odersky and felixmulder April 27, 2017 12:42
Both IntelliJ and Eclipse fail horribly when trying to import a project
using `unmanagedSourceDirectories` containing files (and not
directories), but this is necessary for our build since we only want to
compile a subset of the backend folder. We workaround this by
linking (or copying on OS when symbolic links do not work) the files we
want to compile in sbt `managedSources`.

Make sure to run `sbt compile` at least once before running `sbt
eclipse` or importing the project in IntelliJ.
Copy link
Contributor

@odersky odersky left a comment

Choose a reason for hiding this comment

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

Looks good, even though I am not familiar with the details here. But it surely is no worse than the status quo 😄

@odersky odersky merged commit 28d4057 into scala:master Apr 27, 2017
@allanrenucci allanrenucci deleted the fix/ide-import branch December 14, 2017 19:22
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