Skip to content

[MrBot] have submodules#46

Merged
anporumb merged 4 commits intomasterfrom
anporumb/have_libfmt_as_submodule
Jan 23, 2026
Merged

[MrBot] have submodules#46
anporumb merged 4 commits intomasterfrom
anporumb/have_libfmt_as_submodule

Conversation

@anporumb
Copy link

No description provided.

- Remove lib/cppformat directory containing copied fmt 12.1.1 files
- Add fmt library as git submodule at lib/cppformat
- Submodule points to https://github.com/fmtlib/fmt.git at tag 12.1.0
- Remove lib/gtest directory containing copied googletest files
- Add googletest as git submodule at lib/gtest (v1.17.0)
- Submodule points to https://github.com/google/googletest.git
- Update include path in src/tests/CMakeLists.txt for new directory structure
The fmt library triggers C4244 (64-bit to 32-bit conversion) warnings
on x86 builds which are treated as errors. Disable this warning for
the vld target on x86 only.
@anporumb
Copy link
Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@anporumb
Copy link
Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@anporumb anporumb marked this pull request as ready for review January 23, 2026 02:46
@anporumb anporumb enabled auto-merge (squash) January 23, 2026 02:46
@anporumb anporumb changed the title have submodules [MrBot] have submodules Jan 23, 2026
Copy link
Member

@parth21999 parth21999 left a comment

Choose a reason for hiding this comment

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

:shipit:

@anporumb anporumb merged commit 772e027 into master Jan 23, 2026
8 checks passed
@anporumb anporumb deleted the anporumb/have_libfmt_as_submodule branch January 26, 2026 17:57
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.

2 participants