Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 387309 - sys-devel/clang - fix corrected parsing of the path given by --with-cxx-include-root when USE flag system-cxx-headers is used
Summary: sys-devel/clang - fix corrected parsing of the path given by --with-cxx-inclu...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
: 388801 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-10-16 17:55 UTC by stephan wezel
Modified: 2011-11-09 15:40 UTC (History)
1 user (show)

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


Attachments
fixes the cxx_include_root path parsing under gentoo (clang-fix_cxx_include_root_parsing_for_gentoo.patch,1.32 KB, patch)
2011-10-16 17:57 UTC, stephan wezel
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description stephan wezel 2011-10-16 17:55:53 UTC
clang in trunk for v3.0 has changed its handling of the path given by the configure option --with-cxx-include-root

Due this change trying to compile a program with clang fails because the linker doesn't find the crtbegin.o file

Attached is a patch which fix this issue. The patch was made against subversion revision 142115 of the release_30 branche of clang

Reproducible: Always
Comment 1 stephan wezel 2011-10-16 17:57:00 UTC
Created attachment 289965 [details, diff]
fixes the cxx_include_root path parsing under gentoo
Comment 2 Bernard Cafarelli gentoo-dev 2011-11-08 13:53:05 UTC
I should have time soon to test 3.0 rc1 so I will probably get this bug too, but I think modifying line 95 of clang-9999.ebuild like that should do the trick:

CONF_FLAGS="${CONF_FLAGS} --with-cxx-include-root=${CXX_PATH}

(dropping the include/g++-v4)
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2011-11-08 19:31:50 UTC
(In reply to comment #2)
> I should have time soon to test 3.0 rc1 so I will probably get this bug too,
> but I think modifying line 95 of clang-9999.ebuild like that should do the
> trick:
> 
> CONF_FLAGS="${CONF_FLAGS} --with-cxx-include-root=${CXX_PATH}
> 
> (dropping the include/g++-v4)

Tried that on my -9999 and didn't help.
Comment 4 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2011-11-08 19:32:29 UTC
*** Bug 388801 has been marked as a duplicate of this bug. ***
Comment 5 Bernard Cafarelli gentoo-dev 2011-11-09 14:06:20 UTC
Indeed the problem only is in ToolChains.cpp.

Thanks Stephan for the pointer to the problem and the patch, I'll add it to both 3.0_rc1 and 9999 (3.0 will use it anyways)

Of course on the long term, implementing the suggested CXX_GCC_* variables upstream (mentioned in the FIXME) would be cleanier (or an easier way to add new GCC versions, or a configuration file somewhere...)
Comment 6 Bernard Cafarelli gentoo-dev 2011-11-09 15:40:57 UTC
I'll do more work later on llvm/clang ebuilds, but in the meantime, 3.0_rc1 is available and 9999 updated with the patch.

Thanks again for the report and fix Stephan!