<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 29, 2016 at 1:25 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> Yes, this test case passes on Darwin (i.e, with `"/"`). However, it does not pass with `` or `""` on Darwin.<br>
><br>
> I'm a bit curious about this. Wouldn't this test always fail if we pass a directory name instead of a file name?<br>
<br>
</span>Yes, this was my intent. It lets us check the verbose logging.<br>
<span class=""><br>
<br>
> 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 `LLVM_PROFILE_FILE=%t/` or something. Does that work for you?<br>
<br>
</span>Works for me, I'll go ahead and make this change if no one objects soon.<br></blockquote><div><br></div><div><br></div><div>Thanks!</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
vedant<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
><br>
><br>
> -- Sean Silva<br>
><br>
><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>
> Ok, fair enough.<br>
><br>
> vedant<br>
><br>
<br>
</div></div></blockquote></div><br></div></div>