Opened 14 years ago

Closed 14 years ago

Last modified 14 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: Analyzed by developer:

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@… 14 years ago.
A fix for the problem
mplayer-x86_64-detection.2.patch (581 bytes) - added by pawel@… 14 years ago.
A sed-based solution

Download all attachments as: .zip

Change History (8)

comment:1 Changed 14 years ago by diego@…

  • Component changed from core to build system
  • Owner changed from alex@… to diego@…

comment:2 Changed 14 years ago by diego@…

  • Status changed from new to assigned
  • Summary changed from configure doesn't recognize amd64 to configure doesn't recognize amd64 on PLD Linux

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

Changed 14 years ago by pawel@…

A fix for the problem

comment:3 Changed 14 years ago by pawel@…

This patch made it work for me.

comment:4 Changed 14 years ago by diego@…

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

Changed 14 years ago by pawel@…

A sed-based solution

comment:5 Changed 14 years ago by pawel@…

  • attachments.isobsolete changed from 0 to 1

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 Changed 14 years ago by diego@…

  • Resolution set to fixed
  • Status changed from assigned to closed

Patch committed.

Note: See TracTickets for help on using tickets.