Skip to content

ARROW-1248: [Python] Suppress return-type-c-linkage warning in Cython clang builds #888

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.

Sign up for GitHub

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

Jump to bottom
Closed
wesm wants to merge 1 commit into from
Closed

ARROW-1248: [Python] Suppress return-type-c-linkage warning in Cython clang builds #888

wesm wants to merge 1 commit into from

Conversation

@wesm
Copy link
Member

@wesm wesm commented Jul 25, 2017

I also removed some other unused CMake cruft from the build system. I think the warning is innocuous, but these symbols in question will only be callable from C++:

$ nm -g pyarrow/lib.cpython-35m-x86_64-linux-gnu.so | grep unwrap
0000000000055ef0 T pyarrow_unwrap_array
0000000000058590 T pyarrow_unwrap_batch
0000000000054690 T pyarrow_unwrap_buffer
0000000000057d90 T pyarrow_unwrap_column
0000000000054df0 T pyarrow_unwrap_data_type
0000000000055640 T pyarrow_unwrap_field
0000000000055af0 T pyarrow_unwrap_schema
0000000000058190 T pyarrow_unwrap_table
0000000000057880 T pyarrow_unwrap_tensor

@wesm
Suppress return-type-c-linkage warning in Cython clang builds. Remove…
986cf88 
… other python CMake cruft

Change-Id: I0399a6a826207c33ce617f9aa6a9fbd2c7bb2ede
@wesm
Copy link
Member Author

wesm commented Jul 26, 2017

+1

@asfgit asfgit closed this in 2eeaa95 Jul 26, 2017
@wesm wesm deleted the ARROW-1248 branch July 26, 2017 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@wesm