Skip to content

Conversation

@shadchin
Copy link
Contributor

@shadchin shadchin commented Jan 6, 2026

pygit2 also defines this symbol - https://github.com/libgit2/pygit2/blob/master/src/pygit2.c#L411

If you build statically, there will be a name conflict. Common practice is to write static PyMethodDef module_methods[] = ....

Related libgit2/pygit2#1446

Copy link
Member

@seberg seberg left a comment

Choose a reason for hiding this comment

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

Thanks, this shouldn't be used (although it would be nice to change the default linking to avoid this type of thing).

@seberg seberg added the 09 - Backport-Candidate PRs tagged should be backported label Jan 6, 2026
@seberg seberg merged commit 9d48ad4 into numpy:main Jan 6, 2026
76 checks passed
@shadchin shadchin deleted the patch-2 branch January 6, 2026 11:44
charris pushed a commit to charris/numpy that referenced this pull request Jan 6, 2026
Need to use `static` for local functions and struct definitions.
charris pushed a commit to charris/numpy that referenced this pull request Jan 6, 2026
Need to use `static` for local functions and struct definitions.
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

36 - Build Build related PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants