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

[many ports] Remove from CI baseline #25075

Merged
merged 42 commits into from
Jun 17, 2022

Conversation

Thomas1664
Copy link
Contributor

Describe the pull request
I removed several ports from CI baseline and updated their supports instead.

Copy link

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

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

  • ports/devicenameresolver/vcpkg.json

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

If you feel able to do so, please consider replacing the deprecated license identifiers in the following files:

  • ports/fann/vcpkg.json (has deprecated license LGPL-2.1)

Deprecated and non deprecated license identifiers can be found here

ports/devicenameresolver/vcpkg.json Outdated Show resolved Hide resolved
@@ -248,8 +239,6 @@ embree2:x64-linux=skip
embree2:x64-osx=skip
epsilon:arm-uwp=fail
epsilon:x64-uwp=fail
epsilon:x64-windows-static=fail
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know why UWP fails

Copy link
Contributor

Choose a reason for hiding this comment

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

If it is a fail, it is still build by daily CI, so you can get the error log from there.

ports/epsilon/vcpkg.json Outdated Show resolved Hide resolved
ports/fann/vcpkg.json Outdated Show resolved Hide resolved
ports/cppcms/vcpkg.json Outdated Show resolved Hide resolved
ports/cudnn/vcpkg.json Outdated Show resolved Hide resolved
Copy link

@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: checked-in files for epsilon have changed but the version was not updated
version: 0.9.2#7
old SHA: 770bd5c543bccf17c46b5d44fd288128008bde27
new SHA: ae6c5ff3c482e36a74334087042020d74c508181
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
error: checked-in files for fann have changed but the version was not updated
version: 2.2.0#3
old SHA: aa16960a4d6d8e991d5be39ba7103174e0044a99
new SHA: a18c3fc66eb4f234a4435a41d7628bc571c14710
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
error: checked-in files for cudnn have changed but the version was not updated
version: 7.6.5#7
old SHA: 3f520a4807c2a25084b2febff6821bceaf1b8091
new SHA: eac69210d17281676ac80c5f262e674e49dcd3a3
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
error: checked-in files for cppcms have changed but the version was not updated
version: 1.2.1#5
old SHA: 3e8c1d684741e878e07297a9bf008f3b77bcfaa3
new SHA: 22f52d82f20d7d67fc2e4714df718fa363e12bed
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***

@Thomas1664 Thomas1664 marked this pull request as draft June 6, 2022 09:46
@Thomas1664 Thomas1664 marked this pull request as ready for review June 8, 2022 20:27
@Thomas1664 Thomas1664 requested a review from JackBoosY June 9, 2022 07:06
@Thomas1664
Copy link
Contributor Author

@JackBoosY ready for review now

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Jun 17, 2022
@dan-shaw dan-shaw merged commit b49b2bd into microsoft:master Jun 17, 2022
@Thomas1664 Thomas1664 deleted the update-supports branch June 18, 2022 05:29
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 info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants