Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 791811 - sci-mathematics/cgal-5.2.1 doesn't build and install libCGAL library
Summary: sci-mathematics/cgal-5.2.1 doesn't build and install libCGAL library
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major (vote)
Assignee: Gentoo Science Mathematics related packages
URL:
Whiteboard:
Keywords:
: 791694 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-05-24 08:53 UTC by Andrew Savchenko
Modified: 2022-06-08 22:39 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log.gz (cgal-5.2.1:20210524-080653.log.gz,506.48 KB, application/gzip)
2021-05-24 08:53 UTC, Andrew Savchenko
Details
emerge --info cgal (emerge.info,13.24 KB, text/plain)
2021-05-24 08:54 UTC, Andrew Savchenko
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Savchenko gentoo-dev 2021-05-24 08:53:26 UTC
Created attachment 710994 [details]
build.log.gz

Hi!

While cgal-5.2.1 formally builds fine, it doesn't build libCGAL library, which completely breaks everything which uses it (if library is not preserved, e.g. if this is a new build):

$ qlist -e cgal | grep -i '/usr/lib.*libCGAL'
[empty output]

Excerpt from the build.log:

>>> Compiling source in /var/tmp/portage/sci-mathematics/cgal-5.2.1/work/CGAL-5.2.1 ...
 * Working in BUILD_DIR: "/var/tmp/portage/sci-mathematics/cgal-5.2.1/work/cgal-5.2.1_build"
ninja -v -j8 -l10
ninja: no work to do.
>>> Source compiled.

Obviously this is wrong.
Comment 1 Andrew Savchenko gentoo-dev 2021-05-24 08:54:27 UTC
Created attachment 710997 [details]
emerge --info cgal
Comment 2 Andrew Savchenko gentoo-dev 2021-05-24 08:55:12 UTC
*** Bug 791694 has been marked as a duplicate of this bug. ***
Comment 3 Andrew Savchenko gentoo-dev 2021-05-24 08:55:40 UTC
*** Bug 790779 has been marked as a duplicate of this bug. ***
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-05-24 08:56:41 UTC
I suppose it’s: -DCGAL_HEADER_ONLY=ON. I wonder why it’s there?
Comment 6 Andrew Savchenko gentoo-dev 2021-05-24 09:03:14 UTC
(In reply to Sam James from comment #5)
> https://github.com/gentoo/gentoo/pull/20509#discussion_r619079198

Apparently we still need this library for backward compatibility with other packages.
Comment 7 Andrew Savchenko gentoo-dev 2021-05-24 09:50:03 UTC
I think the most reasonable solution will be to add USE flag to control whether library is needed or not. In Gentoo we should provide users the freedom of choose how to build package, not decide for them. Defaults may follow upstream, though, if maintainer feels this is right thing to do.
Comment 8 Joonas Niilola gentoo-dev 2021-05-24 09:51:58 UTC
I had the impression this was working as planned.

(In reply to Andrew Savchenko from comment #7)
> I think the most reasonable solution will be to add USE flag to control
> whether library is needed or not. In Gentoo we should provide users the
> freedom of choose how to build package, not decide for them. Defaults may
> follow upstream, though, if maintainer feels this is right thing to do.

Sounds good to me. I should've done a reverse dep test.
Comment 9 Matthias Maier gentoo-dev 2022-06-08 22:39:46 UTC
USE=+shared has been introduced.