Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#325 closed defect (fixed)

configure doesn't recognize amd64 on PLD Linux

Reported by: pawel@… Owned by: diego@…
Priority: normal Component: build system
Version: 1.0pre7 Severity: minor
Keywords: Cc:
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

The following rule (configure line 475):

if [ "$_cc -dumpmachine | grep x86_64 | cut -d- -f1" = "x86_64" -a \

doesn't catch the amd64 gcc from PLD Linux Distribution's which reponds
"amd64-pld-linux" to gcc -dumpmachine . It eventually leads to build failures
on assembler code. Extending the above heuristics to detect "amd64" as well as
"x86_64" makes mplayer build just fine.

Attachments (2)

mplayer-x86_64-detection.patch (574 bytes ) - added by pawel@… 19 years ago.
A fix for the problem
mplayer-x86_64-detection.2.patch (581 bytes ) - added by pawel@… 19 years ago.
A sed-based solution

Download all attachments as: .zip

Change History (8)

comment:1 by diego@…, 19 years ago

Component: corebuild system
Owner: changed from alex@… to diego@…

comment:2 by diego@…, 19 years ago

Status: newassigned
Summary: configure doesn't recognize amd64configure doesn't recognize amd64 on PLD Linux

I'll apply your patch once you send it ;-)

by pawel@…, 19 years ago

A fix for the problem

comment:3 by pawel@…, 19 years ago

This patch made it work for me.

comment:4 by diego@…, 19 years ago

This patch is non-portable. Please find a way to accomplish this without grep -E.

by pawel@…, 19 years ago

A sed-based solution

comment:5 by pawel@…, 19 years ago

attachments.isobsolete: 01

Here's a variant of the previous patch using sed instead of grep -E.

If you prefer using other text processing tools (eg. awk), I am willing to
prepare a version using those as well. Using plain grep is also possible.

comment:6 by diego@…, 19 years ago

Resolution: fixed
Status: assignedclosed

Patch committed.

Note: See TracTickets for help on using tickets.