Skip to content
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

Don't use the version of libheif that's known to be broken on Mac #2847

Merged
merged 1 commit into from Jan 28, 2021

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Jan 28, 2021

This was causing failed CI tests and probably confused users.
libheif 1.10 is very broken on MacOS. The author says that 1.11 will come soon.

@lgritz lgritz merged commit d985b0f into AcademySoftwareFoundation:master Jan 28, 2021
@lgritz lgritz deleted the lg-mac branch January 28, 2021 23:17
@EwoutH
Copy link

EwoutH commented Feb 12, 2021

Is it correct that libheif is still disabled in the CI? Or did I read that incorrect?

@lgritz
Copy link
Collaborator Author

lgritz commented Feb 17, 2021

It's enabled for Mac, disabled for Linux and Windows.

I'll try turning on for Linux again and see what happens.

@lgritz
Copy link
Collaborator Author

lgritz commented Feb 17, 2021

Seems to work, so I will re-enable for several of the Linux CI tests.

#2866

@EwoutH
Copy link

EwoutH commented Feb 25, 2021

Great, thanks! Any chance it could also be turned on for Windows and macOS?

libheif v1.11.0 should have some fixes for macOS.

@lgritz
Copy link
Collaborator Author

lgritz commented Feb 26, 2021

It has always been enabled for MacOS.

Enabling on Windows would require building it from scratch, or having the Windows CI tests download a pre-built version, if that's easily available. It would be great to have a volunteer to do that CI work, I'm really strapped for time at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants