-
Notifications
You must be signed in to change notification settings - Fork 852
Update versioning for SM 6.10 2605 preview release #8395
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
Changes from 1 commit
0dd7d61
54c62d7
2276cf5
e3f2ffb
75fa7ad
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| { | ||
| "version": { | ||
| "major": "1", | ||
| "minor": "9", | ||
| "rev": "2602" | ||
| "minor": "10", | ||
| "rev": "2606" | ||
|
tex3d marked this conversation as resolved.
Outdated
|
||
| }, | ||
| "sha": "f255809c3dd408c32349d6b7b16019b8b24e1d2e" | ||
| "sha": "c763461468246fb779fb234b08ad3b460fb93a02" | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,12 +13,12 @@ | |
| #ifdef RC_VERSION_FIELD_2 | ||
| #undef RC_VERSION_FIELD_2 | ||
| #endif | ||
| #define RC_VERSION_FIELD_2 9 | ||
| #define RC_VERSION_FIELD_2 10 | ||
|
|
||
| #ifdef RC_VERSION_FIELD_3 | ||
| #undef RC_VERSION_FIELD_3 | ||
| #endif | ||
| #define RC_VERSION_FIELD_3 2602 | ||
| #define RC_VERSION_FIELD_3 2605 | ||
|
|
||
| #ifdef RC_VERSION_FIELD_4 | ||
| #undef RC_VERSION_FIELD_4 | ||
|
|
@@ -28,7 +28,7 @@ | |
| #ifdef RC_FILE_VERSION | ||
| #undef RC_FILE_VERSION | ||
| #endif | ||
| #define RC_FILE_VERSION "1.9.2602.0" | ||
| #define RC_FILE_VERSION "1.10.2605.0" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. pretty sure this it intended but calling out that its showing 1.10 just in case it wasn't
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, good point. I'm not sure how extensive the impact of the json version is, so I should update these back to 9 for consistency.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hmm, but I'm not sure I want to change the binary file versions this way...
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We used to update the version in the json for the preview, but that was before we drove a
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ok, my latest push adds a preview_minor to the version info in latest-release.json to support overriding the minor version in gen_version.py for preview releases without disturbing the highest released DXIL version.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We could likely merge such a change to main and extend this mechanism to avoid manually updating the DXIL version in hctdb_instrhelp.py separately. The
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yeah, I don't think we do that here but getting that into main would be a nice QOL change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What I said here wouldn't work properly for main yet, but we could think about what would work and adjust accordingly. |
||
|
|
||
| #ifdef RC_FILE_DESCRIPTION | ||
| #undef RC_FILE_DESCRIPTION | ||
|
|
@@ -49,7 +49,7 @@ | |
| #ifdef RC_PRODUCT_VERSION | ||
| #undef RC_PRODUCT_VERSION | ||
| #endif | ||
| #define RC_PRODUCT_VERSION "1.9.2602.0" | ||
| #define RC_PRODUCT_VERSION "1.10.2605.0" | ||
|
|
||
| #ifdef HLSL_TOOL_NAME | ||
| #undef HLSL_TOOL_NAME | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.