> On Jan. 31, 2011, 6:27 a.m., Vadim ProductEngine wrote: > > Why didn't you choose "!/usr/bin/env python"? It seems more flexible.
!/usr/bin/env python, is the standard python way, and i think is what should always be used on python scripts. - Balp ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/129/#review284 ----------------------------------------------------------- On Jan. 28, 2011, 5:56 p.m., Merov Linden wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/129/ > ----------------------------------------------------------- > > (Updated Jan. 28, 2011, 5:56 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/copy_win_scripts/start-client.py b542f8134a2b > indra/develop.py b542f8134a2b > indra/lib/python/indra/util/simperf_host_xml_parser.py b542f8134a2b > indra/lib/python/indra/util/simperf_oprof_interface.py b542f8134a2b > indra/lib/python/indra/util/test_win32_manifest.py b542f8134a2b > indra/newview/generate_breakpad_symbols.py b542f8134a2b > scripts/build_version.py b542f8134a2b > scripts/install.py b542f8134a2b > > 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
