<div dir="auto">That sounds good to me.</div><div dir="auto"><br></div><div dir="auto">The reason why I’m requesting to not always inject -rpath in the manner is due to <div dir="auto">Bug 49199 to workaround this also requires adding some -rpath configurations directly into LDFLAGS but that’s ugly.</div><div dir="auto"><br></div><div dir="auto">Instead having UNIX_LDFLAGS for example on macOS default to passing the needed -rpath,@loader_path/ and allows adding additional paths after this.</div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 20, 2021 at 11:01 AM Huw Davies <<a href="mailto:huw@codeweavers.com">huw@codeweavers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On Fri, Aug 20, 2021 at 10:13:01AM -0400, Dean Greer wrote:<br>
> This makes sense now thank you, I’d still rather see the -rpath behind<br>
> something like UNIX_LDFLAGS then it could be used for other -rpath<br>
> configuration for the Unix.so libraries <br>
> <br>
> For the build directory runs wouldn’t it be better to alter the wine loader<br>
> wrapper script.<br>
<br>
Right, good idea.  Adding $topdir/dlls/ntdll to DYLD_LIBRARY_PATH in the<br>
wine script will cover the build directory case.<br>
<br>
So that leaves the installed case.  The simplest approach would be to<br>
change the install-name to @loader_path/module.so and skip rpaths<br>
entirely.  Is there a reason not to do that?<br>
<br>
Huw.<br>
</blockquote></div></div>