Skip to content

[gettext] Fix wrong variable type#49860

Merged
BillyONeal merged 3 commits intomicrosoft:masterfrom
bansan85:40134-gettext
Feb 13, 2026
Merged

[gettext] Fix wrong variable type#49860
BillyONeal merged 3 commits intomicrosoft:masterfrom
bansan85:40134-gettext

Conversation

@bansan85
Copy link
Contributor

@bansan85 bansan85 commented Feb 6, 2026

Fixes #40134

  • 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.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@BillyONeal
Copy link
Member

Is this patch submitted upstream?

@bansan85
Copy link
Contributor Author

bansan85 commented Feb 7, 2026

The patch comes from upstream. It's the patch that #40134 is reffering to.

Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

This LGTM with a comment for the next person so they know what to do on updates. I submitted that as bansan85#1

@BillyONeal BillyONeal marked this pull request as draft February 10, 2026 02:59
@bansan85
Copy link
Contributor Author

This LGTM with a comment for the next person so they know what to do on updates. I submitted that as bansan85#1

Thanks. I couldn't use patch from upstream. The downloaded source archive haven't the same path.

And updating gettext version looks to be hard.

@SunBlack
Copy link
Contributor

As there exists newer versions of gettext: Wouldn't it better to update the port and check whether the issue still exists?

@dg0yt
Copy link
Contributor

dg0yt commented Feb 10, 2026

As there exists newer versions of gettext: Wouldn't it better to update the port and check whether the issue still exists?

Welcome to the struggle with building gettext with MSVC with vcpkg. Currently gnulib libxml hell.

@bansan85
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 49860 in repo microsoft/vcpkg

Add comment to the patch for where it came from
@bansan85
Copy link
Contributor Author

gobject-introspection fails. Not my fault (I think). I will try again in few days...

@dg0yt
Copy link
Contributor

dg0yt commented Feb 10, 2026

gobject-introspection fails. Not my fault (I think). I will try again in few days...

Appeared between https://dev.azure.com/vcpkg/public/_build/results?buildId=126854&view=results and https://dev.azure.com/vcpkg/public/_build/results?buildId=126963&view=results, but nothing obvious.

@SunBlack
Copy link
Contributor

SunBlack commented Feb 10, 2026

gobject-introspection fails. Not my fault (I think). I will try again in few days...

I don't think a few days will resolve this issue 😅 . I digged a bit into this issue:

  • gobject-introspection relies on distutils, which was part of Python until 3.11
  • distutils is migrated to setuptools, but with breaking changes, as the signature of the init method changed (if AI is correct)
  • vcpkg is using Python 3.12 with setuptools-82.0.0. Therefore you run into this issue

Seems they missed to adjust sth. for setuptools (they check for Python 3.12+ - probably they are just not testing MSVC). Feel free to report this to them - maybe they know a solution for this.

@bansan85
Copy link
Contributor Author

bansan85 commented Feb 10, 2026

@BillyONeal
Copy link
Member

Thanks. I couldn't use patch from upstream. The downloaded source archive haven't the same path.

It's close enough that I think the comment describes where it came from even if that exact patch isn't what's being applied here.

@bansan85
Copy link
Contributor Author

@BillyONeal can you please reopen this MR ? It was wrongly closed by #49918. Thanks

@BillyONeal BillyONeal reopened this Feb 11, 2026
@bansan85 bansan85 marked this pull request as ready for review February 12, 2026 03:16
Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

Thanks!

@BillyONeal BillyONeal merged commit 2232cc8 into microsoft:master Feb 13, 2026
16 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.

[gettext] Build error on x64-mingw-dynamic

4 participants