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

Merov Linden merov at lindenlab.com
Wed Feb 9 15:33:47 PST 2011


-----------------------------------------------------------
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.


Changes
-------

Took comments into account:
- Changed to use the #!/usr/bin/env python shebang line
- Add license info to python files that were missing one
- Reformatted some license header for consistency
- Updated wiki http://wiki.secondlife.com/wiki/Coding_standard#Python to make those aspects (shebang and licensing) clear


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 (updated)
-----

  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/20110209/706a3ade/attachment.htm 


More information about the opensource-dev mailing list