---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/#review558
---
The code has been integrated. Merov, please close the review reque
---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/
---
(Updated Feb. 11, 2011, 6:23 p.m.)
Review request for Viewer.
Changes
---
---
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
---
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
> On Jan. 31, 2011, 11:47 a.m., Kent Quirk wrote:
> > I would much prefer that we put the /usr/bin/env python form, since env can
> > find whichever python is currently set up, including someone that has it in
> > /usr/local/bin, or wherever they've chosen. Env is more likely to be found
> > i
> 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
-
> On Jan. 31, 2011, 11:47 a.m., Kent Quirk wrote:
> > I would much prefer that we put the /usr/bin/env python form, since env can
> > find whichever python is currently set up, including someone that has it in
> > /usr/local/bin, or wherever they've chosen. Env is more likely to be found
> > i
---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/#review286
---
I would much prefer that we put the /usr/bin/env python form, sinc
---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/#review284
---
Why didn't you choose "!/usr/bin/env python"? It seems more flexib
---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/#review276
---
Ship it!
Covers all occurrences of the variant with env, so shoul
---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/129/#review275
---
Ship it!
Looks like a good consistency improvement.
- Nicky
On
11 matches
Mail list logo