[PATCHEs] wafsamba: fix ordering problems with lib-provided and internal RPATHs

Michael Adam obnox at samba.org
Thu Dec 18 14:45:40 MST 2014


On 2014-12-18 at 22:16 +0100, Stefan (metze) Metzmacher wrote:
> Am 18.12.2014 um 22:13 schrieb Michael Adam:
> > 
> > this is nice, but the patch you want to revert from wafadmin is
> > upstream, so it would be wrong to revert it. It was also fixed
> > in newer waf versions. I'd also argue that your fix is not
> > necessarily a better fix, since the bug in question is a proper
> > bug in wafadmin. But it is nice that you found a way to do it in
> > wafsamba that is not too complicated! We can add that
> > additionally if you want.
> > 
> > And I can see whether I can use the ideas for modifying our
> > "wellknown libpaths" to not touch wafadmin.
> 
> Ah, ok then we just should not push the revert patch, but keep
> the otherone to handle '-Wl,-R,' too.

I don't quite understand: Your patch to samba treats nothing
the original wafadmin patch does not catch. It even uses the
same checks. :-)

> The wellknown libpath patch is attached:-)

Hey, you've been quicker than me, as usual.. ;)

I see that the actual code that retrieves the
standard libpaths is the same as the one I sent a
few weeks ago and that we had worked on together.
You have only found a better place to make use of it. :-)

I will test it.

Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20141218/7146fa13/attachment.pgp>


More information about the samba-technical mailing list