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

build on all 4 supported Travis architectures #360

Merged
merged 8 commits into from
Feb 25, 2020

Conversation

mr-c
Copy link
Contributor

@mr-c mr-c commented Feb 18, 2020

No description provided.

@taoliu
Copy link
Contributor

taoliu commented Feb 18, 2020

Thanks! I did this before to add arm64 e792658 but Travis can't build Numpy and Cython.

@mr-c
Copy link
Contributor Author

mr-c commented Feb 18, 2020

You are welcome @taoliu !
Lets see if sudo fixes the problems..

@taoliu
Copy link
Contributor

taoliu commented Feb 19, 2020

I am trying arm64 and see this error while compiling numpy:

https://travis-ci.com/taoliu/MACS/jobs/288971199

  gcc: numpy/core/src/multiarray/methods.c
  In file included from numpy/core/src/umath/loops.c.src:50:0:
  numpy/core/src/umath/loops.c.src: In function ‘LONG_right_shift’:
  numpy/core/src/umath/loops.c.src:694:47: internal compiler error: in expand_shift_1, at expmed.c:2349
       BINARY_LOOP_FAST(@type@, @type@, *out = npy_rshift@c@(in1, in2));
                                                 ^
  numpy/core/src/umath/fast_loop_macros.h:180:9: note: in definition of macro ‘BASE_BINARY_LOOP_S_INP’
           op; \
           ^
  numpy/core/src/umath/loops.c.src:694:5: note: in expansion of macro ‘BINARY_LOOP_FAST’
       BINARY_LOOP_FAST(@type@, @type@, *out = npy_rshift@c@(in1, in2));
       ^
  Please submit a full bug report,
  with preprocessed source if appropriate.
  See <file:///usr/share/doc/gcc-5/README.Bugs> for instructions.

Since I saw Numpy can pass Debian autobuilder https://buildd.debian.org/status/package.php?p=numpy&suite=sid, maybe there is some specific FLAG for gcc?

@taoliu
Copy link
Contributor

taoliu commented Feb 20, 2020

Now, I can test arm64 and reproduce the error

https://travis-ci.com/taoliu/MACS/builds/149693293

Add --no-cache-dir to pip: https://github.com/taoliu/MACS/blob/fix_test_multi_arch/.travis.yml

@mr-c
Copy link
Contributor Author

mr-c commented Feb 20, 2020

Thanks @taoliu ! I got hit by a GItHub bug yesterday (my commit wasn't showing up in this PR while it was clearly visible on my branch) so I had given up for a bit.

I'll keep poking on a Python 32-bit x86 Travis CI solution.

@taoliu
Copy link
Contributor

taoliu commented Feb 21, 2020

@mr-c Thanks for the update! I have tried to set 'dist: bionic' in my branch:

https://github.com/taoliu/MACS/blob/8372b149f3e41bae19f97dff7943a9fa0dc21648/.travis.yml#L1-L37

And now MACS2 can be built under the four architectures and I can see the errors from the cmdlinetest.

@taoliu taoliu merged commit e815bbd into macs3-project:master Feb 25, 2020
@taoliu taoliu requested review from taoliu and removed request for taoliu February 25, 2020 14:42
.travis.yml Show resolved Hide resolved
@taoliu taoliu added the v2.2.7 v2.2.7 features label Mar 25, 2020
@mr-c mr-c deleted the travis_s390x branch August 9, 2023 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.2.7 v2.2.7 features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants