<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 3, 2016 at 9:34 AM, Robinson, Paul <span dir="ltr"><<a href="mailto:paul.robinson@sony.com" target="_blank">paul.robinson@sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
> -----Original Message-----<br>
> From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@lists.llvm.org</a>] On Behalf<br>
> Of Sean Silva via llvm-commits<br>
> Sent: Monday, May 02, 2016 5:36 PM<br>
> To: <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> Subject: [lld] r268345 - Temporarily disable this test on llvm-clang-lld-<br>
> x86_64-scei-ps4-windows10pro-fast<br>
><br>
> Author: silvas<br>
> Date: Mon May  2 19:36:15 2016<br>
> New Revision: 268345<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=268345&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=268345&view=rev</a><br>
> Log:<br>
> Temporarily disable this test on llvm-clang-lld-x86_64-scei-ps4-<br>
> windows10pro-fast<br>
><br>
> `REQUIRES: shell` is not appropriate because that would mean that there<br>
> are no windows bots testing this, and that is precisely where it needs<br>
> the most testing.<br>
><br>
> Rafael or Rui are working on generating an archive directly, which<br>
> should avoid this issue.<br>
> We can try to move the bot to a shorter build directory path.<br>
<br>
</span>If you "fix" the bot to use short paths, we'll never find these things...<br>
The short/long path thing is just a fact of life on Windows and we do<br>
need to tolerate it.<br></blockquote><div><br></div><div>I tend to agree. Hopefully Rafael's archive patch will actually solve the issue.</div><div><br></div><div>There is a bit of a slippery slope though, as every test needs *some* amount of file path length slack. The question is how much is reasonable.</div><div>For example, we could put the build dir at a path of length `PATH_MAX - 1`. Obviously then we won't be able to even create `$BUILDDIR/lib/` or whatnot. Ultimately we might settle at `PATH_MAX - 42` which is enough for the build (`42` is just for the sake of choosing a number; I don't know the actual number). So then does this mean that our tests basically assume that we have 42 characters of slack on customer machines?</div><div><br></div><div>I'm not sure what the right tradeoff is. In the case of a test like this that essentially "doubles" the number of characters in a path, it is probably clear that we need to be careful about it. But what about a test that needs 43 extra characters? 47 extra characters? 127 extra characters?</div><div><br></div><div>Thoughts?</div><div><br></div><div>-- Sean Silva</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--paulr<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Modified:<br>
>     lld/trunk/test/ELF/reproduce.s<br>
><br>
> Modified: lld/trunk/test/ELF/reproduce.s<br>
> URL: <a href="http://llvm.org/viewvc/llvm-" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-</a><br>
> project/lld/trunk/test/ELF/reproduce.s?rev=268345&r1=268344&r2=268345&view<br>
> =diff<br>
> ==========================================================================<br>
> ====<br>
> --- lld/trunk/test/ELF/reproduce.s (original)<br>
> +++ lld/trunk/test/ELF/reproduce.s Mon May  2 19:36:15 2016<br>
> @@ -1,5 +1,10 @@<br>
>  # REQUIRES: x86<br>
><br>
> +# XXX: Temporary hack to work around windows path length limitation due<br>
> to<br>
> +# the build dir for llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast.<br>
> +# When we directly generate an archive this won't be an issue.<br>
> +# XFAIL: x86_64-scei-ps4<br>
> +<br>
>  # RUN: rm -rf %t.dir<br>
>  # RUN: mkdir -p %t.dir/build1<br>
>  # RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o<br>
> %t.dir/build1/foo.o<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>