Skip to content
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

Keep source of external packages in *build* tree #23

Merged
merged 1 commit into from
Apr 22, 2015

Conversation

evaleev
Copy link
Member

@evaleev evaleev commented Apr 22, 2015

cleaner, easier to work with multiple build dirs

@justusc
Copy link
Member

justusc commented Apr 22, 2015

What is the issue with current setup?

@evaleev
Copy link
Member Author

evaleev commented Apr 22, 2015

aesthetics: source pollution (extra effort needed to avoid indexing, etc.), extra effort needed to work with branches (switching branches in the source may require reclone which only happens by doing a build).

I'd say the ExternalPackage mechanism of cmake calls for keeping stuff in build directory. If we used git modules the story would change.

justusc added a commit that referenced this pull request Apr 22, 2015
Keep source of external packages in *build* tree
@justusc justusc merged commit 445cd2e into master Apr 22, 2015
@justusc
Copy link
Member

justusc commented Apr 22, 2015

The original intent was to avoid having to pull these dependencies for multiple builds. However, the tagging system that CMake uses to track changes in these dependencies does not allow that behavior.

Given this and your "pain points" it looks like a good change.

@evaleev evaleev deleted the ev-move-madness-source-to-build branch April 25, 2015 17:25
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.

None yet

2 participants