Summary: | [NRWT] the nrwt check httpd with --no-http option | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kristóf Kosztyó <kkristof> | ||||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | dpranke, ossy, tony, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 67847 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Kristóf Kosztyó
2011-09-09 01:38:00 PDT
Created attachment 106850 [details]
proposed fix
Comment on attachment 106850 [details]
proposed fix
Nice catch, r=me.
Unfortunately I got same error message without --no-http if I don't have apache. Could you file another bug report about it? It would be better if NRWT exits with a nicer mode. Comment on attachment 106850 [details] proposed fix Clearing flags on attachment: 106850 Committed r94851: <http://trac.webkit.org/changeset/94851> All reviewed patches have been landed. Closing bug. It broke a unittest: Traceback (most recent call last): File "/ramdisk/qt-linux-release/build/Tools/Scripts/webkitpy/layout_tests/controllers/manager_unittest.py", line 239, in test_needs_servers self.assertTrue(manager.needs_servers()) File "/ramdisk/qt-linux-release/build/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 820, in needs_servers return any(self._test_requires_lock(test_name) for test_name in self._test_files) and self._options.http AttributeError: 'MockOptions' object has no attribute 'http' Created attachment 106867 [details]
fix for the test-webkitpy
Comment on attachment 106867 [details]
fix for the test-webkitpy
Argh, it is an ugly hack. :-/ We should find a proper fix for it.
Dirk, Tony, could you give us some hint how should we fix this webkitpy test in a proper way? Original patch was rolled out by http://trac.webkit.org/changeset/94853 Created attachment 106871 [details]
proposed fix
Indeed it was an ugly hack, maybe this will be better.
Comment on attachment 106871 [details] proposed fix Clearing flags on attachment: 106871 Committed r94863: <http://trac.webkit.org/changeset/94863> All reviewed patches have been landed. Closing bug. Comment on attachment 106871 [details]
proposed fix
Hm. Ideally you'd pass in the http flag to get_manager_with_tests and add a third test to make sure that the right thing happens with http=False as well.
|