Re: [Mesa3d-dev] softpipe/llvmpipe build regression

[Xavier C. <cha...@gm...>]

On Sun, Mar 21, 2010 at 6:23 PM, George Sapountzis
<gsa...@gm...> wrote:
> On Sun, Mar 21, 2010 at 6:50 PM, George Sapountzis
> <gsa...@gm...> wrote:
>> Can you please try a clean build ?
>>
>> softpipe_create_screen is defined in the newly added file
>> soft_screen.c and maybe you did not rebuild drm/sw
>>
>> If I actually broke this as well, admittedly I should have stayed away
>> from the computer today
>>
>>
>> On Sun, Mar 21, 2010 at 6:42 PM, Xavier Chantry
>> <cha...@gm...> wrote:
>>> scons dri=no drivers=softpipe or llvmpipe
>>> glxgears: symbol lookup error:
>>> /home/xavier/app/mesa/build/linux-x86_64/lib/libGL.so.1: undefined
>>> symbol: gallium_soft_create_screen
>>>
>
> ah, this is the scons build, should be fixed in master now.
>
> sorry for that, it seems that I cannot read in addition to type today ...
>

Thanks that did it.

it seems scons would also benefit from -Wl,-no-undefined , but it
might also break the dri builds
(see [PATCH] dri: test whether the built drivers have unresolved symbols)

I guess I could just define LDFLAGS locally or use that simple patch :

diff --git a/SConstruct b/SConstruct
index 6ed44dd..304d73e 100644
--- a/SConstruct
+++ b/SConstruct
@@ -74,6 +74,8 @@ if os.environ.has_key('CXXFLAGS'):
 if os.environ.has_key('LDFLAGS'):
        env['LINKFLAGS'] += SCons.Util.CLVar(os.environ['LDFLAGS'])

+env['LINKFLAGS'].append("-Wl,-no-undefined")
+
 Help(opts.GenerateHelpText(env))

 # replicate options values in local variables