I'm all for getting rid of ORWT. I've observed some wrong code paths there that are probably not even used anymore. It makes more difficult to hack on a code which almost nobody uses and whose part of it is wrong and misleading.<br>
<br>NRWT is not that easy thought, but I see the unittests as an improvement. So we can keep the same standard of the rest of webkit contributions of writing a test to keep the behavior you're introducing.<br><br>I would like to help on removing qt dependency of ORWT.<div>
<br></div><div>Regards,<br><br><div class="gmail_quote">On Fri, Jun 8, 2012 at 4:15 PM, Dirk Pranke <span dir="ltr"><<a href="mailto:dpranke@chromium.org" target="_blank">dpranke@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Ossy,<br>
<br>
Thanks for your reply ...<br>
<div class="im"><br>
On Fri, Jun 8, 2012 at 12:46 AM, Osztrogonac Csaba <<a href="mailto:oszi@inf.u-szeged.hu">oszi@inf.u-szeged.hu</a>> wrote:<br>
</div><div class="im">> Hi,<br>
><br>
> Dirk Pranke írta:<br>
><br>
>> I believe most if not all of the ports have started using either<br>
>> TestExpectations files or a combination of TestExpectations files<br>
>> (except for the Apple Win port).<br>
>><br>
>> Can we explicitly switch to the TestExpectations files at this point<br>
>> and drop support for Skipped files on the other ports (and perhaps<br>
>> disable old-run-webkit-tests for all but apple win)?<br>
><br>
><br>
> Until NRWT can't handle cascaded TestExpectations -<br>
> <a href="https://bugs.webkit.org/show_bug.cgi?id=65834" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=65834</a>,<br>
> Qt port can't drop supporting Skipped files. We have many tests skipped in<br>
> qt-5.0, qt-5.0-wk1,<br>
> qt-5.0-wk2, wk2 Skipped lists. We can't migrate all of them to the only one<br>
> TestExpectations.<br>
><br>
<br>
</div>Sorry, you're right, I should have mentioned that we would need to<br>
support cascaded files before we can do anything. As Ojan mentioned, I<br>
am actively working on that.<br>
<br>
One thing that would be helpful would be to how each port would like<br>
to support using cascading files and modifiers. Assuming each of you<br>
is free to do as you like, you can pick from several paths, e.g. you<br>
could use one file, or one file per operating system, or one file per<br>
os plus one file per version plus maybe a webkit-2 specific file<br>
(roughly what most ports do today, I think), or something else?<br>
<div class="im"><br>
<br>
> And I disagree with disabling ORWT at all. Qt port still support using ORWT<br>
> locally.<br>
> It is better for gardening than NRWT. NRWT regularly has problems with<br>
> generating<br>
> new results for a given platform dir (qt,qt-5.0,qt-5.0-wk1,...), it doesn't<br>
> support<br>
> the good --skipped=only option .<br>
<br>
</div>I am not aware of these issues ... can you explain further, or file<br>
bugs? In particular, I'm not sure I fully understand how all of the<br>
different Qt version/platform combinations work, so maybe there are<br>
issues there that can be improved?<br>
<br>
If ORWT is easier for you to use than NRWT, I would love to understand<br>
why, so that hopefully I can improve NRWT for you as well.<br>
<div class="im"><br>
> If folks don't want to use it, just not use, but disabling for everyone by fiat isn't a friendly thing.<br>
<br>
</div>I'm sorry, I was under the impression that no one (apart from Apple<br>
Win) was using ORWT any more, and I thought we had reached feature<br>
parity between the two tools (or at least that NRWT supported<br>
everything that ORWT did, obviously ORWT doesn't support reftests or a<br>
bunch of other things that NRWT does).<br>
<br>
I'm not about to remove a tool that people are still using :). That<br>
said, having two tools that are both widely used and do essentially<br>
the same thing is unfortunate, so it would be good to converge<br>
where/when/if we can, I think.<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Dirk<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Rafael Brandao @ INdT<br>
</div>