Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 59228 - mythdvd-0.15 broken compiler arguments under amd64/x86_64
Summary: mythdvd-0.15 broken compiler arguments under amd64/x86_64
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: High normal
Assignee: AMD64 Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-02 22:12 UTC by James Couzens
Modified: 2004-09-11 03:48 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description James Couzens 2004-08-02 22:12:39 UTC
When compiling GCC is being hard-set -march=pentiumpro and of course this leads to failure given my environment is also setting -m64 causing cc1plus to yield:

cc1plus: error: CPU you selected does not support x86-64 instruction set


Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Lars Hagstrom 2004-08-15 07:01:25 UTC
All the myth plugins need to have their src_compile() function modified.
The code
local cpu="`get-flag march || get-flag mcpu`"
  if [ ! -z "${cpu}" ] ; then
    sed -e "s:pentiumpro:${cpu}:g" -i "settings.pro" || die "sed failed"
  fi
Should be replaced with
  if [ "${ARCH}" == "amd64" ]; then
    sed -e "s:-march=pentiumpro::" -e "/DEFINES += MMX/d" -i settings.pro
  else
    local cpu="`get-flag march || get-flag mcpu`"
    if [ "${cpu}" ] ; then
      sed -e "s:pentiumpro:${cpu}:g" -i "settings.pro" || die "sed failed"
    fi
  fi

(In fact it is adding three lines before and one after..)
This, and adding ~amd64 to the keywords make at least mythdvd mythweather mythgallery and mythvideo build.
Comment 2 Daniel Ahlberg (RETIRED) gentoo-dev 2004-09-11 03:48:09 UTC
Fixed in 0.16. thanks