Re: [webkit-dev] Skipped files are going away
Last call on this ... I will be removing the files later today unless something goes awry :). -- Dirk On Fri, Sep 21, 2012 at 3:41 PM, Dirk Pranke dpra...@chromium.org wrote: Hi all, Now that we're on the new TestExpectations syntax, I'm planning to remove the Skipped files (and support for them from NRWT). I have a change posted that will add minimal support for the syntax to old-run-webkit-tests in https://bugs.webkit.org/show_bug.cgi?id=97276 ; basically any file or directory listed in a TestExpectations file will be skipped *regardless of the expectation listed*. I can make things smarter if need be, but it's not clear to me how many people if any are still using ORWT so the need doesn't seem to be there. Please comment on the bug ASAP if you disagree and would like ORWT to be smarter in this area :). Once that change lands, my plan is to copy all of the entries from the existing Skipped files over into their corresponding TestExpectations files, and then delete the Skipped files, and then delete the code that supports the Skipped files in NRWT and ORWT. I hope to do this early next week. If you have any objections, now would be a good time to speak up! Note that all of this is being done in the oft-requested name of simplifying the number of different ways we have to handle failing tests. Also, I will remind people that the new TestExpectations syntax is a superset of the old Skipped file syntax, so there shouldn't be any good reason to keep the old files around. However, if you know something I don't, please mention it :). Thanks, -- Dirk ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo/webkit-dev
Re: [webkit-dev] Skipped files are going away
Now landed in http://trac.webkit.org/changeset/129788 . Let me know if there are any problems. -- Dirk On Thu, Sep 27, 2012 at 10:15 AM, Dirk Pranke dpra...@chromium.org wrote: Last call on this ... I will be removing the files later today unless something goes awry :). -- Dirk On Fri, Sep 21, 2012 at 3:41 PM, Dirk Pranke dpra...@chromium.org wrote: Hi all, Now that we're on the new TestExpectations syntax, I'm planning to remove the Skipped files (and support for them from NRWT). I have a change posted that will add minimal support for the syntax to old-run-webkit-tests in https://bugs.webkit.org/show_bug.cgi?id=97276 ; basically any file or directory listed in a TestExpectations file will be skipped *regardless of the expectation listed*. I can make things smarter if need be, but it's not clear to me how many people if any are still using ORWT so the need doesn't seem to be there. Please comment on the bug ASAP if you disagree and would like ORWT to be smarter in this area :). Once that change lands, my plan is to copy all of the entries from the existing Skipped files over into their corresponding TestExpectations files, and then delete the Skipped files, and then delete the code that supports the Skipped files in NRWT and ORWT. I hope to do this early next week. If you have any objections, now would be a good time to speak up! Note that all of this is being done in the oft-requested name of simplifying the number of different ways we have to handle failing tests. Also, I will remind people that the new TestExpectations syntax is a superset of the old Skipped file syntax, so there shouldn't be any good reason to keep the old files around. However, if you know something I don't, please mention it :). Thanks, -- Dirk ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo/webkit-dev
Re: [webkit-dev] Skipped files are going away
Sorry for the long stream of messages on this ... in case it's not clear, all I did with this change was cut and paste the entries from one file into the other (since the TestExpectations file syntax is a superset of the Skipped syntax) and do some minimal de-duping so that run-webkit-tests --lint-test-files would pass. Different ports use different organization methods for these files, and I didn't attempt to follow each port, nor reorganize entries that were being skipped but really should've been failures or flakes or something else. I leave each port to do their own cleanup, but of course I'll be happy to troubleshoot/answer questions, etc :). -- Dirk On Thu, Sep 27, 2012 at 12:24 PM, Dirk Pranke dpra...@chromium.org wrote: Now landed in http://trac.webkit.org/changeset/129788 . Let me know if there are any problems. -- Dirk On Thu, Sep 27, 2012 at 10:15 AM, Dirk Pranke dpra...@chromium.org wrote: Last call on this ... I will be removing the files later today unless something goes awry :). -- Dirk On Fri, Sep 21, 2012 at 3:41 PM, Dirk Pranke dpra...@chromium.org wrote: Hi all, Now that we're on the new TestExpectations syntax, I'm planning to remove the Skipped files (and support for them from NRWT). I have a change posted that will add minimal support for the syntax to old-run-webkit-tests in https://bugs.webkit.org/show_bug.cgi?id=97276 ; basically any file or directory listed in a TestExpectations file will be skipped *regardless of the expectation listed*. I can make things smarter if need be, but it's not clear to me how many people if any are still using ORWT so the need doesn't seem to be there. Please comment on the bug ASAP if you disagree and would like ORWT to be smarter in this area :). Once that change lands, my plan is to copy all of the entries from the existing Skipped files over into their corresponding TestExpectations files, and then delete the Skipped files, and then delete the code that supports the Skipped files in NRWT and ORWT. I hope to do this early next week. If you have any objections, now would be a good time to speak up! Note that all of this is being done in the oft-requested name of simplifying the number of different ways we have to handle failing tests. Also, I will remind people that the new TestExpectations syntax is a superset of the old Skipped file syntax, so there shouldn't be any good reason to keep the old files around. However, if you know something I don't, please mention it :). Thanks, -- Dirk ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo/webkit-dev
[webkit-dev] Skipped files are going away
Hi all, Now that we're on the new TestExpectations syntax, I'm planning to remove the Skipped files (and support for them from NRWT). I have a change posted that will add minimal support for the syntax to old-run-webkit-tests in https://bugs.webkit.org/show_bug.cgi?id=97276 ; basically any file or directory listed in a TestExpectations file will be skipped *regardless of the expectation listed*. I can make things smarter if need be, but it's not clear to me how many people if any are still using ORWT so the need doesn't seem to be there. Please comment on the bug ASAP if you disagree and would like ORWT to be smarter in this area :). Once that change lands, my plan is to copy all of the entries from the existing Skipped files over into their corresponding TestExpectations files, and then delete the Skipped files, and then delete the code that supports the Skipped files in NRWT and ORWT. I hope to do this early next week. If you have any objections, now would be a good time to speak up! Note that all of this is being done in the oft-requested name of simplifying the number of different ways we have to handle failing tests. Also, I will remind people that the new TestExpectations syntax is a superset of the old Skipped file syntax, so there shouldn't be any good reason to keep the old files around. However, if you know something I don't, please mention it :). Thanks, -- Dirk ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo/webkit-dev