Bug 159395 – Rename run-webkit-httpd to run-webkit-test-servers

• Status: NEW
• Component: Tools / Tests

[youenn fablet]

This script is not only launching https but also web socket and web-platform-test servers

[youenn fablet]

Created attachment 282682 [details]
Patch

[youenn fablet]

Follow-up to bug 152486

[Alex Christensen]

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.

[youenn fablet]

Would run-test-servers work?
It only tab-conflicts with run-test-runner.

[Daniel Bates]

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.

[youenn fablet]

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.

[Ryosuke Niwa]

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.

[youenn fablet]

I also spotted Tools/Scripts/webkitperl/httpd.pm.
Is it used somewhere?