Bug 1382453

Summary: clang missing dependency on compiler-rt
Product: [Fedora] Fedora Reporter: Nathaniel McCallum <npmccallum>
Component: clangAssignee: Tom Stellard <tstellar>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: airlied, davejohansen, jv+fedora, sbergman, tstellar
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-16 13:12:13 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Nathaniel McCallum 2016-10-06 17:57:56 UTC
When I install a minimal container and then install clang inside of it, the installation is successful but clang is unusable without compiler-rt installed. Instead, clang should have a dependency on compiler-rt. This is reproducable on F24, F25 and rawhide.

Comment 1 Jan Včelák 2016-11-19 11:56:30 UTC
Unusable? It works for me unless I need address sanitizer.

Comment 2 Tom Stellard 2017-03-10 19:36:53 UTC
Can you provide an example of what wasn't working?

Comment 3 Tom Stellard 2017-03-13 10:40:52 UTC
Re-opening, compiler-rt shouldn't be a hard dependency of clang.  We need more information about what wasn't working.

Comment 4 Nathaniel McCallum 2017-03-13 14:46:34 UTC
The address sanitizer flag caused autoconf to error with "Unable to create executables." A CFLAG shouldn't cause compilation to fail.

Comment 5 Tom Stellard 2017-03-13 15:17:07 UTC
Ok, I think we should see if we can split the address sanitizer libs out into a package separate from the runtime library.  I will take  a look at this.

Comment 6 Tom Stellard 2017-03-16 13:12:13 UTC
I was a little concerned that clang might try to use the compiler-rt builtin library by default if it was installed, but this doesn't seem to be the case  at the moment.  It don't think there is much value in splitting the package right now, so I'm re-closing this bug.