Skip to content

[opensubdiv] Various improvements#24467

Closed
theblackunknown wants to merge 4 commits intomicrosoft:masterfrom
theblackunknown:opensubdiv-opengl
Closed

[opensubdiv] Various improvements#24467
theblackunknown wants to merge 4 commits intomicrosoft:masterfrom
theblackunknown:opensubdiv-opengl

Conversation

@theblackunknown
Copy link
Copy Markdown
Contributor

@theblackunknown theblackunknown commented Apr 28, 2022

Describe the pull request

What does your PR fix?

This PR:

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

all, No

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

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@ghost
Copy link
Copy Markdown

ghost commented Apr 28, 2022

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

❌ theblackunknown sign now
You have signed the CLA already but the status is still pending? Let us recheck it.

@theblackunknown theblackunknown changed the title [opensubdiv] OpenGL backend [opensubdiv] Various improvements Apr 28, 2022
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.

You have modified or added at least one vcpkg.json where you should check the license field.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/opensubdiv/vcpkg.json

Valid values for the license field can be found in the documentation

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.

You have modified or added at least one vcpkg.json where you should check the license field.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/opensubdiv/vcpkg.json

Valid values for the license field can be found in the documentation

@theblackunknown
Copy link
Copy Markdown
Contributor Author

theblackunknown commented Apr 28, 2022

Did something changed recently about the CLA ?
I have signed it in the past and have contributed recently without requiring to sign it again.

Something strange is that it says that the open source projects if affiliated to SAP which I though vcpkg was maintained by microsoft...

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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for opensubdiv but no changes to version or port version.
-- Version: 3.4.3#5
-- Old SHA: c0d413da81ccb867d4498ea4220616a6270d3dab
-- New SHA: 5400fa1040be46057990dba7cb9253b3cb558530
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

You have modified or added at least one vcpkg.json where you should check the license field.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/opensubdiv/vcpkg.json

Valid values for the license field can be found in the documentation

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.

You have modified or added at least one vcpkg.json where you should check the license field.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/opensubdiv/vcpkg.json

Valid values for the license field can be found in the documentation

@Adela0814 Adela0814 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Apr 29, 2022
@theblackunknown
Copy link
Copy Markdown
Contributor Author

@LilyWangL @Adela0814 Thansk for having a look at this PR.
I have reported on the cpp Slack that re-signing the CLA looks strange as I have recently been able to contribute with my previous signed CLA (cf. https://cpplang.slack.com/archives/C7BFF7RCJ/p1651178905353099)
@vicroms recommended to hold any signature until we get more information.

@theblackunknown
Copy link
Copy Markdown
Contributor Author

I am closing this PR due to the strange error about CLA check, I have opened #24615 which supersed this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants