Patch attached. Don't know if this is consistent with your philosophy
for handling CFLAGS/CXXFLAGS, but it works for me on x86 and amd64.
Cheers,

Joel

On Wed, May 04, 2005 at 10:26:10AM +0000, Simon Kenyon wrote:
> On Tuesday 03 May 2005 14:45, Joel David Elkins wrote:
> > Thanks for sending these out. Mine were too crufty, I've replaced them
> > with yours.
> >
> > I noticed that at the plugins (mythvideo at least) don't set CFLAGS in
> > settings.pro. This caused problems for me on amd64. Had to fix up the
> > eclass a bit to accomodate this. Also, where are the missing themes
> > (Titivillus etc.)?  Overall nice work, thanks again.
> 
> have not done the themes stuff yet (see later message - or earlier depending 
> on your point of view)
> if you have a fix to make the ebuild play better on an amd64 please let me 
> know and i'll add it. i intend to put myth on such a box when i work up the 
> courage (and budget). i know they are cheap, but with 4 combined 
> backends/frontends and one laptop frontend i need a good excuse to get 
> another.
> --
> simon
> _______________________________________________
> mythtv-dev mailing list
> [email protected]
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

-- 
Joel Elkins <[EMAIL PROTECTED]>
(alternate) <[EMAIL PROTECTED]>
--- /home/jde/myth-portage/eclass/myth-cvs.eclass       2005-04-30 
07:21:01.000000000 -0500
+++ portage/eclass/myth-cvs.eclass      2005-05-01 08:07:44.000000000 -0500
@@ -1,6 +1,6 @@
 # Copyright 1999-2004 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: $
+# $Header: /var/cvsroot/portage/eclass/myth-cvs.eclass,v 1.4 2005/05/01 
04:03:46 jde Exp $
 
 inherit cvs eutils flag-o-matic multilib versionator toolchain-funcs
 
@@ -43,22 +43,28 @@
 
        cvs_src_unpack ${A} ; cd ${S}
 
-       if use debug ; then
-               FEATURES="${FEATURES} nostrip"
+       if [ -f 'settings.pro' ]; then
+               if use debug ; then
+                       FEATURES="${FEATURES} nostrip"
+                       sed \
+                               -e 's:#CONFIG += debug:CONFIG += debug:' \
+                               -e 's:CONFIG += release:#CONFIG += release:' \
+                               -i 'settings.pro' || die "Setting debug failed"
+               fi
+
                sed \
-                       -e 's:#CONFIG += debug:CONFIG += debug:' \
-                       -e 's:CONFIG += release:#CONFIG += release:' \
-                       -i 'settings.pro' || die "Setting debug failed"
-       fi
+                       -e "s:QMAKE_CXXFLAGS_RELEASE =.*:QMAKE_CXXFLAGS_RELEASE 
= ${CXXFLAGS}:" \
+                       -i 'settings.pro' || die "Settings 
QMAKE_CXXFLAGS_RELEASE failed"
 
-       sed \
-               -e "s:PREFIX = /usr/local:PREFIX = /usr:" \
-               -i 'settings.pro' || die "Settings PREFIX failed"
+               sed \
+                       -e "s:PREFIX = /usr/local:PREFIX = /usr:" \
+                       -i 'settings.pro' || die "Settings PREFIX failed"
+       fi
 
        if atleast_0.18 && hasq ${_MODULE} ${MYTHPLUGINS} ; then
                cd ${_MODULE}
        fi
-                                                                               
                                                                                
        if !  use nls ; then
+       if !  use nls ; then
                sed \
                        -e "s:i18n::" \
                        -i ${pkg_pro} || die "Disable i18n failed"
_______________________________________________
mythtv-dev mailing list
[email protected]
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

Reply via email to