Bug 1382453 - clang missing dependency on compiler-rt
Summary: clang missing dependency on compiler-rt
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: clang
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tom Stellard
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-06 17:57 UTC by Nathaniel McCallum
Modified: 2017-03-16 13:12 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-16 13:12:13 UTC
Type: Bug

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

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.
Note You need to log in before you can comment on or make changes to this bug.