Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 202704
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo Shell Tools Herd Bugs <shell-tools@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Thomas Anderson (tanderson) <tanderson@gentoo.org>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
color-1.1-fix-cc-gcc.patch Fix cc->gcc stuff patch Thomas Anderson (tanderson) 2007-12-18 15:45 0000 318 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 202704 depends on: Show dependency tree
Bug 202704 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2007-12-18 15:42 0000
When trying to compile app-misc/color-1.1 I hit an error. After looking arounda
bit I narrowed it down to the sed statement in src_unpack(). The problem is
that it does s/cc/$(tc-getCC)/g where the original thing in the Makefile is
'gcc'. Thus the sed statement replaces 'cc' with your compiler name and you get 

gx86_64-pc-linux-gnu-gcc -march=athlon64 -O2 -pipe    -c -o color.o color.c

instead of 

x86_64-pc-linux-gnu-gcc -march=athlon64 -O2 -pipe    -c -o color.o color.c


I'm attaching a patch.

Reproducible: Always

------- Comment #1 From Thomas Anderson (tanderson) 2007-12-18 15:45:41 0000 -------
Created an attachment (id=138834) [details]
Fix cc->gcc stuff

I should also mention that the problem is not in color-1.2 because in 1.2
upstream changed from gcc->cc and the bug is therefore not reproducable on that
version.

------- Comment #2 From Sven Wegener 2008-02-10 14:01:53 0000 -------
Thanks. As discussed on IRC I've changed the ebuild to

        emake CC="$(tc-getCC)" CFLAGS="${CFLAGS}" || die "emake failed"

Overriding the values this way is much saner than messing around with sed.

Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug