Summary: | Rename run-webkit-httpd to run-webkit-test-servers | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | youenn fablet <youennf> | ||||
Component: | Tools / Tests | Assignee: | youenn fablet <youennf> | ||||
Status: | NEW --- | ||||||
Severity: | Normal | CC: | ap, dbates, gsnedders, lforschler, rniwa | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=152486 | ||||||
Attachments: |
|
Description
youenn fablet
2016-07-03 23:31:17 PDT
Created attachment 282682 [details]
Patch
Follow-up to bug 152486 Comment on attachment 282682 [details]
Patch
Is this really necessary? This makes tab completion even worse. If it's renamed, I think it should be a different name. Right now I have Tools/Scripts in my PATH, so when I want to run-webkit-tests, I just type run-w<tab>-t<tab>. This would make that slower.
Would run-test-servers work? It only tab-conflicts with run-test-runner. I do not feel that the name of the script, run-webkit-httpd, is ambiguous enough to necessitate a rename. Although it could be claimed that the name of this script is a bit disingenuous with respect to starting a web socket socket (Web Socket does use HTTP for its handshake), I do not feel that this script is ambiguous with respect to starting the web platform test server as the web platform test server is an HTTP server. And I do not feel that the name of this script is disingenuous enough in the Web Socket case to necessitate a rename. With respect to starting the web platform test server, any confusion is likely from the script's historic lack of support for starting the web platform test server. Following the fix for bug #152486, run-webkit-httpd supports starting the web platform test server. We should consider announcing such support on webkit-dev. This announcement may lead to feedback about the name of the script run-webkit-httpd. I'll send a mail. To be noted that there are also run-webkit-websocketserver, new-run-webkit-httpd and new-run-webkit-websocketserver scripts. It might be good to remove some of them. I don't like the new name "run-webkit-test-servers" because "test severs" don't tell me what kind of server it is given there's also web socket server, etc... If anything, I'd call it run-webkit-test-httpd-servers but that might be too long. I think the status quo: run-webkit-httpd is good enough. I also spotted Tools/Scripts/webkitperl/httpd.pm. Is it used somewhere? |