Opened 6 months ago

Closed 6 months ago

#32244 closed defect (fixed)

homebrew: csdp installation broken

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.4
Component: packages: optional Keywords:
Cc: dimpase Merged in:
Authors: Dima Pasechnik Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: 37aedbf (Commits, GitHub, GitLab) Commit: 37aedbfcaca0d1480ef67261b0010a4424c384ce
Dependencies: Stopgaps:

Status badges

Description

Prompted by ​https://groups.google.com/g/sage-support/c/2ufrP1M4mRs/m/bpMfD-oKBwAJ I tried to install csdp but .... 

[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c easysdp.c  -fno-common -DPIC -o .libs/libsdp_la-easysdp.o
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c writeprob.c  -fno-common -DPIC -o .libs/libsdp_la-writeprob.o
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c solvesys.c  -fno-common -DPIC -o .libs/libsdp_la-solvesys.o
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c makefill.c  -fno-common -DPIC -o .libs/libsdp_la-makefill.o
[csdp-6.2.p0] /bin/bash ../libtool  --tag=CC   --mode=compile gcc -DHAVE_CONFIG_H -I. -I..  -I../include -DNOSHORTS  -O2 -g -march=native -c -o libsdp_la-linesearch.lo `test -f 'linesearch.c' || echo './'`linesearch.c
[csdp-6.2.p0] /bin/bash ../libtool  --tag=CC   --mode=compile gcc -DHAVE_CONFIG_H -I. -I..  -I../include -DNOSHORTS  -O2 -g -march=native -c -o libsdp_la-matvec.lo `test -f 'matvec.c' || echo './'`matvec.c
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c mat_multsp.c  -fno-common -DPIC -o .libs/libsdp_la-mat_multsp.o
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c norms.c  -fno-common -DPIC -o .libs/libsdp_la-norms.o
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c matvec.c  -fno-common -DPIC -o .libs/libsdp_la-matvec.o
[csdp-6.2.p0] libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I.. -I../include -DNOSHORTS -O2 -g -march=native -c linesearch.c  -fno-common -DPIC -o .libs/libsdp_la-linesearch.o
[csdp-6.2.p0] easysdp.c:436:4: error: implicit declaration of function 'checkc' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
[csdp-6.2.p0]    checkc(n,C,printlevel);
[csdp-6.2.p0]    ^
[csdp-6.2.p0] easysdp.c:442:4: error: implicit declaration of function 'checkconstraints' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
[csdp-6.2.p0]    checkconstraints(n,k,C,constraints,printlevel);
[csdp-6.2.p0]    ^
[csdp-6.2.p0] 2 errors generated.
[csdp-6.2.p0] make[5]: *** [libsdp_la-easysdp.lo] Error 1

Change History (7)

comment:1 Changed 6 months ago by dimpase

  • Authors set to Dima Pasechnik
  • Branch set to u/dimpase/packages/csdp/fixclang
  • Commit set to e5687266f18a26d411547ebbf345f854aa34a21c
  • Status changed from new to needs_review

New commits:

​e568726patch for Apple's clang 12

comment:2 Changed 6 months ago by mkoeppe

  • Reviewers set to Matthias Koeppe
  • Status changed from needs_review to positive_review

Thanks, this works.

comment:3 Changed 6 months ago by mkoeppe

  • Status changed from positive_review to needs_work

But there is another copy of the github.org URL in src/sage/features/csdp.py

comment:4 Changed 6 months ago by git

  • Commit changed from e5687266f18a26d411547ebbf345f854aa34a21c to 37aedbfcaca0d1480ef67261b0010a4424c384ce

Branch pushed to git repo; I updated commit sha1. New commits:

​37aedbforg->com, http->https for cvxopt

comment:5 Changed 6 months ago by dimpase

  • Status changed from needs_work to needs_review

ok, hopefully it's all of these

comment:6 Changed 6 months ago by mkoeppe

  • Status changed from needs_review to positive_review

comment:7 Changed 6 months ago by vbraun

  • Branch changed from u/dimpase/packages/csdp/fixclang to 37aedbfcaca0d1480ef67261b0010a4424c384ce
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.