-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
problem with building pandas - not using gcc as compiler #14373
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
Comments
please let me know how to attach a text or zip file - if interested! |
pls show architecture, version of cython & python. as well as how you got the source. and version of gcc. |
AIX 5.3, (not gcc, but xlc), 64-bit pip install pandas pip download pandas root@x065:[/data/prj/python/pipbuilds]ls *.gz |
I repeated the process, on AIX 6.1 - 32-bit mode, python built using gcc rather xlc. AIX 6.1, python-2.7.12, gcc-4.7.4 This ends one(?) file earlier - I think the first .cpp (packer.cpp), with different warnings: The "error" being this line: FYI: when using xlc - _packer.o completed. 45482770 116 -rw-r----- 1 root system 117901 Oct 7 10:46 ./pandas-0.19.0/build/temp.aix-5.3-2.7/pandas/msgpack/_packer.o Note the directory above is build/temp.aix5.3-2.7 not aix6.1-2.7 |
we don't support aix directly, meaning that it's not tested at all we also make no guarantees about big endian (powerpc) |
Well, I would consider working on a patch. I am just a humble packager with some programming skills. In other words - I accept hints and tips :) |
@aixtools no real idea what is going on here, maybe @llllllllll has some |
And how would I get, rather this question - get any attention from @llllllllll - as I have never been properly introduced. |
@aixtools there is nothing to do, he will answer when/if he can. otherwise others may be able to offer assistance. |
It looks like the compiler does not support case ranges like: switch(x) {
case a ... b:
} The current CPP branching defines diff --git a/pandas/src/msgpack/unpack_template.h b/pandas/src/msgpack/unpack_template.h
index 95af673..fba372d 100644
--- a/pandas/src/msgpack/unpack_template.h
+++ b/pandas/src/msgpack/unpack_template.h
@@ -17,7 +17,7 @@
*/
#ifndef USE_CASE_RANGE
-#if !defined(_MSC_VER)
+#ifdef __GNUC__
#define USE_CASE_RANGE
#endif
#endif In the short term, you can define |
Many thanks to @llllllllll . As this was closed had fallen low on my radar. However, I hear from a colleague this took care of the problem. Kudos! |
@aixtools if you'd like to do a PR with your change would be great. |
Have to learn how to PR I guess. :) I have git working again (with the strict SSL requirements) - so, eventually, it shall come. |
I tried doing everything locally (on my server) but could not find the exact syntax for a pull-request. So, tried a couple of paths to get a PR done. Hope this is satisfactory! |
USE_CASE_RANGE is a GNU C feature. This change will activate USE_CASE_RANGE on any platform when using GNU C and not on any platform when a different compiler is being used. closes pandas-dev#14373
USE_CASE_RANGE is a GNU C feature. This change will activate USE_CASE_RANGE on any platform when using GNU C and not on any platform when a different compiler is being used. closes #14373
USE_CASE_RANGE is a GNU C feature. This change will activate USE_CASE_RANGE on any platform when using GNU C and not on any platform when a different compiler is being used. closes pandas-dev#14373
USE_CASE_RANGE is a GNU C feature. This change will activate USE_CASE_RANGE on any platform when using GNU C and not on any platform when a different compiler is being used. closes pandas-dev#14373 (cherry picked from commit 794f792)
"""the example is merely
pip install pandas - which fails, so download and run
python ./setup.py build
The text was updated successfully, but these errors were encountered: