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

Partial revert of 837181e2f60198de3701a1f87e66aaf8e69d0e07 #3132

Merged
merged 1 commit into from
Jul 23, 2019

Conversation

Coeur
Copy link
Contributor

@Coeur Coeur commented Jul 23, 2019

Reasons for making this change:

Previous commit on this file (#2566) was incorrectly supported by a 10-years old Stack Overflow answer. There aren't any demonstrated extra files (apart from xcuserdata/ already listed on top) that need to be excluded from the .xcodeproj/ folder, and as such this folder shouldn't be added to .gitignore.

I'm leaving the change on /*.gcno, but updating the description about it. We could note that most Xcode users are using Clang (default compiler) instead of Gcc nowadays.

@shiftkey
Copy link
Member

@Coeur thanks!

@shiftkey shiftkey merged commit 9df8aa6 into github:master Jul 23, 2019
@Coeur
Copy link
Contributor Author

Coeur commented Jul 23, 2019

That was fast.

@shiftkey
Copy link
Member

That was fast.

You caught me at the start of my work day 😉

@Coeur
Copy link
Contributor Author

Coeur commented Jul 23, 2019

For the record, I'd also like to share my Stack Overflow answer from a few months ago:

Regarding Xcode 9 and newer, the only line you really need in your .gitignore is:

xcuserdata/

Nothing else for Xcode 9+.

@Coeur Coeur deleted the Xcode branch July 23, 2019 12:28
dklinzh added a commit to dklinzh/git.ignore that referenced this pull request Jul 25, 2019
tokuhira pushed a commit to tokuhira/gitignore that referenced this pull request Aug 1, 2019
tobetchi added a commit to artteknika/Magic-Sand-for-RealSense that referenced this pull request Sep 4, 2019
r2pgl pushed a commit to r2pgl/gitignore that referenced this pull request Mar 15, 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.

2 participants