<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 29, 2016 at 1:11 PM, Vedant Kumar <span dir="ltr"><<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><br>
> On Mar 29, 2016, at 1:10 PM, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, Mar 29, 2016 at 1:03 PM, Vedant Kumar <<a href="mailto:vsk@apple.com">vsk@apple.com</a>> wrote:<br>
><br>
> > On Mar 29, 2016, at 1:00 PM, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
> ><br>
> ><br>
> ><br>
> > On Tue, Mar 29, 2016 at 12:55 PM, Vedant Kumar <<a href="mailto:vsk@apple.com">vsk@apple.com</a>> wrote:<br>
> > Darwin is funny. I tried an empty string also, but the test failed :(.<br>
> ><br>
> > Interesting. Are the quotes necessary? For the record, for PS4, both `"/"` (your patch), `/`, and `` (i.e. empty) work.<br>
><br>
> It actually fails (i.e, proceeds with the write) with and without the quotes.<br>
><br>
> Sorry, maybe I should clarify. For my testing on PS4, this test works fine with either `"/"` (your patch), `/`, and `` (i.e. empty). Did you intend to say that `"/"` works (i.e. the test case passes) on Darwin? I'm not sure what to make of your statement "with and without the quotes".<br>
<br>
</span>Yes, this test case passes on Darwin (i.e, with `"/"`). However, it does not pass with `` or `""` on Darwin.<br></blockquote><div><br></div><div>I'm a bit curious about this. Wouldn't this test always fail if we pass a directory name instead of a file name? Then we would call `fopen("some-directory-name/", ...)` which should fail even as root. It seems a bit less magical for the test to use `<span style="font-size:12.8px">LLVM_PROFILE_FILE=</span>%t/` or something. Does that work for you?</div><div><br></div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><br>
<br>
> > That said, it's important the test is portable. I'll revert and try again if the Windows bots have issues.<br>
> ><br>
> > I haven't tested a windows-targeting configuration (and don't know if we have a bot for that). The closest I have is windows-hosted targeting PS4.<br>
><br>
> Hm, it could be that the public Windows bots have never run check-profile. The old version of the code used chmod etc. without a `REQUIRES: shell` line.<br>
><br>
> Should we add a require line to avoid breaking future Windows bots?<br>
><br>
> Let's see if anything breaks first.<br>
<br>
</span>Ok, fair enough.<br>
<span class=""><font color="#888888"><br>
vedant</font></span></blockquote></div><br></div></div>