Skip to content

[CI] build failing #129

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.

Sign up for GitHub

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

Jump to bottom
Closed
mayeut opened this issue Apr 5, 2021 · 6 comments
Closed

[CI] build failing #129

mayeut opened this issue Apr 5, 2021 · 6 comments

Comments

@mayeut
Copy link
Contributor

mayeut commented Apr 5, 2021

CI builds are failing on:

  • ci/circleci: manylinux-x64_cp37-cp37m_upload-sdist
  • ci/circleci: manylinux-x86_cp37-cp37m
  • travis-ci: macOS

PR #128 should fix the issue with manylinux.
I gave up trying to fix macOS.
@jcfr
Copy link
Contributor

jcfr commented Apr 7, 2021

travis-ci: macOS

In the short term, suggested path forward is to try running brew uninstall --cask --force java before brew update

For reference here is the error:

[travis:install_pyenv.py] Executing: brew update
==> Downloading https://github.com/Homebrew/homebrew-portable-ruby/releases/download/2.6.3_2/portable-ruby-2.6.3_2.yosemite.bottle.tar.gz
######################################################################## 100.0%
==> Pouring portable-ruby-2.6.3_2.yosemite.bottle.tar.gz
==> Homebrew is run entirely by unpaid volunteers. Please consider donating:
https://github.com/Homebrew/brew#donations
Updated 2 taps (homebrew/cask and homebrew/core).
==> java has been moved to Homebrew.
To uninstall the cask, run:
brew uninstall --cask --force java
==> Installing java...
Warning: You are using macOS 10.13.
We (and Apple) do not provide support for this old version.
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Twitter or any other official channels. You are responsible for resolving
any issues you experience while you are running this
old version.

Source: https://travis-ci.com/github/scikit-build/cmake-python-distributions/jobs/496798097

jcfr added a commit to mayeut/cmake-python-distributions that referenced this issue Apr 7, 2021
@jcfr
ci: Attempt to fix travis failures uninstalling java
c1ae577 
See scikit-build#129
@mayeut
Copy link
Contributor Author

mayeut commented Apr 7, 2021

tried that => failed with unknown arg
finally succeeded to remove java, it failed later (timeout) rebuilding gcc for something else: https://travis-ci.com/github/scikit-build/cmake-python-distributions/jobs/496097513

and the other macOS at that point failed with other errors I did not check this one or try to fix it: https://travis-ci.com/github/scikit-build/cmake-python-distributions/jobs/496097514

@jcfr
Copy link
Contributor

jcfr commented Apr 7, 2021

In the meantime, I will revert my change and integrate the PR.

@mayeut
Copy link
Contributor Author

mayeut commented Apr 7, 2021

@jcfr,

I went another way in #131 than trying to fix brew on macOs.
It's not clean enough to be merged as-is but it's too late for me to continue this tonight.
Feel free to experiment some clean-ups in this branch if you want to, otherwise I'll have another look at it this week.

@jcfr
Copy link
Contributor

jcfr commented Apr 7, 2021

Thanks so much 🙏

It's not clean enough to be merged as-is but it's too late for me to continue this tonight.
Feel free to experiment some clean-ups in this branch if you want to, otherwise I'll have another look at it this week.

I understand, and I will make sure to comment here if I make some more progress

jcfr added a commit to mayeut/cmake-python-distributions that referenced this issue Apr 9, 2021
@mayeut @jcfr
ci: Fix macOS on TravisCI
b3d04a4 
This commit fixes macOS continuous integration applying the following changes:
* remove the use of brew/pyenv
* install official python distribution
* install certifi using script from cibuildwheel
* create a virtual environment in  ~/.pyenv/versions/${PYTHON_VERSION} sub-directory

For reference, this commit also reverts the following commits:
* 28b0d4b (ci: Update travis to fix "libintl" loading error)
* 0be7aea (ci: Fix macOS build using pip2 instead of pip)
* 6ec58d7 (ci: Tweak cache and before_cache steps)

See scikit-build#129

Co-authored-by: Jean-Christophe Fillion-Robin <[email protected]>
jcfr added a commit to mayeut/cmake-python-distributions that referenced this issue Apr 9, 2021
@mayeut @jcfr
ci: Fix macOS on TravisCI
fdac18b 
This commit fixes macOS continuous integration applying the following changes:
* remove the use of brew/pyenv
* install official python distribution
  - update from 2.7.15 to 2.7.18
  - udpate from 3.7.8 to 3.7.9
* install certifi using script from cibuildwheel
* create a virtual environment in  ~/.pyenv/versions/${PYTHON_VERSION} sub-directory

For reference, this commit also reverts the following commits:
* 28b0d4b (ci: Update travis to fix "libintl" loading error)
* 0be7aea (ci: Fix macOS build using pip2 instead of pip)
* 6ec58d7 (ci: Tweak cache and before_cache steps)

See scikit-build#129

Co-authored-by: Jean-Christophe Fillion-Robin <[email protected]>
@jcfr
Copy link
Contributor

jcfr commented Apr 9, 2021

Closing. Fixed in #128 and #131

@jcfr jcfr closed this as completed Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jcfr @mayeut