<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 19, 2017 at 4:30 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><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"><div class="gmail_extra">I fixed things the existing way in r308548.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I would like it if we could avoid problems like this in the future when people add tests that use the optional hooks. As it stands now, they are unlikely to discover that every optional hook added to a test needs to be written down in CMakeLists.txt. We're currently doing things this way to make the source code of the tests clean and to avoid dependencies on FuzzerInterface.h. The question is, what do we want to compromise on:</div><div class="gmail_extra">1. Make tests ugly, but cross-platform, and representative of what users must do to be cross-platform</div><div class="gmail_extra">2. Maintain status quo, tests break every so often, but now maybe we'll remember why next time</div></div></blockquote><div><br></div><div>I'd really love to keep (most of) the tests independent of libFuzzer and FuzzerInterface.h so that other engines could use them. </div><div>(same applies to real-life fuzz targets). </div><div>And I would also prefer to avoid ugly ifdefs. </div><div>So, let's "Maintain status quo" for now and once the burden on Windows becomes intolerable invent some 4-th way. </div><div><br></div><div>Thanks for the fix! </div><div> </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"><div class="gmail_extra">3. Require users to include FuzzerInterface.h, introducing more dependencies, but it will allow us to magically abstract Windows portability issues</div><span class="gmail-"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 17, 2017 at 5:58 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br><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">Let me add <div>UNSUPPORTED: windows<br></div></div><div class="gmail-m_-7909311895154677609gmail-HOEnZb"><div class="gmail-m_-7909311895154677609gmail-h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 17, 2017 at 5:50 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><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">I didn't have time to get to this today. I can look at it tomorrow. In the meantime, can somebody XFAIL this?</div><div class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875HOEnZb"><div class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 17, 2017 at 5:34 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br><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">+rnk</div><div class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875m_-8508138096441484821HOEnZb"><div class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875m_-8508138096441484821h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 17, 2017 at 12:25 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The output on the bot isn't terribly informative and I don't have a<br>
windows machine. Could somebody run the failing command for me and paste<br>
the output? It's as follows:<br>
<span class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875m_-8508138096441484821m_-2164058171308100812im gmail-m_-7909311895154677609gmail-m_-2391903779577258875m_-8508138096441484821m_-2164058171308100812HOEnZb"><br>
  "LLVMFuzzer-FlagsTest" "--foo-bar" "-runs=10" "-ignore_remaining_args=1" "--baz" "-help=1" "test"<br></span><div class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875m_-8508138096441484821m_-2164058171308100812HOEnZb"><div class="gmail-m_-7909311895154677609gmail-m_-2391903779577258875m_-8508138096441484821m_-2164058171308100812h5"><br></div></div></blockquote></div></div></div></div></blockquote></div></div></div></div></blockquote></div></div></div></div></blockquote></div></div></span></div>
</blockquote></div><br></div></div>