[Scons-users] version 2.5.0 and SHLIBVERSIONFLAGS

[Tim Jenness]

Thank you! I was just about to sit down and disentangle our system.

I am working around the problem for now by just forcing SHLIBVERSIONFLAGS to “”.

— 
Tim Jenness


> On May 10, 2016, at 16:47 , Paweł Tomulik <ptomulik at meil.pw.edu.pl> wrote:
> 
> W dniu 10.05.2016 o 23:30, Tim Jenness pisze:
>> I have a scons package that breaks badly in v2.5.0 on Linux because scons started adding $SHLIBVERSIONFLAGS (which really means “-Wl,Bsymbolic” is added to the g++ link line) to my LoadableModule targets (which happen to be SWIG generated files). I’m definitely not setting SHLIBVERSION and SharedLibraryModule doesn’t have the problem. Is it possible that there is a problem with LoadableModule not correctly working out that there is no shared library version defined? If I clear SHLIBVERSIONFLAGS in my SConstruct file the build works as before.
>> 
>> Should I file a ticket on this? Am I setting a SHLIBVERSION specifically for LoadableModule without realising it?
>> 
>> — 
>> Tim Jenness
>> _______________________________________________
>> Scons-users mailing list
>> Scons-users at scons.org
>> https://pairlist4.pair.net/mailman/listinfo/scons-users
>> 
> 
> I can confirm, it's the case :(
> 
> The minimal example:
> 
> # Sconstruct
> env = Environment()
> env.LoadableModule('foo', 'foo.c')
> 
> 
> The issue exists because env['LDMODULEVERSION'] is initially set to
> "$SHLIBVERSION" (non-empty string), but this (non empty string
> '$SHLIBVERSION') is not correctly handled by a function called
> __libversionflags in SCons/Defaults.py (the variable is not
> substituted). This is a bug, but fixing it shouldn't be hard.
> 
> 
> A quick workaround may consist of overwritting env['__libversionflags']
> as follows:
> 
> 
> def __libversionflags(env, version_var, flags_var):
>  try:
>    if env.subst('$'+version_var):
>      return env[flags_var]
>  except KeyError:
>    pass
>  return None
> 
> env = Environment(__libversionflags = __libversionflags)
> env.LoadableModule('foo', 'foo.c')
> 
> 
> Similar fixed implementation of __libversionflags should go to
> Defaults.py to fix the bug.
> 
> Best regards!
> -- 
> Pawel Tomulik
> _______________________________________________
> Scons-users mailing list
> Scons-users at scons.org
> https://pairlist4.pair.net/mailman/listinfo/scons-users