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: ignore xcode workspaces and projects #1562

Closed
wants to merge 1 commit into from
Closed

gitignore: ignore xcode workspaces and projects #1562

wants to merge 1 commit into from

Conversation

r-52
Copy link
Contributor

@r-52 r-52 commented Apr 29, 2015

Calling ./configure --xcode creates xcode projects and a workspace for io.js for each dep. This commit adds them to the git ignore like vs project files.

Calling ./configure --xcode creates xcode projects and a
workspace for io.js.
@Fishrock123
Copy link
Contributor

Have also ran into this. +1

@Fishrock123 Fishrock123 added the build Issues and PRs related to build files or the CI. label Apr 29, 2015
@petkaantonov
Copy link
Contributor

imo such files belong to your personal/global .gitignore file since they are not specific to a project

@r-52 r-52 closed this Apr 29, 2015
@r-52 r-52 deleted the feature/ignore-project-files branch April 29, 2015 17:58
@petkaantonov
Copy link
Contributor

that was just an imo, no need to close right away :P

@r-52 r-52 restored the feature/ignore-project-files branch April 29, 2015 17:59
@r-52
Copy link
Contributor Author

r-52 commented Apr 29, 2015

Sorry, was little bit toooo fast with some edits

@r-52 r-52 reopened this Apr 29, 2015
@vkurchatkin
Copy link
Contributor

LGTM. We have Visual Studio stuff in already, so this makes sense

@Fishrock123
Copy link
Contributor

An option on our build produces it. I think we should ignore it.

@jbergstroem
Copy link
Member

I agree with @Fishrock123. Files generated by build or configure options should be ignored. LGTM.

@brendanashworth
Copy link
Contributor

Agh, we should get consensus first. I'm +1 for adding it if we generate it.

@chrisdickinson
Copy link
Contributor

LGTM.

@bnoordhuis
Copy link
Member

LGTM as well. I think that's enough consensus?

silverwind pushed a commit that referenced this pull request Apr 30, 2015
Calling ./configure --xcode creates xcode projects and a
workspace for io.js.

PR-URL: #1562
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Reiss <me@silverwind.io>
@silverwind
Copy link
Contributor

Thanks! Landed in 801b47a

@silverwind silverwind closed this Apr 30, 2015
@rvagg rvagg mentioned this pull request May 2, 2015
Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request May 19, 2015
Calling ./configure --xcode creates xcode projects and a
workspace for io.js.

PR-URL: nodejs#1562
Reviewed-By: Vladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Reiss <me@silverwind.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants