Opened 4 years ago

Closed 4 years ago

#51539 closed defect (fixed)

qrupdate requires destroot.target commands

Reported by: chrisjogorman Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: haspatch Cc: kurthindenburg (Kurt Hindenburg), dstrubbe (David Strubbe)
Port: qrupdate

Description

qrupdate builds but doesn't install on my machine. It fails to call install-shlib and install-staticlib after making the libraries. Adding destroot.target install install-shlib install-staticlib allows the build to install.

Attachments (1)

51539.diff (284 bytes) - added by chrisjogorman 4 years ago.

Download all attachments as: .zip

Change History (11)

Changed 4 years ago by chrisjogorman

Attachment: 51539.diff added

comment:1 Changed 4 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:2 Changed 4 years ago by kurthindenburg (Kurt Hindenburg)

Cc: dstrubbe@… added

dstrubbe, I see you updated this recently. This patch does appear to fix this issue.

Also the tests fail for me.

comment:3 Changed 4 years ago by dstrubbe (David Strubbe)

Sorry about that, there was a patch that I removed in r148974 that caused this issue. I put it back, r149149.

comment:4 in reply to:  2 Changed 4 years ago by dstrubbe (David Strubbe)

Replying to khindenburg@…:

Also the tests fail for me.

What fails? What variants are you using? All the combinations I tried on my machine pass.

comment:5 Changed 4 years ago by kurthindenburg (Kurt Hindenburg)

It is one of these x86_64 errors - I can open a new ticket w/ the full log if you're interested - I'm using the default variants on latest OS version.

/opt/local/bin/gfortran-mp-5 -m64 -fimplicit-none -O3 -funroll-loops  -o tqr1up tqr1up.f utils.o ../libqrupdate.a -lblas
Undefined symbols for architecture x86_64:
  "_cgeqrf_", referenced from:
      _cqrgen_ in utils.o

comment:6 Changed 4 years ago by dstrubbe (David Strubbe)

That line should be

/opt/local/bin/gfortran-mp-5 -m64 -fimplicit-none -O3 -funroll-loops  -o tqr1up tqr1up.f utils.o ../libqrupdate.a -lvecLibFort

Is this really +accelerate variant?

comment:7 Changed 4 years ago by kurthindenburg (Kurt Hindenburg)

yes

qrupdate @1.1.2_4+accelerate+gcc5 (active)

comment:8 Changed 4 years ago by mf2k (Frank Schima)

Keywords: qrupdate install-shlib install-staticlib removed

comment:9 Changed 4 years ago by mf2k (Frank Schima)

Keywords: haspatch added

comment:10 Changed 4 years ago by dstrubbe (David Strubbe)

Resolution: fixed
Status: newclosed

Ok, can you make another ticket about the test failures and put the full log? I'm closing this about the installation issue.

Note: See TracTickets for help on using tickets.