Skip to content
This repository has been archived by the owner on Feb 15, 2023. It is now read-only.

1.0.x broken on macOS #27

Closed
rgommers opened this issue Mar 24, 2018 · 11 comments
Closed

1.0.x broken on macOS #27

rgommers opened this issue Mar 24, 2018 · 11 comments
Labels

Comments

@rgommers
Copy link
Contributor

See https://travis-ci.org/MacPython/scipy-wheels/builds/357678450, ends repeatably with sudo: pip: command not found. Looks like multibuild was updated for various macOS issues, so looks like the right fix is to update to latest multibuild also on the 1.0.x branch. Opening this issue to ask if anyone sees an issue with that.

rgommers added a commit to rgommers/scipy-wheels that referenced this issue Mar 24, 2018
@rgommers
Copy link
Contributor Author

Okay, gave that a try in 170955c

@rgommers
Copy link
Contributor Author

Great, those builds errored after 5 minutes and TravisCI is not showing the full build log even with "raw log". That needs reverting (tomorrow), guess we'll try and cherry-pick the relevant macOS fixes. Not sure how well that works with a git submodule - is it a good idea to just add some commits directly to a submodule??

@rgommers rgommers added the bug label Mar 24, 2018
@pv
Copy link
Contributor

pv commented Mar 24, 2018

I think it's not possible to add commits to a submodule, it always has to refer to a commit available on upstream.

@matthew-brett
Copy link
Contributor

matthew-brett commented Mar 24, 2018 via email

@pv
Copy link
Contributor

pv commented Mar 24, 2018

Note that the new travis builds were terminated with an exclamation mark, not x --- I restarted the build in case this is just intermittent failure.

@pv
Copy link
Contributor

pv commented Mar 24, 2018

Nope, still fails. Pushed a commit into v1.0.x that makes it use the same multibuild version as the weekly master branch builds, which seemed to be in working order.

@pv
Copy link
Contributor

pv commented Mar 24, 2018

Travis build seems to pass OK with that version of multibuild.

@rgommers
Copy link
Contributor Author

Ah I updated to multibuild/master while multibuild/devel is the one that worked. Thanks Pauli.

@matthew-brett would be useful to fast-forward master to devel perhaps?

@matthew-brett
Copy link
Contributor

Multibuild master is nearly the same as multibuild devel though - just a few tiny differences. The version of multibuild in the scipy-wheels / master branch is a fair bit behind. I'm just trying to work out where the damage comes from :

@matthew-brett
Copy link
Contributor

Now building correctly on multibuild master - so not sure what the problem was: https://travis-ci.org/matthew-brett/scipy-wheels/builds/357931159 - maybe https://www.traviscistatus.com/incidents/n0f1kmm5k4x5

@matthew-brett
Copy link
Contributor

Aha - sorry - there is a bug in the suppress macro - but the libwebp and szip builds are failing - search the log for tar: Unrecognized archive format

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants