Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 213902 - app-office/plan uses generic gcc command to compile
Summary: app-office/plan uses generic gcc command to compile
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-19 09:01 UTC by Chris Slycord
Modified: 2008-10-29 04:25 UTC (History)
2 users (show)

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


Attachments
diff between fixed ebuild and one in tree (plan-1.9.ebuild.diff,753 bytes, text/plain)
2008-03-19 09:02 UTC, Chris Slycord
Details
patch to fix Makefile to respect ${CC} instead of hardcoded reference to gcc (makefile.patch,659 bytes, text/plain)
2008-03-19 09:03 UTC, Chris Slycord
Details
Fixed patch (makefile.patch,639 bytes, text/plain)
2008-08-27 17:42 UTC, Chris Slycord
Details
New diff between current ebuild and one that uses new patch correctly (the_diff,334 bytes, text/plain)
2008-08-27 17:43 UTC, Chris Slycord
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Slycord 2008-03-19 09:01:17 UTC
+++ This bug was initially created as a clone of Bug #194233 +++

During the compile phase for app-office/plan (version 1.9), gcc is invoked instead of a CHOST-specific command (like i686-pc-linux-gnu-gcc). This could cause troubles if trying to do, say, a cross-compile and the system gcc gets called (since normally it would use the CHOST for the cross compiler).

Reproducible: Always

Steps to Reproduce:
1. Install app-office/plan
2. Look at output during either the configure or compile phases
3.

Actual Results:  
gcc is invoked

Expected Results:  
A CHOST-specific compiler name to be invoked

NOTE: A simple ebuild fix and Makefile patch to follow
2nd note: Ebuild also includes minor fix to elog message (ebuild in tree displays message about version number 1.8 instead of the current version)
Comment 1 Chris Slycord 2008-03-19 09:02:16 UTC
Created attachment 146554 [details]
diff between fixed ebuild and one in tree
Comment 2 Chris Slycord 2008-03-19 09:03:14 UTC
Created attachment 146555 [details]
patch to fix Makefile to respect ${CC} instead of hardcoded reference to gcc
Comment 3 Markus Meier gentoo-dev 2008-05-28 20:46:35 UTC
fixed in cvs (along other things), thanks for the patch.
Comment 4 Evgeny Stambulchik 2008-08-26 11:51:13 UTC
The patch uses the full path for the file to be patched: /var/tmp/portage/app-office/plan-1.9/work/plan-1.9/src/Makefile

Please change it to src/Makefile - otherwise, for those of us overriding PORTAGE_TMPDIR the ebuild fails.

Also, please consider adding the patch to plan-1.9-gentoo.tar.bz2 instead.
Comment 5 Chris Slycord 2008-08-27 17:16:20 UTC
Reopening due to comment #4

Comment 6 Chris Slycord 2008-08-27 17:42:35 UTC
Created attachment 163917 [details]
Fixed patch
Comment 7 Chris Slycord 2008-08-27 17:43:37 UTC
Created attachment 163918 [details]
New diff between current ebuild and one that uses new patch correctly
Comment 8 Jeroen Roovers (RETIRED) gentoo-dev 2008-10-29 04:25:28 UTC
I fixed the ${P}-makefile.patch in CVS. I did not add it to the patch tarball as that would involve setting up a versioned tarball system (as is not presently available, so I would have to push it under a new naming scheme). Even without the ebuild patch it unpacks and patches fine - it doesn't need the `cd "${S}"' as long as the path is `plan-1.9/src/Makefile'.