Skip to content

Bump default version number.#1161

Merged
MarkCallow merged 9 commits intomainfrom
issue1112
Apr 24, 2026
Merged

Bump default version number.#1161
MarkCallow merged 9 commits intomainfrom
issue1112

Conversation

@MarkCallow
Copy link
Copy Markdown
Collaborator

@MarkCallow MarkCallow commented Apr 23, 2026

Various other issues were encountered and fixed while checking the version number related changes.

  1. Fix -a option of mkversion. It did not visit lib/src and ended with an error due to a misplaced fi.
  2. Fix RE in ktx convert that resulted in an extra space in an edited KTXwriterScParams when the original zcmp option had a parameter.
  3. Fix ktxTexture2_IsHDR which was only checking for a FLOAT qualifier on DFD samples for the ASTC color model. Add a test for it.

Fixes #1112.

* Move misplaced `fi` to correct place and fix indentation.
* Add lib/src to target directories.
that resulted an extra space in edited KTXwriterScParams.
Check all color models, not just ASTC, for FLOAT qualifier.
Update to test_decode_astc() due to input file changing to SFLOAT after
regeneration.
@MarkCallow MarkCallow merged commit 6c474d8 into main Apr 24, 2026
41 checks passed
@MarkCallow MarkCallow deleted the issue1112 branch April 24, 2026 02:17
richgel999 added a commit to BinomialLLC/KTX-Software-Binomial-Fork that referenced this pull request Apr 25, 2026
Various other issues were encountered and fixed while checking the
version number related changes.

1. Fix `-a` option of `mkversion`. It did not visit `lib/src` and ended
   with an error due to a misplaced `fi`.
2. Fix RE in _ktx convert_ that resulted in an extra space in an edited
   KTXwriterScParams when the original `zcmp` option had a parameter.
3. Fix `ktxTexture2_IsHDR` which was only checking for a FLOAT qualifier
   on DFD samples for the ASTC color model. Add a test for it.

Fixes KhronosGroup#1112.
# Conflicts:
#	interface/python_binding/tests/test_ktx_texture2.py
#	tests/resources/ktx2/Desk_small_zstd_15.ktx2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update default version when major version is updated

1 participant