----------------------------------------------------------- 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 > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
