Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change __Pyx_zeros and __Pyx_minusones scope to static. #87

Merged
merged 1 commit into from
Feb 8, 2012

Conversation

bfroehle
Copy link
Contributor

@bfroehle bfroehle commented Feb 8, 2012

Compiling two Cython modules (which use the Cython buffer interface) into a static Python binary results in a link error since the symbols __Pyx_zeros and __Pyx_minusones are exported multiple times. Changing their scope to the current translation unit fixes the issue.

Compiling two Cython modules (which use the Cython buffer interface)
into a static Python binary results in a link error since the symbols
`__Pyx_zeros` and `__Pyx_minusones` are exported multiple times.  Changing
their scope to the current translation unit fixes the issue.
@robertwb
Copy link
Contributor

robertwb commented Feb 8, 2012

Thanks!

robertwb added a commit that referenced this pull request Feb 8, 2012
Change __Pyx_zeros and __Pyx_minusones scope to static.
@robertwb robertwb merged commit 3aa13fa into cython:master Feb 8, 2012
bfroehle added a commit to bfroehle/scipy that referenced this pull request Jul 24, 2012
Manually apply the fix from cython/cython#87, rather than regenerate the
entire cython file.

Having multiple definitions of these symbols prevents liking SciPy into
a static Python binary.
bfroehle added a commit to bfroehle/scipy that referenced this pull request Apr 29, 2013
Manually apply the fix from cython/cython#87, rather than regenerate the
entire cython file.

Having multiple definitions of these symbols prevents liking SciPy into
a static Python binary.
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