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

gitignore: add cmake build artifacts #1534

Closed

Conversation

thoughtpolice
Copy link
Contributor

No description provided.

@thoughtpolice thoughtpolice force-pushed the wip/gitignore-cmake branch from 28ae46f to 48af092 Compare May 4, 2019 23:05
@alexmiller-apple
Copy link
Contributor

Can I suggest just using a build directory that's outside of the source directory? The existing build/ directory is meant as scripts and tooling to do the building, not as the intended build output directory.

@thoughtpolice
Copy link
Contributor Author

Well, "in nixos", the build system sort of automatically invokes cmake for me during the configure stage and automatically moves to the build directory, and build is generally the "normal" place people run cmake builds. I can change that but -- is there any directory already covered by gitignore, then? It's tedious to see it clutter up git status output.

@mpilman
Copy link
Contributor

mpilman commented May 5, 2019

I tend to agree with @thoughtpolice in a way that I think it is very ugly that we have a build directory in the first place. IMHO we should move these things somewhere else (build-scripts?). build is kind of the default for cmake (as in: people and tools use it as default - cmake doesn't care).

But this will probably need changes in the CI. @AlvinMooreSr do you have any thoughts on this?

@alexmiller-apple alexmiller-apple self-assigned this May 6, 2019
@AlvinMooreSr
Copy link
Contributor

I am not against accepting a PR against the release-6.1 branch which changes our build directory to build-tools.

@alexmiller-apple
Copy link
Contributor

Okay, then let me go create a set of PRs to copy build/ to build-tools/, so that we can adjust the CI without a window of failures.

@mpilman
Copy link
Contributor

mpilman commented May 7, 2020

This is not the correct fix. I created an issue to track this and we should fix this before 7.0

@mpilman mpilman closed this May 7, 2020
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.

4 participants