[opensource-dev] Review Request: OPEN-45: Correctly include the pass-through options when displaying the configure and build commands

Oz Linden oz at lindenlab.com
Wed Mar 16 20:19:50 PDT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/213/
-----------------------------------------------------------

Review request for Viewer and Alain Linden.


Summary
-------

Extended the __call__ method of Executable to pass in the command type as a string and the dry_run flag so that the logging of the command being run could use the infrastructure there to exactly assemble the options and arguments for the logging of the command to be run.  The callers then don't need to do the logging separately or do anything with dry_run other than pass it down to the executable.


This addresses bug open-45.
    http://jira.secondlife.com/browse/open-45


Diffs
-----

  autobuild/autobuild_tool_build.py abc1014d5ad6 
  autobuild/autobuild_tool_configure.py abc1014d5ad6 
  autobuild/executable.py abc1014d5ad6 

Diff: http://codereview.secondlife.com/r/213/diff


Testing
-------


Thanks,

Oz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110317/f6e9ace8/attachment-0001.htm 


More information about the opensource-dev mailing list