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

[opengl-registry/egl-registry] update #36856

Merged
merged 3 commits into from
Feb 21, 2024
Merged

Conversation

moritz-h
Copy link
Contributor

  • 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.
  • 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.

This PR updates the opengl-registry and egl-registry ports to the latest version. In addition both ports have broken copyright information as both repositories are using individual licenses for each file. I tried to fix this by including a vendored copyright file for the egl-registry port. For opengl-registry I have just included a note that all files are individually licensed. A proper fix for this seems to be already worked on in #28644.

@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Feb 20, 2024
@@ -0,0 +1,28 @@
## include/KHR/khrplatform.h
Copy link
Member

Choose a reason for hiding this comment

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

@moritz-h, Thanks for your PR, do the two copyrights you added exist in the upstream source code? We do not recommend users to add copyright that does not exist in the upstream.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The copyright text is directly copied from https://github.com/KhronosGroup/EGL-Registry/blob/main/api/KHR/khrplatform.h
The file paths I used as heading are from perspective of the installed port not in the source repo.

I know putting the copyright here is not a clean solution, but solving this for these repos is probably more complicated and worked on in #28644. So my goal was just update the ports and maybe make the copyright a little less broken than before, but I would still see this as workaround until the other PR is ready.

@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Feb 21, 2024
@vicroms vicroms merged commit d0d63ec into microsoft:master Feb 21, 2024
16 checks passed
@moritz-h moritz-h deleted the opengl branch February 22, 2024 12:28
TomKatom pushed a commit to TomKatom/vcpkg that referenced this pull request Feb 23, 2024
- [x] Changes comply with the [maintainer
guide](https://github.com/microsoft/vcpkg-docs/blob/main/vcpkg/contributing/maintainer-guide.md).
- [x] SHA512s are updated for each updated download.
- [x] The "supports" clause reflects platforms that may be fixed by this
new version.
- [x] Any fixed [CI
baseline](https://github.com/microsoft/vcpkg/blob/master/scripts/ci.baseline.txt)
entries are removed from that file.
- [x] Any patches that are no longer applied are deleted from the port's
directory.
- [x] The version database is fixed by rerunning `./vcpkg x-add-version
--all` and committing the result.
- [x] Only one version is added to each modified port's versions file.

This PR updates the `opengl-registry` and `egl-registry` ports to the
latest version. In addition both ports have broken copyright information
as both repositories are using individual licenses for each file. I
tried to fix this by including a vendored copyright file for the
`egl-registry` port. For `opengl-registry` I have just included a note
that all files are individually licensed. A proper fix for this seems to
be already worked on in microsoft#28644.
Osyotr pushed a commit to Osyotr/vcpkg that referenced this pull request Feb 26, 2024
- [x] Changes comply with the [maintainer
guide](https://github.com/microsoft/vcpkg-docs/blob/main/vcpkg/contributing/maintainer-guide.md).
- [x] SHA512s are updated for each updated download.
- [x] The "supports" clause reflects platforms that may be fixed by this
new version.
- [x] Any fixed [CI
baseline](https://github.com/microsoft/vcpkg/blob/master/scripts/ci.baseline.txt)
entries are removed from that file.
- [x] Any patches that are no longer applied are deleted from the port's
directory.
- [x] The version database is fixed by rerunning `./vcpkg x-add-version
--all` and committing the result.
- [x] Only one version is added to each modified port's versions file.

This PR updates the `opengl-registry` and `egl-registry` ports to the
latest version. In addition both ports have broken copyright information
as both repositories are using individual licenses for each file. I
tried to fix this by including a vendored copyright file for the
`egl-registry` port. For `opengl-registry` I have just included a note
that all files are individually licensed. A proper fix for this seems to
be already worked on in microsoft#28644.
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.

3 participants