Skip to content

Fix #328 checkout to wrong folder using submodules#330

Open
jankorichter wants to merge 1 commit intouwolfer:intellij14from
jankorichter:origin/issue_328
Open

Fix #328 checkout to wrong folder using submodules#330
jankorichter wants to merge 1 commit intouwolfer:intellij14from
jankorichter:origin/issue_328

Conversation

@jankorichter
Copy link
Copy Markdown

@jankorichter jankorichter commented Oct 23, 2018

refs #328: It works for me now.

Copy link
Copy Markdown
Owner

@uwolfer uwolfer left a comment

Choose a reason for hiding this comment

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

Thanks for your PR!

return Optional.of(repository);
}
}
if (repository.getRoot().getCanonicalPath().equals(project.getBaseDir().getCanonicalPath())) {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think this solution looks a bit too simple to be true :)

Are you really sure that the code above is not required anymore?

@jankorichter
Copy link
Copy Markdown
Author

To be honest: no :( It's seems for me, that Intellij doesn't support git submodules (see https://youtrack.jetbrains.com/issue/IDEA-64024). So Jetbrains's multi roots and git's submodules are similar but different. I recommend that I rewrite my commit to use it as a special but prefered case. Do you agree?

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