Skip to content

Add fortran ignore file #1140

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

Closed
wants to merge 2 commits into from

Conversation

peterfpeterson
Copy link
Contributor

FORTRAN isn't the first language ever written, but it is one of the more enduring ones.

Since gfortran is mostly a front-end for gcc, just use their list with a small addition for module files.

@arcresu
Copy link
Contributor

arcresu commented Jul 16, 2014

Thanks! Yes I'm all too familiar with the endurance of fortran in certain environments! Since it's essentially going to be a mirror of the C++ file, we should make this a symlink (like with the current Clojure template). Other than that, this looks like a good idea.

@peterfpeterson
Copy link
Contributor Author

A symlink would be much better. I'm closing this request.

arcresu added a commit that referenced this pull request Jul 16, 2014
@stuartcampbell
Copy link

Would the .mod for fortran just be added to the c++ one then ?

@peterfpeterson
Copy link
Contributor Author

@sic should comment on the pull request that they did process.

@stuartcampbell
Copy link

I would - what was the number

@arcresu
Copy link
Contributor

arcresu commented Jul 18, 2014

Sorry, I forgot about the .mod bit. Yeah, it's probably best to add it to the C++ template (even though that seems a little odd) because the rules are similar enough to fortran's that they make sense as the same template.

@peterfpeterson peterfpeterson deleted the add_fortran branch July 18, 2014 12:28
u9E9F pushed a commit to u9E9F/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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants