<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 13, 2019 at 4:22 PM Ken Cunningham <<a href="mailto:ken.cunningham.webuse@gmail.com">ken.cunningham.webuse@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><blockquote type="cite"><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)">><i> I did not want to touch that at this point. The gcc bug report is still
</i>><i> open, as far as I can see.
</i>><i>
</i>><i> If someone wants try a build using —with-build-sysroot instead, they are
</i>><i> very welcome to try...
</i>><i>
</i>><i> Cheers chris
</i>><i>
</i>
Chris,
     I'll try your changes with that hunk removed. I don't see why it
shouldn't bootstrap with only that patch applied as the xgcc and final gcc
should now all look via xcrun for the SDK location.
             Jack
</pre></blockquote><div><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)"><br></pre><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)"><br></pre><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)">If I might ask for a little testing on all the systems we support before we do that. </pre><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)">Although I have been pushing the SDKROOT business for a while now, for all recent systems, we have no idea yet how this might affect bootstrapping all the older systems we support.</pre><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)">I’d hate to have to go back and just put this all back in again :></pre><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)"><br></pre><pre style="white-space:pre-wrap;background-color:rgb(255,255,255)">Ken</pre></div></div></blockquote><div><br></div>Actually, removing the section breaks the bootstrap. What is currently unimplemented in the upstream changes is an additional fallback layer where, when SDKROOT is unset, the compiler would execute `xcrun --show-sdk-path` in order to set it. Apple's approach is to use compiler wrappers in /usr/bin which call the actual compilers via xcrun. Iain Sandoe has objected to that approach in the past as resulting in unwanted overhead in compiles, but I think if it were just the final fallback position of the compiler that wouldn't be much of a performance hit. Also, note that if you execute the buried clang compilers in /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin or /Library/Developer/CommandLineTools/usr/bin that they can't find the SDK.</div><div class="gmail_quote">              Jack</div><div class="gmail_quote"><br></div></div></div></div></div>