-
Notifications
You must be signed in to change notification settings - Fork 855
Careful cygwin support #1003
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
Merged
Merged
Careful cygwin support #1003
Changes from 7 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
c1a17fd
Changes to `arch/` directory to support Cygwin.
DWesl 358a7f1
Changes to configure to support Cygwin.
DWesl f498b94
Changes to external/io_netcdf/makefile to support Cygwin.
DWesl bdb519d
Changes to tools/Makefile needed for Cygwin compatibility.
DWesl 7b7e0d4
Changes to arch/Config.pl to support Cygwin.
DWesl 2a722c1
A few changes to get compilation working on Cygwin again.
DWesl af8cfdb
Trigger Jenkins
davegill 272054e
Small comment change to trigger jenkins
davegill 2e83959
Add a README for Cygwin installation instructions.
DWesl 28cddbb
Merge in changes from GitHub, mostly for Jenkens.
DWesl 828aa5e
Mention variables to find dependencies in Cygwin README.
DWesl 289ecdb
Mention how to deal with `gfortran` version mismatch in `.mod` files.
DWesl d2569f0
Remove unneeded sections of the preamble processing.
DWesl File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DWesl
Daniel,
Why do you replicate these 100+ lines of perl? This code already exist from about line 440 through 560.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Processing preamble. I didn't know what
LIB_BUNDLEDandLIB_EXTERNALassumed to be defined, so I just moved everything there.I'll try paring this down sometime tomorrow. It will likely be next week before I can test
dmparconfigurations, but I might be able to get toserialandsmparbuilds before then. I don't remember what I tested the first time through.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cleaned to just the variables referenced in
LIB_EXTERNALandLIB_BUNDLED