Hi,

Thanks Joseph for the review.

> The driver changes are OK.
> 
> I think the new configure options and the new -nodefaultrpaths compiler 
> option need documenting

Doc patch was added, and okay’ed by Iain.


> (I suppose there might be a case for the configure 
> option defined in libtool code being documented somewhere in libtool, if 
> there's somewhere appropriate, but I don't see that in the libtool patch 
> submission).

I will follow up with that when I ping upstream libtool, in a couple of weeks.


> The help text for --enable-darwin-at-rpath refers to it as 
> --enable-darwin-at-path.

Fixed, and everything regenerated. Also, I removed a bit of historical dead-code (inside #ifdef RPATH_SETS_NODEFAULT, where RPATH_SETS_NODEFAULT was not defined anymore in the last version of the patch).


Attached is the final patchset. Joseph, is your previous OK good for pushing, or does it need further review?

Thanks,
FX