All submissions to this site are governed by the Second Life Viewer Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.

Review Board 1.6.11

Welcome to the Second Life Viewer Code Review tool.
See the documentation on our wiki for how to use this site.

OPEN-8: allow specification of the configuration with env var AUTOBUILD_CONFIGURATION

Review Request #141 - Created Feb. 6, 2011 and submitted

Oz Linden Reviewers
viewer
open-8
None autobuild 
This patch adds support for specifying the configurations using the AUTOBUILD_CONFIGURATION environment variable (multiple configurations can be specified using commas to separate the values).
Tested manually with and without the command line argument, and with and without the environment variable set; the appropriate configuration values were used.
Alain Linden
Posted (Feb. 8, 2011, 9:37 a.m.)
I'm not sure AUTOBUILD_CONFIGURATION is a good name for this environment variable.  I know this is long, but AUTOUBUILD_DEFAULT_BUILD_CONFIGURATIONS is more accurate and descriptive (and one could probably drop the AUTOBUILD if we aren't too worried about conflicts with other env variable names)
  1. The name was chosen to match that of the long-form option that it provides the default value for.  Using the AUTOBUILD_ prefix is just good practice, because collisions are a serious issue not to be ignored.

  2. I note that 'default' is generally the name of one of the configurations in the config file (i.e. autobuild.xml). Given that and oz's pattern of naming environment variables after the option names, I'd go with the AUTOBUILD_CONFIGURATION nomenclature.

I'd want to be a bit careful about populating the env var space much further than this, but what you've done here so far seems fine to me.
Ship it!
Jenn
Posted (Feb. 11, 2011, 5:52 p.m.)