Bug 412421 - liquidwar uses -mcpu in a strange way... Patch included
Summary: liquidwar uses -mcpu in a strange way... Patch included
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: liquidwar
Version: 8
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-05 17:02 UTC by Oliver Falk
Modified: 2008-01-11 22:19 UTC (History)
0 users

Fixed In Version: 5.6.4-2.fc8
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-11 22:19:39 UTC


Attachments (Terms of Use)
Remove TARGET_CPU stuff from Makefile... (1.74 KB, patch)
2007-12-05 17:02 UTC, Oliver Falk
no flags Details | Diff

Description Oliver Falk 2007-12-05 17:02:44 UTC
On alpha it doesn't even work and on every other arch, CFLAGS should set mcpu
correctly. Therefor patch included. This patch should be ok for all arches.

Comment 1 Oliver Falk 2007-12-05 17:02:44 UTC
Created attachment 278541 [details]
Remove TARGET_CPU stuff from Makefile...

Comment 2 Hans de Goede 2007-12-12 21:31:25 UTC
Thanks, for the patch, but this is already fixed in rawhide in a more elegant
way: ./configure gets passed --disable-target-opt, which sets $(TARGET_OPT) to
no, resulting in TARGET_CPU_CC getting assigned an empty string as value.


Comment 3 Oliver Falk 2007-12-12 21:59:51 UTC
ok hand. can you update f8 to rawhide version?

Comment 4 Hans de Goede 2007-12-12 22:06:49 UTC
If that helps those working on an alpha version of Fedora sure, I assume thats
the reason for the request? As I see no immediate benefit for end users.


Comment 5 Oliver Falk 2007-12-12 22:15:54 UTC
Yes, that's the reason. I'd like to get as much in sync with F8 as possible :-)

Comment 6 Fedora Update System 2007-12-15 17:45:21 UTC
liquidwar-5.6.4-2.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update liquidwar'

Comment 7 Oliver Falk 2007-12-16 21:39:36 UTC
Thx. WorksForMe(tm)

Comment 8 Fedora Update System 2008-01-11 22:19:38 UTC
liquidwar-5.6.4-2.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.