Skip to content

Conversation

@zbeekman
Copy link
Collaborator

Normally, once we're on the Github Flow workflow, some of these may have deserved their own branch (i.e., issue-28-gcc-flag-bug, issue-31-make-clean, issue-56-bad-links but in the interest of time and paying down technical debt, I have applied all of these on the same branch, in the same PR.)

@zbeekman
Copy link
Collaborator Author

It seemed as though this patch was not fixing the issue. I've added some more debug output and force pushed (to my private branch) an updated version. Hopefully tests pass on the first try.

 See #28 and #27 for further discussion as well PR Homebrew/legacy-homebrew#46547 Fixes #28 Change default optimization flags for GFortran when the user selects a build type, and ensure that all optimization is always off, until GCC patches the GFortran bug. Signed-off-by: Izaak Beekman <zbeekman@gmail.com>
 More context in #28 Signed-off-by: Izaak Beekman <zbeekman@gmail.com>
 - Fixes #31 - Also don't write config time files to source tree
@zbeekman
Copy link
Collaborator Author

I think I have fixed the issue with test 6, however right now Homebrew's CDN seems to be down, and as a consequence brew install mpich is trying to install from source which times out. As a work around, I'm also going to add in my fix for #51 because it grabs the bottles from Github, which acts as our CDN through uploading them to #65.

@codecov-io
Copy link

Current coverage is 38.57%

Branch #71 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.
Review entire Coverage Diff as of c10af33


Uncovered Suggestions

  1. +1.92% via src/mpi/mpi_caf.c#1953...1969
  2. +1.58% via src/mpi/mpi_caf.c#2051...2064
  3. +1.58% via src/mpi/mpi_caf.c#1618...1631
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

@zbeekman
Copy link
Collaborator Author

Merging this even though, on occasion, #28 is still failing on OS X. If this continues, we can tell Travis-CI to ignore the outcome of the OS-X build; it will still do it, but failures there won't impact the build status...

zbeekman added a commit that referenced this pull request Dec 17, 2015
Fix build failures by enforcing `-O0` when gcc < 5.4 - Theoretically fixes #28, see #71 for potential issues - Closes #56 : Bad links in readme - Closes
@zbeekman zbeekman merged commit c6ddc9e into sourceryinstitute:master Dec 17, 2015
@zbeekman zbeekman deleted the issue-28-opt-flag-fix branch December 17, 2015 23:50
zbeekman added a commit that referenced this pull request Dec 18, 2015
Apply changes from PR #71 via cherry-pick to 1.x branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants