Opened 5 years ago

Closed 4 years ago

#21171 closed defect (fixed)

Tachyon patches fail on ARM

Reported by: vbraun Owned by:
Priority: major Milestone: sage-8.0
Component: packages: standard Keywords:
Cc: embray Merged in:
Authors: Konrad K. Dabrowski Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: 9ba1bb2 (Commits, GitHub, GitLab) Commit: 9ba1bb25f49532832701a343e101282e743f00da
Dependencies: Stopgaps:

Status badges


Some fallout from #20933:

Found local metadata for tachyon-0.98.9.p5
Using cached file /home/buildbot/slave/sage_git/build/upstream/tachyon-0.98.9.tar.bz2
Setting up build directory for tachyon-0.98.9.p5
Finished extraction
Host system:
Linux node3eth0 3.5.0-51-highbank #76-Ubuntu SMP Thu May 15 22:16:59 UTC 2014 armv7l armv7l armv7l GNU/Linux
C compiler: gcc
C compiler version:
Using built-in specs.
Target: armv7l-unknown-linux-gnueabihf
Configured with: ../src/configure --prefix=/home/buildbot/slave/sage_git/build/local --with-local-prefix=/home/buildbot/slave/sage_git/build/local --with-gmp=/home/buildbot/slave/sage_git/build/local --with-mpfr=/home/buildbot/slave/sage_git/build/local --with-mpc=/home/buildbot/slave/sage_git/build/local --with-system-zlib --disable-multilib --disable-nls --enable-languages=c,c++,fortran --disable-libitm --with-arch=armv7-a --with-fpu=vfpv3-d16 --with-float=hard  
Thread model: posix
gcc version 4.9.3 (GCC) 
Applying ../patches/Make-arch.patch
patching file unix/Make-arch
Applying ../patches/Make-config.patch
patching file unix/Make-config
Applying ../patches/main.c.patch
patching file demosrc/main.c
sed: can't read ../../patches/Make-arch: No such file or directory
make[3]: Entering directory `/home/buildbot/slave/sage_git/build/local/var/tmp/sage/build/tachyon-0.98.9.p5/src/unix'
make[3]: *** No rule to make target `linux-thr'.
make[3]: Leaving directory `/home/buildbot/slave/sage_git/build/local/var/tmp/sage/build/tachyon-0.98.9.p5/src/unix'
Error building Tachyon Ray Tracer.

real	0m0.154s
user	0m0.020s
sys	0m0.000s
Error installing package tachyon-0.98.9.p5

Change History (7)

comment:1 Changed 5 years ago by embray

I see. That's due to this:

Instead, it could either sed the file in-place, or perhaps better have a platform-specific patch.

comment:2 Changed 5 years ago by vbraun

I'd go with a platform-specific patch ...

comment:3 Changed 4 years ago by Konrad127123

  • Branch set to u/Konrad127123/tachyon_patches_fail_on_arm

comment:4 Changed 4 years ago by Konrad127123

  • Commit set to 9ba1bb25f49532832701a343e101282e743f00da
  • Status changed from new to needs_review

I've done a platform-specific patch. I've tested on ARM (raspbian) and tachyon compiles with this patch.

New commits:

9ba1bb2Fix compiling Tachyon on Arm (#21171)

comment:5 Changed 4 years ago by jdemeyer

  • Milestone changed from sage-7.3 to sage-8.0
  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_review to positive_review

Author name please....

comment:6 Changed 4 years ago by Konrad127123

  • Authors set to Konrad K. Dabrowski

comment:7 Changed 4 years ago by vbraun

  • Branch changed from u/Konrad127123/tachyon_patches_fail_on_arm to 9ba1bb25f49532832701a343e101282e743f00da
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.