diff mbox

[U-Boot,3/6] moveconfig: Tidy up the documentation and add hints

Message ID 20170515114736.17521-4-sjg@chromium.org
State Superseded
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass May 15, 2017, 11:47 a.m. UTC
The newest clean-up features are not mentioned in the docs. Fix this and
add a few hints for particular workflows that are hopefully helpful.

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

 tools/moveconfig.py | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

Comments

Tom Rini May 15, 2017, 12:26 p.m. UTC | #1
On Mon, May 15, 2017 at 05:47:33AM -0600, Simon Glass wrote:
> The newest clean-up features are not mentioned in the docs. Fix this and
> add a few hints for particular workflows that are hopefully helpful.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
> 
>  tools/moveconfig.py | 18 ++++++++++++++++--
>  1 file changed, 16 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/moveconfig.py b/tools/moveconfig.py
> index 7a302d9404..059abb8858 100755
> --- a/tools/moveconfig.py
> +++ b/tools/moveconfig.py
> @@ -115,6 +115,19 @@ use your own.  Instead of modifying the list directly, you can give
>  them via environments.
>  
>  
> +Tips and trips
> +--------------
> +
> +To sync only X86 defconfigs:
> +
> +   grep -l X86 configs/* | ./tools/moveconfig.py -s -d -

To concrete my last comment:
./tools/moveconfig.py -s -d <(grep -l X86 configs/*)
Masahiro Yamada May 19, 2017, 5:16 p.m. UTC | #2
2017-05-15 20:47 GMT+09:00 Simon Glass <sjg@chromium.org>:

> @@ -169,7 +182,8 @@ Available options
>
>   -y, --yes
>     Instead of prompting, automatically go ahead with all operations. This
> -   includes cleaning up headers and CONFIG_SYS_EXTRA_OPTIONS.
> +   includes cleaning up header, CONFIG_SYS_EXTRA_OPTIONS, the config whitelist
> +   and the README.
>


I think   "headers" is OK as-is.  (plural)
(Usually, two or more headers are cleaned up)
diff mbox

Patch

diff --git a/tools/moveconfig.py b/tools/moveconfig.py
index 7a302d9404..059abb8858 100755
--- a/tools/moveconfig.py
+++ b/tools/moveconfig.py
@@ -115,6 +115,19 @@  use your own.  Instead of modifying the list directly, you can give
 them via environments.
 
 
+Tips and trips
+--------------
+
+To sync only X86 defconfigs:
+
+   grep -l X86 configs/* | ./tools/moveconfig.py -s -d -
+
+To process CONFIG_CMD_FPGAD only for a subset of configs based on path match:
+
+   ls configs/{hrcon*,iocon*,strider*} | \
+       ./tools/moveconfig.py -Cy CONFIG_CMD_FPGAD -d -
+
+
 Available options
 -----------------
 
@@ -128,7 +141,7 @@  Available options
 
  -d, --defconfigs
   Specify a file containing a list of defconfigs to move.  The defconfig
-  files can be given with shell-style wildcards.
+  files can be given with shell-style wildcards. Use '-' to read from stdin.
 
  -n, --dry-run
    Perform a trial run that does not make any changes.  It is useful to
@@ -169,7 +182,8 @@  Available options
 
  -y, --yes
    Instead of prompting, automatically go ahead with all operations. This
-   includes cleaning up headers and CONFIG_SYS_EXTRA_OPTIONS.
+   includes cleaning up header, CONFIG_SYS_EXTRA_OPTIONS, the config whitelist
+   and the README.
 
 To see the complete list of supported options, run