Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

"profile" doesn't belong in Objective-C.gitignore #956

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

JimRoepcke commented Feb 19, 2014

I've never seen a valid argument for keeping this line in this file, yet I've seen dozens of people complain about it. It's bitten me and lots of others in the past.

Collaborator

arcresu commented Feb 20, 2014

Thanks for this! I'm not familiar with Objective C, and I can't easily find anything about this file by searching, so would you be able to explain why people have suggested it be ignored and your case against it?

arcresu referenced this pull request Apr 4, 2014

Contributor

trevorsheridan commented Apr 4, 2014

Reading some blog posts it seems like this was a mystery to some people all the way back in 2009. I have no idea what the historical reasoning behind this is. One theory in reading most people's .gitignores, is they include a comment above the "profile" line saying something like "OS X junk", and it's typically coupled with the .DS_Store. So this leads me to believe that a profile (not .profile) was something generated by OS X. Either that or maybe convention where the output of profiles(1) was a file called profile? I'm shooting in the dark here which is probably reason enough to just remove this line until someone says otherwise.

Collaborator

arcresu commented Apr 4, 2014

Yep, sounds good. Thanks for doing the research. I'll fix that up

@arcresu arcresu closed this in d598b29 Apr 4, 2014

arcresu added a commit that referenced this pull request Apr 6, 2014

drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014

drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014

kaiserhl pushed a commit to kaiserhl/gitignore that referenced this pull request Jul 8, 2017

kaiserhl pushed a commit to kaiserhl/gitignore that referenced this pull request Jul 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment