Bug 764338 – media-gfx/prusaslicer-2.2.0-r1: CMake Error at src/CMakeLists.txt:77 (add_executable): Target "PrusaSlicer" links to target "Qhull::qhullcpp" but the target was not found

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Toralf Förster 2021-01-07 15:06:54 UTC]

-- Configuring done
CMake Error at src/CMakeLists.txt:77 (add_executable):
  Target "PrusaSlicer" links to target "Qhull::qhullcpp" but the target was
  not found.  Perhaps a find_package() call is missing for an IMPORTED
  target, or an ALIAS target is missing?

  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.1_desktop_gnome_systemd-20210104-123916

  -------------------------------------------------------------------

gcc-config -l:
 [1] x86_64-pc-linux-gnu-10.2.0 *
/usr/lib/llvm/11
11.0.0
Available Python interpreters, in order of preference:
  [1]   python3.8
  [2]   python3.9 (fallback)
Available Rust versions:
  [1]   rust-1.48.0 *
The following VMs are available for generation-2:
*)	AdoptOpenJDK 8.272_p10 [openjdk-bin-8]
Available Java Virtual Machines:
  [1]   openjdk-bin-8  system-vm

The Glorious Glasgow Haskell Compilation System, version 8.8.4

  timestamp(s) of HEAD at this tinderbox image:
/var/db/repos/gentoo	Thu Jan  7 12:06:00 AM UTC 2021

emerge -qpvO media-gfx/prusaslicer
[ebuild  N    ] media-gfx/prusaslicer-2.2.0-r1  USE="gui -test"

[Comment 1 Toralf Förster 2021-01-07 15:06:55 UTC]

Created attachment 681730 [details]
emerge-info.txt

[Comment 2 Toralf Förster 2021-01-07 15:06:57 UTC]

Created attachment 681733 [details]
emerge-history.txt

[Comment 3 Toralf Förster 2021-01-07 15:06:58 UTC]

Created attachment 681736 [details]
environment

[Comment 4 Toralf Förster 2021-01-07 15:06:59 UTC]

Created attachment 681739 [details]
etc.portage.tbz2

[Comment 5 Toralf Förster 2021-01-07 15:07:00 UTC]

Created attachment 681742 [details]
logs.tbz2

[Comment 6 Toralf Förster 2021-01-07 15:07:02 UTC]

Created attachment 681745 [details]
media-gfx:prusaslicer-2.2.0-r1:20210107-004939.log

[Comment 7 Toralf Förster 2021-01-07 15:07:03 UTC]

Created attachment 681748 [details]
temp.tbz2

[Comment 8 Sam James 2021-01-08 16:05:32 UTC]

toralf, could you run this again with the latest qhull?

[Comment 9 Toralf Förster 2021-01-08 16:21:20 UTC]

(In reply to Sam James from comment #8)
> toralf, could you run this again with the latest qhull?

Sure, but the emerge history shows a 

2021-01-06T06:42:59 >>> media-libs/qhull-2020.2-r2

was that package changed w/ a rev bump ?

[Comment 10 Ionen Wolkens 2021-01-08 18:54:04 UTC]

@sam just noticed pkgconfig files are in /usr/lib/pkgconfig despite 64bit, and qhullcpp.pc points to -lqhullcpp but that library is not installed because it's static-only.

[Comment 11 Ionen Wolkens 2021-01-08 21:18:55 UTC]

Created attachment 682015 [details, diff]
re-add static-libs + fix pkgconfig.patch

Lazily dumping qhull patch here for sam to review. I did try to build matplotlib to ensure it doesn't break that again. Not entirely happy with this mess, if anyone has a better fix please use it.

As for prusaslicer... with qhull-2015.2 it was failing to detect the system library and used its bundled copy, seems it uses both qhull_r and qhullcpp(static-only)

From a quick look it (at least) needs:
1. [static-libs] added to qhull (closes this bug)
2. := to qhull
3. [nls] added to boost (ran into boost locale missing)
4. a fix for bug #761541 which I had to work around but haven't looked into further

Gave me:
"-- Using qhull from system."

[Comment 12 Ionen Wolkens 2021-01-08 21:27:35 UTC]

Oh wait, cmake files are wrong too.

[Comment 13 Ionen Wolkens 2021-01-08 21:37:04 UTC]

Created attachment 682018 [details, diff]
re-add static-libs + fix paths.patch

[Comment 14 Ionen Wolkens 2021-01-08 21:41:25 UTC]

Oh and please revbump it too, I forgot to include in the patch.

[Comment 15 Sam James 2021-01-09 01:32:54 UTC]

*** Bug 764494 has been marked as a duplicate of this bug. ***

[Comment 16 Larry the Git Cow 2021-02-05 02:06:47 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4fc943a7edd78e5ba3c0d42f053ca1e966e0032a

commit 4fc943a7edd78e5ba3c0d42f053ca1e966e0032a
Author:     Ionen Wolkens <[e-mail]>
AuthorDate: 2021-01-12 07:35:53 +0000
Commit:     Sam James <[e-mail]>
CommitDate: 2021-02-05 02:06:42 +0000

    media-gfx/prusaslicer: 2.3.0 version bump + fixes
    
    ebuild changes from 2.2.0-r1 to 2.3.0:
    + REQUIRED_USE on gui for tests (bug #760096)
    + patch to add missing miniz header (bug #761541)
    + qhull:=[static-libs] (qhull_r.so subslot + qhullcpp.a, bug #764338)
    + force GTK3 (bug #765031)
    + GUI deps optional with USE=-gui
    + hopefully all missing dependencies
    - Release build type, not needed
    - previous patches (applied upstream)
    + use .desktop files provided by the package
    + update licenses
    + minor cleanups
    
    Closes: https://bugs.gentoo.org/760096
    Closes: https://bugs.gentoo.org/761541
    Closes: https://bugs.gentoo.org/764338
    Closes: https://bugs.gentoo.org/765031
    Package-Manager: Portage-3.0.13, Repoman-3.0.2
    Signed-off-by: Ionen Wolkens <[e-mail]>
    Closes: https://github.com/gentoo/gentoo/pull/19059
    Signed-off-by: Sam James <[e-mail]>

 media-gfx/prusaslicer/Manifest                     |  1 +
 .../files/prusaslicer-2.3.0-miniz-zip-header.patch |  9 +++
 media-gfx/prusaslicer/prusaslicer-2.3.0.ebuild     | 86 ++++++++++++++++++++++
 3 files changed, 96 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=02bc620cff189e56e3363a93d22a729e23387720

commit 02bc620cff189e56e3363a93d22a729e23387720
Author:     Ionen Wolkens <[e-mail]>
AuthorDate: 2021-01-08 20:55:59 +0000
Commit:     Sam James <[e-mail]>
CommitDate: 2021-02-05 02:06:42 +0000

    media-libs/qhull: re-add static-libs + various fixes
    
    CMakeLists.txt claims some libraries are intended to be static-only
    and so weren't installed.
    
    Shared:
    - libqhull.so (deprecated, but needed by matplotlib)
    - libqhull_r.so
    
    Static-only:
    - libqhullcpp.a (needed by at least prusaslicer)
    - libqhullstatic.a
    - libqhullstatic_r.a
    
    Also:
    - install missing deprecated qhull.pc
    - install cmake/pc files in right libdir
    - fix double prefix in pc files
    
    Bug: https://bugs.gentoo.org/764338
    Package-Manager: Portage-3.0.12, Repoman-3.0.2
    Signed-off-by: Ionen Wolkens <[e-mail]>
    Signed-off-by: Sam James <[e-mail]>

 .../files/qhull-2020.2-deprecated-pkgconfig.patch  |  8 +++++++
 ...ull-2020.2-r2.ebuild => qhull-2020.2-r3.ebuild} | 26 +++++++++++++++++-----
 2 files changed, 29 insertions(+), 5 deletions(-)