commit 423c27245fbd2a3d44468faaad8fdfc728817a48
author Vic Yang <victoryang@chromium.org> Wed Sep 24 16:05:06 2014 +0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Oct 01 19:46:30 2014 +0000
tree e4468c367a6f32d843ca9abd65bd9c1567164b03
parent 7d693c497e970f9cd5cce2ca5bdec3fbb1029c0c

Enable both presenter and controller by default

Most of the time, we want to run factory toolkit in standalone mode.
Let's set the default this way, and we can always use
--no-enable-presenter and --no-eanble-device when needed.

BUG=chrome-os-partner:32256
TEST=make smartlint

Change-Id: I15c3baee648b33a70b7dc6d27432bf7b747f5303
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/219673
Reviewed-by: Ricky Liang <jcliang@chromium.org>

py/toolkit/installer.py

diff --git a/py/toolkit/installer.py b/py/toolkit/installer.py
index 83ef29a..53980f7 100755
--- a/py/toolkit/installer.py
+++ b/py/toolkit/installer.py
@@ -351,20 +351,21 @@
       help="Run goofy in presenter mode on startup")
   parser.add_argument('--no-enable-presenter', dest='enable_presenter',
       action='store_false', help=argparse.SUPPRESS)
-  parser.set_defaults(enable_presenter=False)
+  parser.set_defaults(enable_presenter=True)
 
   parser.add_argument('--enable-device', dest='enable_device',
       action='store_true',
       help="Run goofy in device mode on startup")
   parser.add_argument('--no-enable-device', dest='enable_device',
       action='store_false', help=argparse.SUPPRESS)
+  parser.set_defaults(enable_device=True)
+
   parser.add_argument('--init-umpire-board', dest='umpire_board',
       nargs='?', default=None,
       help="Locally install Umpire server for specific board")
   parser.add_argument('--exe-path', dest='exe_path',
       nargs='?', default=None,
       help="Current self-extracting archive pathname")
-  parser.set_defaults(enable_device=True)
 
   args = parser.parse_args()