From a1cde743e755d826c86aeb006d2027ad2e5c8ce3 Mon Sep 17 00:00:00 2001 From: Josh Bell Date: Thu, 13 Mar 2008 22:26:47 +0000 Subject: Duh... I didn't test with default arguments, apparently. * SConstruct doesn't like None as a default value; "False" will do nicely, since later logic tests for truth * Was using [] instead of get() for a value that might not be present (and wanted None in that case) --- indra/newview/viewer_manifest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'indra') diff --git a/indra/newview/viewer_manifest.py b/indra/newview/viewer_manifest.py index a030a64401..b19e4e2c9c 100755 --- a/indra/newview/viewer_manifest.py +++ b/indra/newview/viewer_manifest.py @@ -97,7 +97,7 @@ class ViewerManifest(LLManifest): """Channel reported for login and upgrade purposes ONLY; used for A/B testing""" # NOTE: Do not return the normal channel if login_channel is not specified, as # some code may branch depending on whether or not this is present - return self.args['login_channel'] + return self.args.get('login_channel') def channel(self): return self.args['channel'] -- cgit v1.2.3