-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
[libjpeg-turbo] fix incompatibility for c11 and c17 #15234
[libjpeg-turbo] fix incompatibility for c11 and c17 #15234
Conversation
Could you please resolve the conflicts? |
Looks like you missed something in latest commit. The CONTROL and portfile.cmake files are also needed here. |
docs/maintainers/maintainer-guide.md should not be included in this PR. |
Alright, It seems that I have touched it accidently and fixed it. I'll redo the PR. |
Could you please resolve the conflicts? |
The CONTROL file is not updated in this PR. Please bump port-version. |
It's already |
We should update it as 2. Please refer to this doc https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/control-files.md#port-version |
As in the doc, |
Port-Version has been updated to 1 in vcpkg master branch. https://github.com/microsoft/vcpkg/blob/master/ports/libjpeg-turbo/CONTROL#L3 So it should be 2 in this PR. If there is no Port-Version in your branch, you should merge master to your branch, then update it. |
…github/vcpkg into fix-incompatibility-for-c11-and-c17
LGTM now except for the features. Have you tested these features in this PR? Can they work fine? |
No, I have only Windows on my machine, it depends on the checks here. |
All features have passed with the following triplets:
|
Thanks :) |
Describe the pull request
#15195