Skip to content

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

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -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.

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:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. 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: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.

4 participants