-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[libjxl] update to 0.8.0 #29171
[libjxl] update to 0.8.0 #29171
Conversation
d44cd75
to
d568c3e
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.
Thanks for the update!
--- a/third_party/CMakeLists.txt | ||
+++ b/third_party/CMakeLists.txt | ||
@@ -45,36 +45,7 @@ else() | ||
@@ -45,59 +45,7 @@ else() | ||
endif() | ||
|
||
# brotli |
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.
I'm going to merge this as is because it has already passed testing but in the future it might be a good idea to disable this block with an if(0)
rather than deleting the whole content so that we aren't including so much of upstream's code in the registry.
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.
I was about to add a more detailed comment...
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.
Do you need this backed off?
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.
No, there is no regression. But I will check the port and its patches.
./vcpkg x-add-version --all
and committing the result.