All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 05/13] buildman: Sort command line options
Date: Fri,  8 Aug 2014 05:10:22 -0600	[thread overview]
Message-ID: <1407496230-12511-6-git-send-email-sjg@chromium.org> (raw)
In-Reply-To: <1407496230-12511-1-git-send-email-sjg@chromium.org>

These options have got slightly out of order. Fix them.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

Changes in v5: None
Changes in v4:
- Fix typo in commit message

Changes in v3:
- Add new patch to sort command line options

Changes in v2: None

 tools/buildman/buildman.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/tools/buildman/buildman.py b/tools/buildman/buildman.py
index 42847ac..da6025a 100755
--- a/tools/buildman/buildman.py
+++ b/tools/buildman/buildman.py
@@ -70,6 +70,9 @@ parser.add_option('-c', '--count', dest='count', type='int',
 parser.add_option('-C', '--force-reconfig', dest='force_reconfig',
        action='store_true', default=False,
        help='Reconfigure for every commit (disable incremental build)')
+parser.add_option('-d', '--detail', dest='show_detail',
+       action='store_true', default=False,
+       help='Show detailed information for each board in summary')
 parser.add_option('-e', '--show_errors', action='store_true',
        default=False, help='Show errors and warnings')
 parser.add_option('-f', '--force-build', dest='force_build',
@@ -78,9 +81,6 @@ parser.add_option('-f', '--force-build', dest='force_build',
 parser.add_option('-F', '--force-build-failures', dest='force_build_failures',
        action='store_true', default=False,
        help='Force build of previously-failed build')
-parser.add_option('-d', '--detail', dest='show_detail',
-       action='store_true', default=False,
-       help='Show detailed information for each board in summary')
 parser.add_option('-g', '--git', type='string',
        help='Git repo containing branch to build', default='.')
 parser.add_option('-H', '--full-help', action='store_true', dest='full_help',
@@ -96,6 +96,9 @@ parser.add_option('--list-tool-chains', action='store_true', default=False,
        help='List available tool chains')
 parser.add_option('-n', '--dry-run', action='store_true', dest='dry_run',
        default=False, help="Do a try run (describe actions, but no nothing)")
+parser.add_option('-o', '--output-dir', type='string',
+       dest='output_dir', default='..',
+       help='Directory where all builds happen and buildman has its workspace (default is ../)')
 parser.add_option('-Q', '--quick', action='store_true',
        default=False, help='Do a rough build, with limited warning resolution')
 parser.add_option('-s', '--summary', action='store_true',
@@ -110,9 +113,6 @@ parser.add_option('-T', '--threads', type='int',
        default=None, help='Number of builder threads to use')
 parser.add_option('-u', '--show_unknown', action='store_true',
        default=False, help='Show boards with unknown build result')
-parser.add_option('-o', '--output-dir', type='string',
-       dest='output_dir', default='..',
-       help='Directory where all builds happen and buildman has its workspace (default is ../)')
 
 parser.usage = """buildman -b <branch> [options]
 
-- 
2.0.0.526.g5318336

  parent reply	other threads:[~2014-08-08 11:10 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-08 11:10 [U-Boot] [PATCH v5 0/13] Add some missing buildman features and deprecate MAKEALL Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 01/13] buildman: Fix a few typos Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 02/13] buildman: Add some notes about moving from MAKEALL Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 03/13] buildman: Allow building of current source tree Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 04/13] buildman: Move BuilderThread code to its own file Simon Glass
2014-08-08 11:10 ` Simon Glass [this message]
2014-08-08 11:10 ` [U-Boot] [PATCH v5 06/13] buildman: Refactor output options Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 07/13] buildman: Add verbose option to display errors as they happen Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 08/13] buildman: Remove unused non-incremental build method code Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 09/13] buildman: Add an option to specify the buildman config file Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 10/13] buildman: Add a message indicating there are no errors Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 11/13] buildman: Add a few more toolchain examples to the README Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 12/13] buildman: Introduce an 'and' operator for board selection Simon Glass
2014-08-08 17:46   ` York Sun
2014-08-09  4:35     ` Simon Glass
2014-08-08 11:10 ` [U-Boot] [PATCH v5 13/13] RFC: Deprecate MAKEALL Simon Glass
2014-08-08 19:06 ` [U-Boot] [PATCH v5 0/13] Add some missing buildman features and deprecate MAKEALL Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1407496230-12511-6-git-send-email-sjg@chromium.org \
    --to=sjg@chromium.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.