-
Notifications
You must be signed in to change notification settings - Fork 163
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
dev-python/cadquery: updating dependencies #1211
base: master
Are you sure you want to change the base?
Conversation
da2515d
to
53942bc
Compare
UPD: Nevermind, I had to revert fix for the |
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
Signed-off-by: Vladimir Shapranov <equidamoid@gmail.com>
53942bc
to
476457b
Compare
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.
When adding 3.11 it is not required to drop 3.10. 3.10 is still supported by us so please keep it around if possible. Have you also tested 3.12? If we can also add 3.12 now it will save work later.
@@ -4,7 +4,7 @@ | |||
EAPI=7 | |||
|
|||
CMAKE_ECLASS=cmake | |||
PYTHON_COMPAT=( python3_10 pypy3 ) | |||
PYTHON_COMPAT=( python3_11 pypy3 ) |
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.
3.10 doesn't work in this version?
@@ -27,6 +27,8 @@ KEYWORDS="~amd64 ~x86" | |||
IUSE="c examples +python static-libs" | |||
REQUIRED_USE="python? ( ${PYTHON_REQUIRED_USE} !abi_x86_32 !abi_x86_x32 )" | |||
|
|||
PATCHES=( "${FILESDIR}" ) |
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 is this here?
Now we have the additional problem of cadquery depending on |
Sorry, didn't have time to look into this. Will try to come back to it soon. |
Needs bump and port to newer python targets Work on-going in: #1211 Possibly re-add later. Signed-off-by: Andrew Ammerlaan <andrewammerlaan@gentoo.org>
This has been treecleaned for now, but we can still re-add it later. We need to:
|
The actual cadquery changes are coming later in a separate PR, pywrap needs some patching