Skip to content

[mimalloc] Install minject tool on windows platform#51244

Merged
BillyONeal merged 2 commits into
microsoft:masterfrom
RT2Code:mimalloc
Apr 21, 2026
Merged

[mimalloc] Install minject tool on windows platform#51244
BillyONeal merged 2 commits into
microsoft:masterfrom
RT2Code:mimalloc

Conversation

@RT2Code

@RT2Code RT2Code commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

Add installation of the minject tool on Windows. The minject executables are provided by mimalloc and can be used to injects the mimalloc dll and ensures that it comes first in the import table.

  • 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.

@BillyONeal BillyONeal left a comment

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.

Thanks!

vcpkg_cmake_config_fixup(CONFIG_PATH lib/cmake/mimalloc)

if(VCPKG_TARGET_IS_WINDOWS)
file(INSTALL

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.

Q: Should this use vcpkg_copy_tools?
A: No, that's relative to SOURCE_PATH not CURRENT_PACKAGES_DIR.

No change requested.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

minject is a vendored closed-source static executable.
vcpkg_copy_tools can handle other locations, but it is pointless for static tools.

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.

Yeah that's the conclusion I came to but because I had the question I figured I should have it on record that I considered it before pushing the merge button.

@BillyONeal BillyONeal merged commit 590594f into microsoft:master Apr 21, 2026
16 checks passed
@RT2Code RT2Code deleted the mimalloc branch April 21, 2026 15:40
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