On Wed, 2005-08-10 at 12:46 +0000, Ferris McCormick wrote:
> On Tue, 2005-08-09 at 19:40 -0700, Donnie Berkholz wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> > 
> 
> > Good catch. I was going to get to the rest of the GL headers after
> > getting Mesa working.
> 
> Another comment about Mesa.  Mesa, as distributed, has "built in"
> architectures it can build for, and it substitutes its own idea of
> appropriate CFLAGS rather than using the user's request.  For sparc, at
> least, these are not appropriate, as can be seen from current
> xorg-x11-6.8.99.15, say, for which libGL does get built using
> make.conf's CFLAGS.
> 
OK, now I have looked at the X modular ebuild for mesa-6.3.1.1, and half
of what I said is irrelevant.
1.  The good news:  CFLAGS are fine for X11R7 libGL build;
2.  But to get sparc assembler built correctly, the following patch
seems correct:
================================
--- /home/cvsroot/gentoo-x86/media-libs/mesa/mesa-6.3.1.1.ebuild
2005-08-10 06:42:09.000
000000 +0000
+++ mesa-6.3.1.1.ebuild 2005-08-10 15:03:22.000000000 +0000
@@ -17,7 +17,7 @@
        http://xorg.freedesktop.org/extras/${LIBDRM_P}.tar.gz";
 LICENSE="LGPL-2"
 SLOT="0"
-KEYWORDS="~x86"
+KEYWORDS="~x86 ~sparc"
 IUSE="motif"
 
 RDEPEND="dev-libs/expat
@@ -62,6 +62,10 @@
 
        # Set up linux-dri configs
        echo "OPT_FLAGS = ${CFLAGS}" >> ${HOSTCONF}
+       if use sparc; then
+               echo "ASM_FLAGS = -DUSE_SPARC_ASM" >> ${HOSTCONF}
+               echo 'ASM_SOURCES = $(SPARC_SOURCES) $(SPARC_API)' >>
${HOSTCONF}
+       fi
        echo "CC = $(tc-getCC)" >> ${HOSTCONF}
        echo "CXX = $(tc-getCXX)" >> ${HOSTCONF}
        echo "DRM_SOURCE_PATH=\$(TOP)/../${LIBDRM_P}" >> ${HOSTCONF}
@@ -72,7 +76,7 @@
        # Documented in configs/default
        if use motif; then
                # Add -lXm
-               echo "GLW_LIB_DEPS = -L$(LIB_DIR) -l$(GL_LIB)
$(EXTRA_LIB_PATH) -lXt -lX11 -lXm
" >> ${HOSTCONF}
+               echo 'GLW_LIB_DEPS = -L$(LIB_DIR) -l$(GL_LIB)
$(EXTRA_LIB_PATH) -lXt -lX11 -lXm
' >> ${HOSTCONF}
============================================
(Alternative is to create a complete linux-dri-sparc config file; I can
do that if you like.)

3.  I changed the double-quoted echo at GLW_LIB ... to single-quoted
echo, because I think we need to pass in the $(LIB_DIR), etc. as-is (at
least, the ebuild doesn't know what they are and portage complains
otherwise).

I've also attached the little strawman patch to this note, to make it
easier to read.  I can file a Bug for the change if you like.

Regards,
Ferris

-- 
Ferris McCormick (P44646, MI) <[EMAIL PROTECTED]>
Developer, Gentoo Linux (Sparc, Devrel)
--- /home/cvsroot/gentoo-x86/media-libs/mesa/mesa-6.3.1.1.ebuild        
2005-08-10 06:42:09.000000000 +0000
+++ mesa-6.3.1.1.ebuild 2005-08-10 15:03:22.000000000 +0000
@@ -17,7 +17,7 @@
        http://xorg.freedesktop.org/extras/${LIBDRM_P}.tar.gz";
 LICENSE="LGPL-2"
 SLOT="0"
-KEYWORDS="~x86"
+KEYWORDS="~x86 ~sparc"
 IUSE="motif"
 
 RDEPEND="dev-libs/expat
@@ -62,6 +62,10 @@
 
        # Set up linux-dri configs
        echo "OPT_FLAGS = ${CFLAGS}" >> ${HOSTCONF}
+       if use sparc; then
+               echo "ASM_FLAGS = -DUSE_SPARC_ASM" >> ${HOSTCONF}
+               echo 'ASM_SOURCES = $(SPARC_SOURCES) $(SPARC_API)' >> 
${HOSTCONF}
+       fi
        echo "CC = $(tc-getCC)" >> ${HOSTCONF}
        echo "CXX = $(tc-getCXX)" >> ${HOSTCONF}
        echo "DRM_SOURCE_PATH=\$(TOP)/../${LIBDRM_P}" >> ${HOSTCONF}
@@ -72,7 +76,7 @@
        # Documented in configs/default
        if use motif; then
                # Add -lXm
-               echo "GLW_LIB_DEPS = -L$(LIB_DIR) -l$(GL_LIB) $(EXTRA_LIB_PATH) 
-lXt -lX11 -lXm" >> ${HOSTCONF}
+               echo 'GLW_LIB_DEPS = -L$(LIB_DIR) -l$(GL_LIB) $(EXTRA_LIB_PATH) 
-lXt -lX11 -lXm' >> ${HOSTCONF}
                # Add GLwMDrawA.c
                echo "GLW_SOURCES = GLwDrawA.c GLwMDrawA.c" >> ${HOSTCONF}
        fi

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to