Skip to content

Issue1263 durack1 Fix Travis OS X pyexpat build error#1674

Merged
doutriaux1 merged 2 commits intomasterfrom
issue1263_durack1_FixTravisOSXPyexpatBuildError
Nov 12, 2015
Merged

Issue1263 durack1 Fix Travis OS X pyexpat build error#1674
doutriaux1 merged 2 commits intomasterfrom
issue1263_durack1_FixTravisOSXPyexpatBuildError

Conversation

@durack1
Copy link
Copy Markdown
Member

@durack1 durack1 commented Nov 11, 2015

Fix #1263 - Travis OS X pyexpat build error - build against system expat

@aashish24 @doutriaux1 please review and merge - this will fix the travis OS X build errors, but it's a kludge as the expat build errors are likely an artifact of problems with the build environment not being set up correctly - when running a Python 2.7.10 build (which includes expat and pyexpat) this completes successfully with no warnings outside of the cmake build command - take a peek at PCMDI/pcmdi_metrics#285 for more info.. PCMDI/pcmdi_metrics#279 was the issue that triggered this PR

@durack1
Copy link
Copy Markdown
Member Author

durack1 commented Nov 11, 2015

@durack1
Copy link
Copy Markdown
Member Author

durack1 commented Nov 12, 2015

@aashish24 @jbeezley this is the PR that @doutriaux1 wanted you folks to take a look at..

Comment thread .travis.yml
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jbeezley did you add this line?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aashish24 nope that was me - and it's redundant and did nothing.. For it to work to extend timeouts it needs to precede the command line arguments

@durack1
Copy link
Copy Markdown
Member Author

durack1 commented Nov 12, 2015

@aashish24 any luck with reviewing this/figuring out what was/is happening? We're waiting on the PR to be merged so we can fix the downstream issue at PCMDI/pcmdi_metrics#279 along with some other fixes in the queue..

doutriaux1 added a commit that referenced this pull request Nov 12, 2015
…yexpatBuildError

Issue1263 durack1 Fix Travis OS X pyexpat build error
@doutriaux1 doutriaux1 merged commit 709468c into master Nov 12, 2015
@doutriaux1 doutriaux1 deleted the issue1263_durack1_FixTravisOSXPyexpatBuildError branch November 12, 2015 22:41
@aashish24
Copy link
Copy Markdown
Contributor

thanks @durack1

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.

Fix travis pyexpat build time error

4 participants