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

openvr: Bump deps and upgrade for conan v2 #16913

Closed
wants to merge 4 commits into from

Conversation

paulharris
Copy link
Contributor

Specify library name and version: lib/1.0


@paulharris paulharris changed the title openvr: Bump deps openvr: Bump deps and upgrade for conan v2 Apr 6, 2023
@conan-center-bot

This comment has been minimized.

@paulharris
Copy link
Contributor Author

Seems like a pretty cryptic error...
On Linux locally, I get this error from the test_package:

Unable to read VR Path Registry from /home/paul/.config/openvr/openvrpaths.vrpath

Which I assume is enough to test the library was able to link and execute...

@conan-center-bot

This comment has been minimized.

Comment on lines 35 to 43
@property
def _source_subfolder(self):
return "source_subfolder"
def _compilers_minimum_version(self):
return {
"Visual Studio": "15",
"msvc": "14.1",
"gcc": "5",
"clang": "5",
"apple-clang": "5.1",
}
Copy link
Contributor

@SpaceIm SpaceIm May 7, 2023

Choose a reason for hiding this comment

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

Besides wrong msvc version, why do you restrict more than before? It was raising for gcc < 5 only, so we know it works for Visual Studio 2015 for example. You should keep previous logic.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok then, I've reverted :)

@conan-center-bot

This comment has been minimized.

@CLAassistant
Copy link

CLAassistant commented May 18, 2023

CLA assistant check
All committers have signed the CLA.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

paulharris and others added 2 commits July 8, 2023 23:20
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
@conan-center-bot

This comment has been minimized.

@ghost
Copy link

ghost commented Jul 22, 2023

I detected other pull requests that are modifying openvr/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Sorry, the system is under maintenance and it doesn't accept builds right now.
Please, check https://status.conan.io to obtain more information.
Thanks for your understanding and help with the Conan Center Index!


Conan v2 pipeline ❌

Note: Conan v2 builds may be required once they are on the v2 ready list

The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future.

See details:

Sorry, the system is under maintenance and it doesn't accept builds right now.
Please, check https://status.conan.io to obtain more information.
Thanks for your understanding and help with the Conan Center Index!

@stale
Copy link

stale bot commented Oct 15, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@paulharris
Copy link
Contributor Author

I'm happy to close this in preference for #18788, which appears to supersede this PR.

@paulharris paulharris closed this Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants