Re: [U-Boot] [PATCH v3 06/18] buildman: Move full help code into the control module

2014-09-10 Thread Simon Glass
Applied to u-boot-x86/buildman.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v3 06/18] buildman: Move full help code into the control module

2014-09-05 Thread Simon Glass
There is no good reason to keep this code separate. Move it into control.py
so it is easier to test.

Signed-off-by: Simon Glass s...@chromium.org
---

Changes in v3: None
Changes in v2: None

 tools/buildman/buildman.py | 6 --
 tools/buildman/control.py  | 8 
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/tools/buildman/buildman.py b/tools/buildman/buildman.py
index c4de857..70c2142 100755
--- a/tools/buildman/buildman.py
+++ b/tools/buildman/buildman.py
@@ -63,12 +63,6 @@ options, args = cmdline.ParseArgs()
 # Run our meagre tests
 if options.test:
 RunTests()
-elif options.full_help:
-pager = os.getenv('PAGER')
-if not pager:
-pager = 'more'
-fname = os.path.join(os.path.dirname(sys.argv[0]), 'README')
-command.Run(pager, fname)
 
 # Build selected commits for selected boards
 else:
diff --git a/tools/buildman/control.py b/tools/buildman/control.py
index 06c9229..408d9b1 100644
--- a/tools/buildman/control.py
+++ b/tools/buildman/control.py
@@ -84,6 +84,14 @@ def DoBuildman(options, args):
 options: Command line options object
 args: Command line arguments (list of strings)
 
+if options.full_help:
+pager = os.getenv('PAGER')
+if not pager:
+pager = 'more'
+fname = os.path.join(os.path.dirname(sys.argv[0]), 'README')
+command.Run(pager, fname)
+return 0
+
 gitutil.Setup()
 
 bsettings.Setup(options.config_file)
-- 
2.1.0.rc2.206.gedb03e5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot