<div dir="ltr">Following up.<div><br></div><div>TL;DR: it's pointless. Spaces in paths are not safe and do not work. Even if we were to make our own scripts safe - which the feature tree above does - configure will barf if there are spaces in the source tree path, and libtool will fail assembling a safe compile command if there are spaces in the build path.</div><div><div><br></div><div>The feature branch is at <a href="https://github.com/kinkie/squid/tree/securify-test-builds">https://github.com/kinkie/squid/tree/securify-test-builds</a></div><div></div></div><div><br></div><div>This was tested on MacOS Catalina with</div><div>- autoconf 2.69</div><div>- libtool 2.4.6</div><div>- gmake 3.81</div><div>- clang 11</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 18, 2020 at 8:40 AM Francesco Chemolli <<a href="mailto:gkinkie@gmail.com">gkinkie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi all,<div>  as a followup to discussion on PR 555, I was looking into safely handling source code paths with spaces.</div><div><br></div><div>Turns out worrying is pointless: after doing the needed changes, doing a test build with</div><div>builddir = "/Users/kinkie/src/test build with space"</div><div>srcdir = "/Users/kinkie/src/squid dir with space"</div><div>on MacOS catalina with homebrew</div><div><br></div><div>resulted in configure failing with error:</div><div>checking whether build environment is sane... configure: error: unsafe srcdir value: '/Users/kinkie/src/squid dir with space/test-suite/..'<br clear="all"><div><br></div><div>If there is no regression with my changes, I'll submit the PR anyway, but its benefits will be limited.</div><div><br></div>-- <br><div dir="ltr">    Francesco</div></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">    Francesco</div>