Summary: | valgrind 3.7.0: configure fails "checking for a supported version of gcc" when compiler is called x86_64-pc-linux-gnu-gcc | ||
---|---|---|---|
Product: | [Developer tools] valgrind | Reporter: | blueness |
Component: | general | Assignee: | Julian Seward <jseward> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bart.vanassche+kde, christophe.lyon, kalleturtle |
Priority: | NOR | ||
Version: | 3.7.0 | ||
Target Milestone: | --- | ||
Platform: | Gentoo Packages | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: |
Fix gcc regex to get version
Another approach Patch matching braces. Fix compiler version check |
Description
blueness
2011-11-12 01:25:45 UTC
Created attachment 65536 [details]
Fix gcc regex to get version
Created attachment 65588 [details]
Another approach
I have also made a patch for this and i believe mine is a bit more fleshed out and better explained. So i figured id offer it up as an alternative.
Created attachment 65786 [details]
Patch matching braces.
The patch "another approach" does not work because it maches the version number which is between braces, which is provided by --pkgversion= of GCC's configure script. The provider of GCC can choose an arbitrary string here. For instance, Code Sourcery's 2011.03 release prints: arm-none-eabi-gcc (Sourcery G++ Lite 2011.03-42) 4.5.2 And the above mentioned patch returns 2011.03-42. For GCC, the 'right' version number is the one after the string in braces. For Clang, I don't know. I propose the 3rd patch (matching braces) Created attachment 65829 [details]
Fix compiler version check
Proposed patch - tested with gcc only. Should work for clang too.
Should be fixed by r12274 (trunk) - thanks for all the contributed patches. |