Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 701912 - media-libs/assimp-5.0.0 - assimpTargets-release.cmake references /usr/lib
Summary: media-libs/assimp-5.0.0 - assimpTargets-release.cmake references /usr/lib
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
: 702220 702562 (view as bug list)
Depends on:
Blocks: no-symlink-lib
  Show dependency tree
 
Reported: 2019-12-03 19:04 UTC by Eric Timmons
Modified: 2020-06-05 12:53 UTC (History)
4 users (show)

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


Attachments
check if we are building against x86_64 (0001-RFC-cmake-targets-check-lib-or-lib64-path.patch,3.57 KB, patch)
2020-02-26 14:57 UTC, Nikita Shubin
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Timmons 2019-12-03 19:04:54 UTC
The file assimpTargets-release.cmake directs cmake to find libassimp.so in /usr/lib. It seems that this is hardcoded in the assimpTargets-release.cmake.in file from upstream.

Noticed when attempting to compile ROS Melodic from source (the rviz package in particular).

Reproducible: Always
Comment 1 Alexis Ballier gentoo-dev 2019-12-12 11:42:18 UTC
*** Bug 702220 has been marked as a duplicate of this bug. ***
Comment 2 Alexis Ballier gentoo-dev 2019-12-12 11:42:23 UTC
*** Bug 702562 has been marked as a duplicate of this bug. ***
Comment 3 Victor Mataré 2020-01-08 18:14:32 UTC
This is actually upstream bug. If someone finds the time, please report it here: https://github.com/assimp/assimp/issues

possibly related:
https://github.com/assimp/assimp/issues/2719
Comment 4 Nikita Shubin 2020-02-26 14:57:16 UTC
Created attachment 615958 [details, diff]
check if we are building against x86_64

As a possible solution, needs checking.

 This patch is against v5.0.1 version !
Comment 5 Victor Mataré 2020-03-31 22:12:57 UTC
I posted a PR with a more correct/portable fix at https://github.com/assimp/assimp/pull/3115. Hopefully they'll accept that, but I'm guessing it'll have to be tested on Windows first. It does cleanly fix the situation on my Gentoo box, though.
Comment 6 Larry the Git Cow gentoo-dev 2020-06-05 12:48:53 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6110fc259680b841e7c2a7d6a168b7666d6d90bf

commit 6110fc259680b841e7c2a7d6a168b7666d6d90bf
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2020-06-05 12:21:45 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2020-06-05 12:48:43 +0000

    media-libs/assimp: 5.0.1 version bump, switch to cmake.eclass, EAPI-7
    
    - Add IUSE="doc"
    - Drop IUSE="tools" as Qt5 based viewer was removed upstream
      commit 13586eb0739146dc609fa83bcd71fee1bb66f4b7
    - GNUInstallDirs++
      patch from commit cbf94fd62ff831879d10f99aa7766d391ae8a9b7
    
    Thanks-to: Victor Mataré <matare@fh-aachen.de>
    Closes: https://bugs.gentoo.org/701912
    Package-Manager: Portage-2.3.100, Repoman-2.3.22
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 media-libs/assimp/Manifest                         |   2 +
 media-libs/assimp/assimp-5.0.1.ebuild              |  62 ++++++
 .../assimp/files/assimp-5.0.1-GNUInstallDirs.patch | 239 +++++++++++++++++++++
 3 files changed, 303 insertions(+)