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

Michael Adam obnox at samba.org
Thu Dec 18 15:02:15 MST 2014


On 2014-12-18 at 22:49 +0100, Stefan (metze) Metzmacher wrote:
> Am 18.12.2014 um 22:45 schrieb Michael Adam:
> > 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. :-)
> 
> I only handles -Wl,-R but not -Wl,-R,

Oh right, somehow I must have looked skewly.

> >> 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 start to understand the waf magic:-)

Great!
-------------- 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/21bfb7a0/attachment-0001.pgp>


More information about the samba-technical mailing list