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

[libwebp] Update, cleanup #23864

Merged
merged 21 commits into from
Apr 13, 2022
Merged

[libwebp] Update, cleanup #23864

merged 21 commits into from
Apr 13, 2022

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Mar 30, 2022

  • What does your PR fix?

    Updates libwebp to 1.2.2.
    Cleanup of features, portfile, patches.
    Drop non-standard debug postfix.
    Drop non-standard config polyfill.

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    unchanged, no

  • Does your PR follow the maintainer guide?

    yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    yes

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 4f778cf094a65f836167eb301bdf62b7ba632c68 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 851d4de..723f144 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -4141,7 +4141,7 @@
       "port-version": 1
     },
     "libwebp": {
-      "baseline": "1.2.1",
+      "baseline": "1.2.2",
       "port-version": 0
     },
     "libwebsockets": {
diff --git a/versions/l-/libwebp.json b/versions/l-/libwebp.json
index 935958c..a188e24 100644
--- a/versions/l-/libwebp.json
+++ b/versions/l-/libwebp.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "d09abc022500076444cb82cb2cc0a90a1c1668f3",
+      "version": "1.2.2",
+      "port-version": 0
+    },
     {
       "git-tree": "ca5a8b2ed264617e8a35d22e1e6ac18ebe99900f",
       "version": "1.2.1",

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/libwebp/vcpkg.json

Valid values for the license field can be found in the documentation

@LilyWangLL LilyWangLL added the category:port-update The issue is with a library, which is requesting update new revision label Mar 30, 2022
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/libwebp/vcpkg.json
  • ports/osgearth/vcpkg.json
  • ports/skia/vcpkg.json

Valid values for the license field can be found in the documentation

@dg0yt dg0yt marked this pull request as ready for review April 7, 2022 07:23
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/libwebp/vcpkg.json
  • ports/osgearth/vcpkg.json
  • ports/skia/vcpkg.json

Valid values for the license field can be found in the documentation

@LilyWangLL LilyWangLL requested a review from Cheney-W April 12, 2022 08:59
@LilyWangLL LilyWangLL added info:reviewed Pull Request changes follow basic guidelines requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function and removed info:reviewed Pull Request changes follow basic guidelines labels Apr 13, 2022
@Cheney-W
Copy link
Contributor

All features passed with x64-windows triplet.

@Cheney-W Cheney-W added info:reviewed Pull Request changes follow basic guidelines and removed requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function labels Apr 13, 2022
@strega-nil-ms
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants