-
Notifications
You must be signed in to change notification settings - Fork 896
wheel build for headless fails on 4.7.0 for Py38 #836
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
Comments
Update: found a thread on the OpenCV forum about this exact problem. The OP there was indeed able to get it done by downgrading to a tagged commit. So I tried that with f9de34e (which is tagged build log
So this time it did not even build OpenCV properly, failing to compile against Numpy. |
Looking more closely at this, it's probably independent – let me know if I should spawn a new issue. EDIT: So this secondary problem could be a duplicate of #713. It seems there's a conflict for Python 3.8 between the numpy version requirements featured by
As the log shows, EDIT: Actually, the log shows that within the skbuild, 1.17.3 was used. And if I instead replace the build-time requirement to the range So to sum up,
|
Ah, I think that works because CI build ( In contrast, the commit hash does not. So I wonder why the actual implementation … Line 68 in 6b73d90
+ ) … Line 205 in 6b73d90
|
The separator used to be a |
fixes #836 i.e. - ability to build untagged commits manually using pip wheel (ensuring the version conforms to PEP 440, and to our actual documentation, by using + instead of .) - ability to compile against current OpenCV on Python 3.8 (using numpy 1.17.5 instead of 1.17.3 which does not compile anymore)
Expected behaviour
When building manually as per documentation, I expect to get a .whl file which I can then
pip install
.(I know I can also install via PyPI on my platform, but I have to build my own wheels.)
Actual behaviour
However, step 5 (
pip wheel
) fails with a strange error message:full build log
(Note: I had to cut off several long passages due to GH's 64k char size constraint, each marked as
...
.)I can only see two potential problems:
Traceback
anddistutils-precedence
.)Built wheel for opencv-python-headless is invalid: Metadata 1.2 mandates PEP 440 version, but '4.7.0.6b73d90' is not
However, 1. seems to be ignored and later OpenCV gets built and the Python wheels (temporarily), too.
And regarding 2., this is only a warning message. (And I don't see why I must conform to that naming scheme for a local distribution. Furthermore, at least for the CI build, it seems ok to build from a non-tagged commit, as this successful CI log for the same commit shows.)
Steps to reproduce
I followed the instructions for manual builds, except that in my case opencv-python is a submodule, so it's
git submodule update --init --recursive opencv-python
instead ofgit clone --recursive https://github.com/opencv/opencv-python.git
.output of git clone
Commands to build:
My environment is:
Issue submission checklist
opencv-python
The text was updated successfully, but these errors were encountered: