Skip to content

[cgal] Update to 5.4#22308

Merged
ras0219-msft merged 8 commits intomicrosoft:masterfrom
lrineau:cgal-5.4
Feb 8, 2022
Merged

[cgal] Update to 5.4#22308
ras0219-msft merged 8 commits intomicrosoft:masterfrom
lrineau:cgal-5.4

Conversation

@lrineau
Copy link
Copy Markdown
Contributor

@lrineau lrineau commented Jan 3, 2022

  • What does your PR fix?

    Update to CGAL-5.4.

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

    all

  • 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, I did.

@JonLiu1993 JonLiu1993 self-assigned this Jan 4, 2022
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Jan 4, 2022
@lrineau lrineau mentioned this pull request Jan 4, 2022
96 tasks
@lrineau lrineau changed the title (WIP) [cgal] Update to 5.4-beta1 [cgal] Update to 5.4 Feb 1, 2022
@lrineau lrineau marked this pull request as ready for review February 1, 2022 10:21
Copy link
Copy Markdown

@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 2b543e52da6a7a595760f10c1160e808d89c41ed -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/c-/cgal.json b/versions/c-/cgal.json
index 7cc6c29..e1dc39a 100644
--- a/versions/c-/cgal.json
+++ b/versions/c-/cgal.json
@@ -5,11 +5,6 @@
       "version": "5.4",
       "port-version": 0
     },
-    {
-      "git-tree": "e9d6d93929952a260387c0604db0bc72473ff795",
-      "version-semver": "5.4.0-beta1",
-      "port-version": 0
-    },
     {
       "git-tree": "da69aec4c03275e85108ebc2dc2d512057d6ac32",
       "version": "5.3.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/cgal/vcpkg.json

Valid values for the license field are listed at https://spdx.org/licenses/

That is required by the CI.
That is required by the CI.
Copy link
Copy Markdown

@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 2b543e52da6a7a595760f10c1160e808d89c41ed -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/c-/cgal.json b/versions/c-/cgal.json
index e1dc39a..13b6b72 100644
--- a/versions/c-/cgal.json
+++ b/versions/c-/cgal.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "f4443135f45633a2c024df0d4218b8f86db35dcd",
+      "git-tree": "0acdd574cd028b7ca4acd762f01938492c6d4719",
       "version": "5.4",
       "port-version": 0
     },

@lrineau
Copy link
Copy Markdown
Contributor Author

lrineau commented Feb 3, 2022

@JonLiu1993 Now that CGAL-5.4 was published, this PR has been updated, and the CI seems fine with it.

@JonLiu1993
Copy link
Copy Markdown
Contributor

All features are tested successfully in the following triplet:

  • x86-windows
  • x64-windows
  • x64-windows-static

@JonLiu1993 JonLiu1993 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Feb 8, 2022
@ras0219-msft ras0219-msft merged commit f4ebf07 into microsoft:master Feb 8, 2022
@ras0219-msft
Copy link
Copy Markdown
Contributor

Thanks for the update!

@lrineau lrineau deleted the cgal-5.4 branch February 8, 2022 17:16
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:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants