Opened 4 years ago

Closed 3 years ago

#24638 closed defect (fixed)

Fix OpenBLAS build on SPARC

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-8.2
Component: packages: standard Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Dima Pasechnik
Report Upstream: Fixed upstream, but not in a stable release. Work issues:
Branch: e350b30 (Commits, GitHub, GitLab) Commit: e350b306612946bcc407efeebee331441d5728c5
Dependencies: Stopgaps:

Status badges

Change History (14)

comment:1 Changed 4 years ago by jdemeyer

  • Branch set to u/jdemeyer/fix_openblas_build_on_sparc

comment:2 Changed 4 years ago by jdemeyer

  • Commit set to b76a7663c14e5e8ffa7e3003577ef21d8c225eb5
  • Status changed from new to needs_review

New commits:

b76a766Fix OpenBLAS build on SPARC

comment:3 Changed 4 years ago by dimpase

  • Reviewers set to Dima Pasechnik
  • Status changed from needs_review to positive_review

looks good to me.

comment:4 Changed 4 years ago by jdemeyer

  • Status changed from positive_review to needs_work

Still fails to build because of a different error:

openblas_get_config.c:58:3: error: expected ',' or ';' before 'CHAR_CORENAME'
   CHAR_CORENAME
   ^
openblas_get_config.c: In function 'openblas_get_corename':
openblas_get_config.c:81:10: error: 'CHAR_CORENAME' undeclared (first use in this function)
   return CHAR_CORENAME;
          ^
openblas_get_config.c:81:10: note: each undeclared identifier is reported only once for each function it appears in
openblas_get_config.c:85:1: warning: control reaches end of non-void function [-Wreturn-type]
 }
 ^
make[3]: *** [Makefile:113: openblas_get_config.o] Error 1

comment:5 Changed 4 years ago by jdemeyer

  • Description modified (diff)
  • Report Upstream changed from Fixed upstream, but not in a stable release. to Reported upstream. No feedback yet.

comment:6 Changed 4 years ago by jdemeyer

  • Description modified (diff)
  • Report Upstream changed from Reported upstream. No feedback yet. to Fixed upstream, but not in a stable release.

comment:7 Changed 4 years ago by git

  • Commit changed from b76a7663c14e5e8ffa7e3003577ef21d8c225eb5 to a4d4ff4daf2a603f2211d16708fa4cefbbf4dab6

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

a4d4ff4Fix OpenBLAS build on SPARC

comment:8 Changed 4 years ago by jdemeyer

  • Status changed from needs_work to needs_review

comment:9 Changed 4 years ago by git

  • Commit changed from a4d4ff4daf2a603f2211d16708fa4cefbbf4dab6 to e350b306612946bcc407efeebee331441d5728c5

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

e350b30Fix OpenBLAS build on SPARC

comment:10 Changed 4 years ago by jdemeyer

This now actually works.

comment:11 Changed 4 years ago by dimpase

I will test it in due course, I'd proceed with the Atlas build for the time being.

comment:12 Changed 4 years ago by dimpase

This builds with gcc 7.2 on Solaris. Needs testing, still.

comment:13 Changed 4 years ago by dimpase

  • Status changed from needs_review to positive_review

Looks good to go.

comment:14 Changed 3 years ago by vbraun

  • Branch changed from u/jdemeyer/fix_openblas_build_on_sparc to e350b306612946bcc407efeebee331441d5728c5
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.