Idiomatic Linux active_runtime.json search logic #51
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I'm checking to see if there's any paperwork/admin stuff we need to do to put the loader docs up. I suspect not, since it's not normative. Think I'll just put the source for them in this repo, since they don't really affect the spec itself. (that make sense?) |
daveh-lunarg
approved these changes
Jun 13, 2019
|
@rpavlik, I think we're good to go to push this - any objection? |
Windows selects a runtime based on the registry, not a fixed path.
|
Updated to avoid assigning a null |
|
Hearing no dissent, pushing this. |
rpavlik
added a commit
that referenced
this pull request
Aug 14, 2020
This release is mainly for SDK improvements, with only small changes to
the docs. A new error code is provided for xrCreateSession for
developers convenience.
- Registry
- Register ULTRALEAP author ID for Ultraleap. (internal MR 1877)
- Reserve the extension number 98 to 101 for future MSFT
extensions. (internal MR 1879)
- schema: Distinguish parentstruct and structextends attributes in
comments. (internal MR 1881, OpenXR-Docs/#51, internal issue
1396)
- Add a new result code,
XR_ERROR_GRAPHICS_REQUIREMENTS_CALL_MISSING, for runtimes to
return if xrBeginSession is called before calling one of the
xrGetGraphicsRequirements calls. (internal MR 1882,
OpenXR-Docs/#53, internal issue 1397)
- SDK
- Improve language usage in code and comments to be more
respectful. (internal MR 1881)
- Loader: Correct type of “extension_version” in API layer
manifest files to string, while maintaining backwards
compatibility. Remove undocumented and unused
“device_extensions” and “entrypoints” keys. (internal MR 1867,
internal issue 1411)
- Replace usage of std::filesystem::canonical with
PathCchCanonicalize on Windows platform to work around bug on
UWP platforms. This also replaces PathCanonicalize with
PathCchCanonicalize and adds the appropriate library for linking
in. (OpenXR-SDK-Source/#198)
- Support for building more projects when targeting UWP, and
support for all architectures when targeting Win32.
(OpenXR-SDK-Source/#199)
- hello_xr: fix Vulkan image layout transitions. (internal MR
1876)
- validation: Enable three additional checks (on optional arrays
with non- optional counts) that were missing because of a script
error. (internal MR 1881)
rhabacker
pushed a commit
to rhabacker/OpenXR-SDK-Source
that referenced
this pull request
Nov 16, 2022
This release is mainly for SDK improvements, with only small changes to
the docs. A new error code is provided for xrCreateSession for
developers convenience.
- Registry
- Register ULTRALEAP author ID for Ultraleap. (internal MR 1877)
- Reserve the extension number 98 to 101 for future MSFT
extensions. (internal MR 1879)
- schema: Distinguish parentstruct and structextends attributes in
comments. (internal MR 1881, OpenXR-Docs/KhronosGroup#51, internal issue
1396)
- Add a new result code,
XR_ERROR_GRAPHICS_REQUIREMENTS_CALL_MISSING, for runtimes to
return if xrBeginSession is called before calling one of the
xrGetGraphicsRequirements calls. (internal MR 1882,
OpenXR-Docs/KhronosGroup#53, internal issue 1397)
- SDK
- Improve language usage in code and comments to be more
respectful. (internal MR 1881)
- Loader: Correct type of “extension_version” in API layer
manifest files to string, while maintaining backwards
compatibility. Remove undocumented and unused
“device_extensions” and “entrypoints” keys. (internal MR 1867,
internal issue 1411)
- Replace usage of std::filesystem::canonical with
PathCchCanonicalize on Windows platform to work around bug on
UWP platforms. This also replaces PathCanonicalize with
PathCchCanonicalize and adds the appropriate library for linking
in. (OpenXR-SDK-Source/KhronosGroup#198)
- Support for building more projects when targeting UWP, and
support for all architectures when targeting Win32.
(OpenXR-SDK-Source/KhronosGroup#199)
- hello_xr: fix Vulkan image layout transitions. (internal MR
1876)
- validation: Enable three additional checks (on optional arrays
with non- optional counts) that were missing because of a script
error. (internal MR 1881)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Based on #38. Fixes #28.
This updates the search logic to find the first active_runtime.json in standard config file locations, allowing for more graceful integration with Linux distribution tooling and user/administrator expectations.
I opted not to reuse
ReadDataFilesInSearchPathsdue to its complexity and conflation of config/data search paths.If this is to be merged, the loader design doc will also need to be updated. I'm not sure where to find the source files for that.