-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
disable build isolation which causes problems for windows #38
Conversation
Why am I running into such strange error only on Python 3.7 for windows. |
@conda-forge/python @conda-forge/core @conda-forge/scikit-image In previous builds, it seems that windows was pulling in numpy from pypi as outlined here Although windows failed on 3.7. I'm pretty inclined to merge this in.
If somebody wants to tag in for windows, that would be greatly appreciated!!!! |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge/scikit-image does anybody have any ideas why the builds are failing on windows. I've tried:
It builds on my own computer, but not Appveyor |
@jakirkham is probably the best person to answer this... the subprocess call to bld.bat is failing, but no clues as to why. @jakirkham is there a way to ensure that the stdout and stderr from that call get printed on failure? @hmaarrfk if not, that might be a good contribution to make to conda build. Command "C:\bld\scikit-image_1546480104042\_h_env\python.exe -u -c "import setuptools, tokenize;__file__='C:\\Users\\appveyor\\AppData\\Local\\Temp\\1\\pip-req-build-_pr74h7r\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record C:\Users\appveyor\AppData\Local\Temp\1\pip-record-lecq_zi1\install-record.txt --single-version-externally-managed --compile" failed with error code 3221225477 in C:\Users\appveyor\AppData\Local\Temp\1\pip-req-build-_pr74h7r\
Exception information:
Traceback (most recent call last):
File "C:\bld\scikit-image_1546480104042\_h_env\lib\site-packages\pip\_internal\cli\base_command.py", line 143, in main
status = self.run(options, args)
File "C:\bld\scikit-image_1546480104042\_h_env\lib\site-packages\pip\_internal\commands\install.py", line 366, in run
use_user_site=options.use_user_site,
File "C:\bld\scikit-image_1546480104042\_h_env\lib\site-packages\pip\_internal\req\__init__.py", line 49, in install_given_reqs
**kwargs
File "C:\bld\scikit-image_1546480104042\_h_env\lib\site-packages\pip\_internal\req\req_install.py", line 791, in install
spinner=spinner,
File "C:\bld\scikit-image_1546480104042\_h_env\lib\site-packages\pip\_internal\utils\misc.py", line 705, in call_subprocess
% (command_desc, proc.returncode, cwd))
pip._internal.exceptions.InstallationError: Command "C:\bld\scikit-image_1546480104042\_h_env\python.exe -u -c "import setuptools, tokenize;__file__='C:\\Users\\appveyor\\AppData\\Local\\Temp\\1\\pip-req-build-_pr74h7r\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record C:\Users\appveyor\AppData\Local\Temp\1\pip-record-lecq_zi1\install-record.txt --single-version-externally-managed --compile" failed with error code 3221225477 in C:\Users\appveyor\AppData\Local\Temp\1\pip-req-build-_pr74h7r\
Traceback (most recent call last):
File "C:\Miniconda36-x64\Scripts\conda-build-script.py", line 10, in <module>
sys.exit(main())
File "C:\Miniconda36-x64\lib\site-packages\conda_build\cli\main_build.py", line 438, in main
execute(sys.argv[1:])
File "C:\Miniconda36-x64\lib\site-packages\conda_build\cli\main_build.py", line 429, in execute
verify=args.verify)
File "C:\Miniconda36-x64\lib\site-packages\conda_build\api.py", line 201, in build
notest=notest, need_source_download=need_source_download, variants=variants)
File "C:\Miniconda36-x64\lib\site-packages\conda_build\build.py", line 2204, in build_tree
notest=notest,
File "C:\Miniconda36-x64\lib\site-packages\conda_build\build.py", line 1394, in build
windows.build(m, build_file, stats=build_stats)
File "C:\Miniconda36-x64\lib\site-packages\conda_build\windows.py", line 316, in build
check_call_env(cmd, cwd=src_dir, stats=stats)
File "C:\Miniconda36-x64\lib\site-packages\conda_build\utils.py", line 313, in check_call_env
return _func_defaulting_env_to_os_environ('call', *popenargs, **kwargs)
File "C:\Miniconda36-x64\lib\site-packages\conda_build\utils.py", line 293, in _func_defaulting_env_to_os_environ
raise subprocess.CalledProcessError(proc.returncode, _args)
subprocess.CalledProcessError: Command '['cmd.exe', '/c', 'bld.bat']' returned non-zero exit status 1. macOS is also failing on Travis, also not sure why: https://travis-ci.org/conda-forge/scikit-image-feedstock/jobs/474665631#L253-L256 Traceback (most recent call last):
File "/Users/travis/miniconda3/bin/setup_conda_rc", line 7, in <module>
from conda_forge_ci_setup.build_utils import setup_conda_rc
ModuleNotFoundError: No module named 'conda_forge_ci_setup' |
@jni, thanks for chiming in. How do you feel about merging this now, as is, it would fix things on windows for python 2.7 and 3.6???? mac osx errors seem to be related to caching on the servers. It has been happening CI wide. Not too worried about those. |
@hmaarrfk what do you mean "CI wide"? All conda-forge? |
recipe/meta.yaml
Outdated
# sha256: 86a9b3b4f74f231e0a6bcfd3235dcf3f0118df25dac21201da5e064d681e2c50 | ||
url: https://github.com/scikit-image/scikit-image/archive/v{{ version }}.tar.gz | ||
fn: scikit-image-{{ version }}.tar.gz | ||
sha256: 8da6fb09aeefb757735c510650ac0072be3831fa76d9747285f3c6ea1e0c5a08 |
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.
Why are the sha hashes different on GH and PyPI???
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.
pypi includes cythonized files. and probably generated with "python setup.py sdist"
It seems to be isolated to OSX. OSX seems like it is installing 5 month old packages. |
@hmaarrfk I would prefer if we keep PyPI as the source. Would that be fine, given that this PR doesn't fix the situation on 3.7? Or do the other Python versions also fail without that change? |
Yes but OSX for everyone or OSX for NumPy dependent packages or OSX for us? |
It doesn't even get to numpy, it is just the ci-setup stuff on conda-forge. |
Anyway, @jakirkham thinks that this kind of stuff should have been resolved internally in conda-forge, but something might have slipped |
Coolio, if CI passes, minus the expected failures, do feel free to merge. 👍 Thanks yet again for all this painful but essential build work! |
So I only care if windows 2.7 and 3.6 pass. |
Got it! |
The build got futher this time?????? I have a feeling it is related to the specific version of the compiler that appveyor may or may not have upgraded to :/ Could be related to this. |
Here's the relevant bit about Travis CI's macOS builds. It looks like an edge caching issue, which it sounds like Anaconda had some issues with today. Probably gets resolved tomorrow.
To be clear, that is something |
Ok, i'm going to merge this, and start a new CI run just for windows. |
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)