Remove run_webkit_tests dependency on third_party\python_24....

Remove run_webkit_tests dependency on third_party\python_24.

R=dpranke,jeffbailey@google.com
BUG=none
TEST=build bot green


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=43224

[M-A Ruel, 2010-03-29 21:05:02]

http://codereview.chromium.org/1568004/diff/2001/3001
File webkit/tools/layout_tests/run_webkit_tests.bat (right):

http://codereview.chromium.org/1568004/diff/2001/3001#newcode1
webkit/tools/layout_tests/run_webkit_tests.bat:1: python
%~dp0\run_webkit_tests.py %*
keep the @

[Victor Wang, 2010-03-29 21:13:44]

Done

On 2010/03/29 21:05:02, Marc-Antoine Ruel wrote:
> http://codereview.chromium.org/1568004/diff/2001/3001
> File webkit/tools/layout_tests/run_webkit_tests.bat (right):
> 
> http://codereview.chromium.org/1568004/diff/2001/3001#newcode1
> webkit/tools/layout_tests/run_webkit_tests.bat:1: python
> %~dp0\run_webkit_tests.py %*
> keep the @

[Dirk Pranke, 2010-03-29 21:17:00]

So, to be clear, after this change, we'll be assuming that Python is in the
users's path for both shells and command prompts?

Assuming we're good with that (and I think we are), LGTM.

[Victor Wang, 2010-03-29 21:28:08]

Yes, I think that's the assumption. We already have python in depot_tools and my
understanding is we don't not want to include another python in
trunk\src\third_party.

Marc / Jeff,
Correct me if I am wrong.

On 2010/03/29 21:17:00, dpranke wrote:
> So, to be clear, after this change, we'll be assuming that Python is in the
> users's path for both shells and command prompts?
> 
> Assuming we're good with that (and I think we are), LGTM.

[Evan Martin, 2010-03-30 16:57:26]

http://codereview.chromium.org/1568004/diff/7001/8003
File webkit/tools/layout_tests/run_webkit_tests.sh (right):

http://codereview.chromium.org/1568004/diff/7001/8003#newcode13
webkit/tools/layout_tests/run_webkit_tests.sh:13: SCRIPT=$(cygpath -wa
"$exec_dir/run_webkit_tests.py")
Since you just fail inside of the .py anyway, I wonder if all of this PATH
munging is necessary.

[Victor Wang, 2010-03-30 17:25:58]

http://codereview.chromium.org/1568004/diff/7001/8003
File webkit/tools/layout_tests/run_webkit_tests.sh (right):

http://codereview.chromium.org/1568004/diff/7001/8003#newcode13
webkit/tools/layout_tests/run_webkit_tests.sh:13: SCRIPT=$(cygpath -wa
"$exec_dir/run_webkit_tests.py")
The script won't fail in py if user has user path set correctly
(depot_tools\python in user path, that's a windows version). But yes, the PATH
stuff should be removed so Python for windows can be used.

On 2010/03/30 16:57:26, Evan Martin wrote:
> Since you just fail inside of the .py anyway, I wonder if all of this PATH
> munging is necessary.