Skip to content

[lunasvg] Update 2.3.8 #32510

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

Merged
merged 7 commits into from
Jul 12, 2023
Merged

[lunasvg] Update 2.3.8 #32510

merged 7 commits into from
Jul 12, 2023

Conversation

sakiodre
Copy link
Contributor

@sakiodre sakiodre commented Jul 11, 2023

  • Changes comply with the maintainer guide
  • SHA512s are updated for each updated download
  • The "supports" clause reflects platforms that may be fixed by this new version (not sure what this means)
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@sakiodre
Copy link
Contributor Author

@microsoft-github-policy-service agree

@@ -24,16 +24,5 @@ if(LUNASVG_BUILD_EXAMPLES)
@@ -23,16 +23,5 @@ if(LUNASVG_BUILD_EXAMPLES)
Copy link
Contributor

Choose a reason for hiding this comment

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

If there is no other change, maybe leave the patch unchanged?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My mistake, I thought since the CMakelist.txt has changed, the patch also needed to be updated

Copy link
Member

Choose a reason for hiding this comment

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

The patch needs to be changed if and only if the old patch doesn't apply cleanly.

@@ -1,8 +1,8 @@
vcpkg_from_github(
OUT_SOURCE_PATH SOURCE_PATH
REPO sammycage/lunasvg
REF e612abda858b53160041381a23422cd2b4f42fbd #2.3.1
SHA512 44f5d013d918cb5af90114a12857bdd2c204caff761516ef98b12b08d8b6215e91f6d963c281500c386f287b9d0ecd5b3d986d4c8c33423c0c34d539d744e09d
REF 87137e791c912432b93982722a8e965628950ca7 #2.3.8
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
REF 87137e791c912432b93982722a8e965628950ca7 #2.3.8
REF "v${VERSION}

... which will have a different SHA512.

BillyONeal
BillyONeal previously approved these changes Jul 11, 2023
@FrankXie05 FrankXie05 added the category:port-update The issue is with a library, which is requesting update new revision label Jul 12, 2023
FrankXie05
FrankXie05 previously approved these changes Jul 12, 2023
@FrankXie05 FrankXie05 added the info:reviewed Pull Request changes follow basic guidelines label Jul 12, 2023
@sakiodre sakiodre dismissed stale reviews from FrankXie05 and BillyONeal via b0a9620 July 12, 2023 02:20
Co-authored-by: Frank <65999885+FrankXie05@users.noreply.github.com>
@FrankXie05
Copy link
Contributor

Every change requires running the command ./vcpkg x-add-version lunasvg --overwrite-version. :)

@BillyONeal BillyONeal merged commit 5269f4f into microsoft:master Jul 12, 2023
@BillyONeal
Copy link
Member

Thanks!

@JonLiu1993 JonLiu1993 mentioned this pull request Jul 14, 2023
6 tasks
@sakiodre sakiodre deleted the master branch July 15, 2023 08:29
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