add '/out' to ignore file for Visual Studio
Add "/out" to ignored directory for Visual Studio CMake configuration templates in CMakeSettings.json
Reasons for making this change:
Default build directory for CMake configuration of VS are going to use 'out' under of the opened folder
Links to documentation supporting these rule changes:
Doc will be released after the release of VS Preview.
@shiftkey any comment on this PR? thanks
@lukka my current thinking on this is that we should be using the CMake.gitignore template for any rule changes, rather than duplicating rules across templates.
@shiftkey I agree with you, however in this case the default output directory for binaries in the CMakeSettings.json file (consumed by Visual Studio, not by CMake) will contains a path like:
"buildRoot": "${projectDir}\\out\\build\\${name}"
hence it would be nice if the .gitignore for VS would already ignore it.
Any follow up please?
Knock knock
I wholeheartedly agree with this change. I just reported the issue, with my rationale, on devcomm here: https://developercommunity.visualstudio.com/content/problem/1164532/the-cmake-project-template-puts-build-artifacts-in.html
The "out" directory used by VS for CMake projects is in line with many of the entries in this file. CMake.gitignore just lists a number of files used/generated by CMake. OTOH, this PR adds an entry to remove the entire VS-defined directory tree of build-generated files (generated by CMake, and by the build tools used under it).
Documentation supporting this pull request is out. See CMake projects in Visual Studio - IDE Integration.
Click the Show All Files button at the top of Solution Explorer to see all the CMake-generated output in the out/build/
folders .
Reconsider this PR. Alternatively, I created a new one that adds "out" to build results: #3540
After more than 1 year, is there going any action taken on this PR?
This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open :smile:
بھائی میں نے اپ لوگوں کا کیا چرا لیا ہے جو اپ لوگ اتنی ای میلز کر رہے ہو مجھے کیا بات ہے میری سمجھ میں ہی نہیں ا رہا ہے گا میں کوئی اتنا پڑھا لکھا تھوڑی ہوں گا مجھے اپ سمجھائیں کیا کہنا چاہ رہے ہیں اپ لوگ Muhammad Afaq
On Sat, May 24, 2025, 9:33 PM github-actions[bot] @.***> wrote:
github-actions[bot] left a comment (github/gitignore#2994) https://github.com/github/gitignore/pull/2994#issuecomment-2906915896
This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open 😄
— Reply to this email directly, view it on GitHub https://github.com/github/gitignore/pull/2994#issuecomment-2906915896, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASKNPBH7RYSWY6HV67NZVN33ACNMXAVCNFSM6AAAAAB52XCKNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSMBWHEYTKOBZGY . You are receiving this because you are subscribed to this thread.Message ID: @.***>
اور یہ 90 ڈیز اور 180 ڈیز کا کیا معاملہ ہے کیا چیز کلوز ہو جائے گی میں نے تو ایسا کچھ نہیں کیا Muhammad Afaq
On Sat, May 24, 2025, 9:38 PM Muhammad Afaq @.***> wrote:
بھائی میں نے اپ لوگوں کا کیا چرا لیا ہے جو اپ لوگ اتنی ای میلز کر رہے ہو مجھے کیا بات ہے میری سمجھ میں ہی نہیں ا رہا ہے گا میں کوئی اتنا پڑھا لکھا تھوڑی ہوں گا مجھے اپ سمجھائیں کیا کہنا چاہ رہے ہیں اپ لوگ Muhammad Afaq
On Sat, May 24, 2025, 9:33 PM github-actions[bot] < @.***> wrote:
github-actions[bot] left a comment (github/gitignore#2994) https://github.com/github/gitignore/pull/2994#issuecomment-2906915896
This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open 😄
— Reply to this email directly, view it on GitHub https://github.com/github/gitignore/pull/2994#issuecomment-2906915896, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASKNPBH7RYSWY6HV67NZVN33ACNMXAVCNFSM6AAAAAB52XCKNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSMBWHEYTKOBZGY . You are receiving this because you are subscribed to this thread.Message ID: @.***>
This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open :smile:
Hey @lukka, considering my alternative PR (#3540) went in, would you be okay with closing this one?