[opensource-dev] Review Request: Use consistent path for all *.py scripts

[Oz Linden]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/#review353
-----------------------------------------------------------

Ship it!


Those changes look good.

For future reference, there's no reason to add date ranges in Copyright headers - the only year needed is the year of first publication.  Adding additional years is not needed and serves no purpose.


- Oz


On Feb. 9, 2011, 3:33 p.m., Merov Linden wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/129/
> -----------------------------------------------------------
> 
> (Updated Feb. 9, 2011, 3:33 p.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> Simple consistency change, using "#!/usr/bin/python" in all python script.
> 
> 
> This addresses bug STORM-937.
>     http://jira.secondlife.com/browse/STORM-937
> 
> 
> Diffs
> -----
> 
>   indra/cmake/run_build_test.py 0ca239e469b3 
>   indra/copy_win_scripts/start-client.py 0ca239e469b3 
>   indra/develop.py 0ca239e469b3 
>   indra/lib/python/indra/util/llperformance.py 0ca239e469b3 
>   indra/lib/python/indra/util/simperf_proc_interface.py 0ca239e469b3 
>   indra/lib/python/indra/util/test_win32_manifest.py 0ca239e469b3 
>   indra/llmessage/tests/test_llsdmessage_peer.py 0ca239e469b3 
>   indra/llmessage/tests/testrunner.py 0ca239e469b3 
>   indra/newview/generate_breakpad_symbols.py 0ca239e469b3 
>   indra/newview/tests/test_llxmlrpc_peer.py 0ca239e469b3 
>   indra/newview/viewer_manifest.py 0ca239e469b3 
>   indra/test/test_llmanifest.py 0ca239e469b3 
>   scripts/build_version.py 0ca239e469b3 
>   scripts/md5check.py 0ca239e469b3 
>   scripts/setup-path.py 0ca239e469b3 
>   scripts/template_verifier.py 0ca239e469b3 
>   scripts/update_version_files.py 0ca239e469b3 
> 
> Diff: http://codereview.secondlife.com/r/129/diff
> 
> 
> Testing
> -------
> 
> Pulled into a test repo and build successfully on all platforms on TC so I guess no bad surprise here.
> 
> 
> Thanks,
> 
> Merov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110211/91339b48/attachment.htm