----------------------------------------------------------- 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
_______________________________________________ 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