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.

Sign up for GitHub

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

Jump to bottom

[CGAL] Upgrade to 5.3 #18392

Merged
merged 13 commits into from Jul 26, 2021
Merged

[CGAL] Upgrade to 5.3 #18392

merged 13 commits into from Jul 26, 2021

Conversation

maxGimeno
Copy link
Contributor

Upgrade CGAL to version 5.3

@maxGimeno maxGimeno mentioned this pull request Jun 11, 2021
Closed
70 tasks
@JonLiu1993 JonLiu1993 self-assigned this Jun 15, 2021
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Jun 15, 2021
@JonLiu1993
Copy link
Member

@maxGimeno ,Thanks for your contribution.
upgrade to the 5.3-beta1 pre-release version, our policy is not recommended, because the pre-release version is unstable and there may be many problems

@maxGimeno
Copy link
Contributor Author

@JonLiu1993 don't worry, it will stay a draft until the release of the CGAL 5.3 version, then I'll update the PR to the stable version and un-draft it. For now it was just to be sure that in the current state we can pass your CI.

@maxGimeno maxGimeno marked this pull request as ready for review July 6, 2021 13:37
@JonLiu1993 JonLiu1993 changed the title CGAL: Upgrade to 5.3 [CGAL] Upgrade to 5.3 Jul 7, 2021
JonLiu1993
JonLiu1993 requested changes Jul 7, 2021
View reviewed changes
Copy link
Member

@JonLiu1993 JonLiu1993 left a comment

Choose a reason for hiding this comment

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

Please add 'FEATURES' in function vcpkg_check_features like:

vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
    FEATURES
    qt WITH_CGAL_Qt5
)

@JonLiu1993
Copy link
Member

@maxGimeno ,After completing the modification, please run the command "./vcpkg x-add-version cgal --overwrite-version", and then submit the changes again.

@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Jul 8, 2021
ras0219-msft
ras0219-msft approved these changes Jul 8, 2021
View reviewed changes
Copy link
Contributor

@ras0219-msft ras0219-msft left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

@JonLiu1993 JonLiu1993 removed the info:reviewed Pull Request changes follow basic guidelines label Jul 12, 2021
@JonLiu1993
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PhoebeHui PhoebeHui added the info:reviewed Pull Request changes follow basic guidelines label Jul 16, 2021
vicroms
vicroms approved these changes Jul 23, 2021
View reviewed changes
Copy link
Member

@vicroms vicroms left a comment

Choose a reason for hiding this comment

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

Please resolve merge conglicts, otherwise LGTM.

JonLiu1993
JonLiu1993 requested changes Jul 23, 2021
View reviewed changes
versions/c-/cgal.json Outdated Show resolved Hide resolved
@JonLiu1993 JonLiu1993 added requires:author-response and removed info:reviewed Pull Request changes follow basic guidelines labels Jul 23, 2021
@JonLiu1993 JonLiu1993 added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Jul 26, 2021
@vicroms vicroms merged commit f232529 into microsoft:master Jul 26, 2021
@lrineau lrineau deleted the CGAL-Upgrade_to_5.3-maxGimeno branch July 27, 2021 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vicroms vicroms vicroms approved these changes

@ras0219-msft ras0219-msft ras0219-msft approved these changes

@PhoebeHui PhoebeHui PhoebeHui approved these changes

@JonLiu1993 JonLiu1993 JonLiu1993 approved these changes

Assignees

@JonLiu1993 JonLiu1993

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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@maxGimeno @JonLiu1993 @vicroms @ras0219-msft @PhoebeHui @BillyONeal