-
Notifications
You must be signed in to change notification settings - Fork 83k
add '/out' to ignore file for Visual Studio #2994
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
base: main
Are you sure you want to change the base?
Conversation
@shiftkey any comment on this PR? thanks |
@lukka my current thinking on this is that we should be using the |
@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:
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.
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 😄 |
بھائی میں نے اپ لوگوں کا کیا چرا لیا ہے جو اپ لوگ اتنی ای میلز کر رہے ہو
مجھے کیا بات ہے میری سمجھ میں ہی نہیں ا رہا ہے گا میں کوئی اتنا پڑھا لکھا
تھوڑی ہوں گا مجھے اپ سمجھائیں کیا کہنا چاہ رہے ہیں اپ لوگ
Muhammad Afaq
…On Sat, May 24, 2025, 9:33 PM github-actions[bot] ***@***.***> wrote:
*github-actions[bot]* left a comment (github/gitignore#2994)
<#2994 (comment)>
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
<#2994 (comment)>,
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)
> <#2994 (comment)>
>
> 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
> <#2994 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ASKNPBH7RYSWY6HV67NZVN33ACNMXAVCNFSM6AAAAAB52XCKNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSMBWHEYTKOBZGY>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: ***@***.***>
>
|
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.