Skip to content

Conversation

pks-gitlab
Copy link

With a recent upgrade to a newer version of MSVC we now get a bunch of warnings when two operands use different enum types. While sensible in theory, in practice we have a couple of non-public enums that extend public enums, like for example with GIT_SUBMODULE_STATUS.

Let's for now disable this warning to unblock our builds. The alternative would be to add casts all over the place, but that feels rather cumbersome.

With a recent upgrade to a newer version of MSVC we now get a bunch of
warnings when two operands use different enum types. While sensible in
theory, in practice we have a couple of non-public enums that extend
public enums, like for example with `GIT_SUBMODULE_STATUS`.

Let's for now disable this warning to unblock our builds. The
alternative would be to add casts all over the place, but that feels
rather cumbersome.
@pks-gitlab pks-gitlab force-pushed the pks-msvc-different-enum-warnings branch from 1dd1fc5 to 5d78d63 Compare August 1, 2025 06:49
@pks-t pks-t requested a review from ethomson August 1, 2025 07:35
@pks-t pks-t mentioned this pull request Aug 1, 2025
@pks-gitlab pks-gitlab mentioned this pull request Aug 4, 2025
Copy link
Member

@pks-t pks-t left a comment

Choose a reason for hiding this comment

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

I'm going to merge this for now to unblock CI pipelines. We can find a more sustainable solution going forward at a later point, if we want to.

@pks-t pks-t merged commit 9d5f1ba into libgit2:main Aug 4, 2025
33 of 34 checks passed
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