-
Notifications
You must be signed in to change notification settings - Fork 34
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
[email protected]_py312: update formula #590
Conversation
building locally will yield the below error on macos Sequoia,
i also believe upstream does not provide a bottle for llvm@15 unfortunately so will have to apply patches to make work with llvm > 15
|
got the same exact error attempting to build
|
it looks numpy2.x changed the dir structure for the header files as seen in the upstream patch for pyside v6 manually patching / editing the CMakeLists.txt file that references numpy in the shiboken2 to change it also appears there are two archives for the pyside source files ie. a
|
7a5ef91
to
baf1e70
Compare
Closes #590. Signed-off-by: Chris <[email protected]>
PR pull merged these changes, so no need to rebase and merge here. |
output from running above command should output something similiar to the below
Not all PRs require passing these checks ie. adding
[no ci]
in the commit message will prevent the CI from running but PRs that change formula files generally should run through the CI checks that way new bottles are built and uploaded to the repository thus not having to build all formula from source but rather installing from a bottle (significantly faster 🐰 ... 🐢)For more information about this template file learn more