Skip to content

Update imgui to v1.92.9 - #53262

Closed
LEE KYOUNGHEON (stripe2933) wants to merge 1 commit into
microsoft:masterfrom
stripe2933:master
Closed

Update imgui to v1.92.9#53262
LEE KYOUNGHEON (stripe2933) wants to merge 1 commit into
microsoft:masterfrom
stripe2933:master

Conversation

@stripe2933

@stripe2933 LEE KYOUNGHEON (stripe2933) commented Aug 6, 2026

Copy link
Copy Markdown
Contributor
  • Changes comply with the maintainer guide.

  • SHA512s are updated for each updated download.

  • The "supports" clause reflects platforms that may be fixed by this new version, or no changes were necessary.

  • Any fixed CI baseline and CI feature baseline entries are removed from that file, or no entries needed to be changed.

  • All patch files in the port are applied and succeed.

  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.

  • Exactly one version is added in each modified versions file.

  • Added metal4 feature that enable Metal 4 backend in macOS.

  • Added default-font-bitmap and default-font-vector features that embed the font data in the library, which are enabled by default.

Comment thread ports/imgui/vcpkg.json
Comment on lines +18 to +19
"default-font-bitmap",
"default-font-vector"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ocornut/imgui#9407

Enabling the features does not add API. Most of the end users expect the library uses its own embedded font by default. Some end users will manually disable these features and expect the behavior difference (application must add font, or it won't startup).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The PR there suggests that it effectively fails the requirement because:

AddFontDefaultXXX() functions will assert.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think we should have these features. They comply with the letter but not the spirit of the rule. Users that want to disable the default embedded fonts should write an overlay port.

Comment thread ports/imgui/vcpkg.json Outdated
@BillyONeal

Copy link
Copy Markdown
Member

Should we skip right to https://github.com/ocornut/imgui/releases/tag/v1.92.9b which claims to hotfix regressions?

@stripe2933

Copy link
Copy Markdown
Contributor Author

I will close this PR and create a new PR using v1.92.9b that does not include font-related updates.

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.

3 participants