-
Notifications
You must be signed in to change notification settings - Fork 101
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
macOS: upgrade earliest supported version to 10.13 #164
Conversation
sharp-libvips/darwin-x64/Toolchain.cmake Lines 4 to 5 in 34c25b5
Should probably be updated as well. |
480bed8
to
3b4b9ce
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.
Great! I'll see if I can resolve this in Meson as well, which would remove the need for this sed replacement and possibly MR https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/78.
Looks like the latest meson v0.64.0 as installed via Homebrew is broken (updated < 1 hour ago). |
Ouch, perhaps removing
IIUC, that would prevent Meson from being updated. But when GitHub updates its macOS runners it probably starts failing again. 😥 |
... looking at the logs, I see this:
Looks like Homebrew is migrating to Python 3.11. We could consider reverting #163 if this happens more often. Perhaps that issue was fixed with commit actions/runner-images@f6632ff? |
Ah, maybe we should try using The sharp CI environment uses this to pin Python v3.10 as the version of |
Prevent use of API newer than deployment target Patch orc to prevent use of pthread_jit functions on 10.x
3b4b9ce
to
f3b513a
Compare
Quickest solution was to temporarily remove This can be added back once every Python-dependent package in Homebrew has been updated. |
Does this mean that libvips older than 10.13 is no longer supported? I have a globally built libvips (vips-8.11.4-Thu Sep 23 09:40:01 UTC 2021) but no matter what I do, npm install --build-from-source still tries to download the latest 8.13.3, which results in the same error:
|
@ongjiinjoo Please see lovell/sharp#3438 for the details behind this. |
The documentation already says
>= 10.13
https://sharp.pixelplumbing.com/install#prebuilt-binaries
Also:
pthread_jit_
functions on 10.x (x64)