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: Add Windows intermediate files #3087

Merged
merged 1 commit into from
Mar 22, 2015

Conversation

CyberShadow
Copy link
Member

No description provided.

@JakobOvrum
Copy link
Member

Yeah this was pretty annoying. LGTM.

@quickfur
Copy link
Member

👍

@quickfur
Copy link
Member

Auto-merge toggled on

quickfur pushed a commit that referenced this pull request Mar 22, 2015
.gitignore: Add Windows intermediate files
@quickfur quickfur merged commit f8e1157 into dlang:master Mar 22, 2015
@andralex
Copy link
Member

I continue to believe that growth of .gitignore files is a symptom of a larger problem: a disorganized build process that leaves junk files left where they don't belong. Am I alone?

@cym13
Copy link

cym13 commented Mar 22, 2015

@andralex : That's exactly the comment I would have written!

@CyberShadow
Copy link
Member Author

@andralex In this instance, you are right. The Phobos win32.mak is less organized in where it creates temporary files than posix.mak, but I didn't want to risk breaking things by moving things around.

@CyberShadow
Copy link
Member Author

BTW, there are some entries in our .gitignore files that shouldn't be there. For example, the Makefile and GNUmakefile entries do not belong in .gitignore, they should be in .git/info/exclude. And I've no idea what the .*.sw* is for.

@schuetzm
Copy link
Contributor

*.*.sw* is for the temporary files that vi creates during editing of a file.

@CyberShadow
Copy link
Member Author

Those should not be in our .gitignore either. They should be in users' global .gitignore.

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.

6 participants