Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 689242 - dev-util/spirv-llvm-translator: insecure RUNPATHs in llvm-spirv
Summary: dev-util/spirv-llvm-translator: insecure RUNPATHs in llvm-spirv
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Marek Szuba
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-07-04 10:04 UTC by Marek Szuba
Modified: 2019-07-04 12:01 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marek Szuba archtester gentoo-dev 2019-07-04 10:04:53 UTC
Partly note to self, partly to preempt user reports of this. The reason for this is likely the ebuilds presently install the llvm-spirv executable by hand, this not giving CMake the opportunity to strip build-time RPATHS from it. Should just add an install directive to the relevant CMakeFile instead - and while at it submit the change upstream.
Comment 1 Larry the Git Cow gentoo-dev 2019-07-04 12:01:34 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6e1dfe5cfd4ba44f6da00e59a038b6033893cc14

commit 6e1dfe5cfd4ba44f6da00e59a038b6033893cc14
Author:     Marek Szuba <marecki@gentoo.org>
AuthorDate: 2019-07-04 12:00:19 +0000
Commit:     Marek Szuba <marecki@gentoo.org>
CommitDate: 2019-07-04 12:01:22 +0000

    dev-util/spirv-llvm-translator: streamline installation of cmdline tool
    
    1. Use the LLVM_BUILD_TOOLS flag from the CMake module AddLLVM to
    control whether to build and install llvm-spirv or not. This should also
    take care of the "insecure RUNPATHs" QA warning;
    
    2. Make the above conditional upon USE=tools.
    
    Closes: https://bugs.gentoo.org/689242
    Signed-off-by: Marek Szuba <marecki@gentoo.org>
    Package-Manager: Portage-2.3.66, Repoman-2.3.11

 dev-util/spirv-llvm-translator/metadata.xml        |  5 ++
 .../spirv-llvm-translator-8.0.1.1-r1.ebuild        | 53 ++++++++++++++++++++++
 2 files changed, 58 insertions(+)