Update sharp to 0.33.4, include av1 support for encoding and decoding by faradaytrs · Pull Request #33 · zoellner/sharp-heic-lambda-layer · GitHub
Skip to content

Update sharp to 0.33.4, include av1 support for encoding and decoding#33

Merged
zoellner merged 6 commits into
zoellner:mainfrom
faradaytrs:main
Aug 19, 2024
Merged

Update sharp to 0.33.4, include av1 support for encoding and decoding#33
zoellner merged 6 commits into
zoellner:mainfrom
faradaytrs:main

Conversation

@faradaytrs

Copy link
Copy Markdown
Contributor

Not sure if it's what you want, but it works with avif and works fine.
I also updated sharp, if you don't want it then i can split PRs.

@zoellner

Copy link
Copy Markdown
Owner

@chriswritests

Copy link
Copy Markdown

I'm also getting the same error as @zoellner

/usr/local/include/vips/vips8:35:10: fatal error: glib-object.h: No such file or directory

Are you using aws/codebuild/amazonlinux2-x86_64-standard:5.0 @faradaytrs?

@faradaytrs

Copy link
Copy Markdown
Contributor Author

I'm also getting the same error as @zoellner

/usr/local/include/vips/vips8:35:10: fatal error: glib-object.h: No such file or directory

Are you using aws/codebuild/amazonlinux2-x86_64-standard:5.0 @faradaytrs?

Yes I am, there was a problem with SHARP_FORCE_GLOBAL_LIBVIPS, I deleted it, and it works fine now. Please try again.

@faradaytrs faradaytrs force-pushed the main branch 2 times, most recently from 4844079 to 0116ebc Compare July 12, 2024 03:22
@faradaytrs

faradaytrs commented Jul 12, 2024

Copy link
Copy Markdown
Contributor Author

There is also an update for libheif, if you plan to release a new version, maybe it's a good idea to include it too.

@chriswritests

Copy link
Copy Markdown

I can confirm that the changes work perfectly fine in production. @faradaytrs, thank you for proposing these improvements. We may need to update the README to reflect these changes.

@zoellner, thank you for putting this repository together. It is a massive timesaver!

@zoellner

Copy link
Copy Markdown
Owner

@zoellner zoellner merged commit 0f2eb54 into zoellner:main Aug 19, 2024
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.

3 participants