Message ID | 1373794066-7349-1-git-send-email-gsi@denx.de |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
On Sun, Jul 14, 2013 at 2:27 AM, Gerhard Sittig <gsi@denx.de> wrote: > adjust instructions for the invocation of Patman's self test: the -t > flag appears to have a different meaning now, refer to the --test option > for the builtin unit test; adjust a directory location and make sure to > run the file which resides in the source directory > > Signed-off-by: Gerhard Sittig <gsi@denx.de> > Acked-by: Simon Glass <sjg@chromium.org> Tom, would you like to pick this fix up now, or should I do it in the next merge window?
On Sun, Jul 14, 2013 at 10:51:08AM -0700, Simon Glass wrote: > On Sun, Jul 14, 2013 at 2:27 AM, Gerhard Sittig <gsi@denx.de> wrote: > > > adjust instructions for the invocation of Patman's self test: the -t > > flag appears to have a different meaning now, refer to the --test option > > for the builtin unit test; adjust a directory location and make sure to > > run the file which resides in the source directory > > > > Signed-off-by: Gerhard Sittig <gsi@denx.de> > > > > Acked-by: Simon Glass <sjg@chromium.org> > > Tom, would you like to pick this fix up now, or should I do it in the next > merge window? I shall pick this up later today, thanks!
On Sun, Jul 14, 2013 at 11:27:45AM +0200, Gerhard Sittig wrote: > adjust instructions for the invocation of Patman's self test: the -t > flag appears to have a different meaning now, refer to the --test option > for the builtin unit test; adjust a directory location and make sure to > run the file which resides in the source directory > > Signed-off-by: Gerhard Sittig <gsi@denx.de> > Acked-by: Simon Glass <sjg@chromium.org> Applied to u-boot/master, thanks!
diff --git a/tools/patman/README b/tools/patman/README index 8cffcd1..c954989 100644 --- a/tools/patman/README +++ b/tools/patman/README @@ -452,12 +452,12 @@ Most of these are indicated by a TODO in the code. It would be nice if this could handle the In-reply-to side of things. -The tests are incomplete, as is customary. Use the -t flag to run them, -and make sure you are in the tools/scripts/patman directory first: +The tests are incomplete, as is customary. Use the --test flag to run them, +and make sure you are in the tools/patman directory first: $ cd /path/to/u-boot - $ cd tools/scripts/patman - $ patman -t + $ cd tools/patman + $ ./patman --test Error handling doesn't always produce friendly error messages - e.g. putting an incorrect tag in a commit may provide a confusing message.
adjust instructions for the invocation of Patman's self test: the -t flag appears to have a different meaning now, refer to the --test option for the builtin unit test; adjust a directory location and make sure to run the file which resides in the source directory Signed-off-by: Gerhard Sittig <gsi@denx.de> --- tools/patman/README | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) changes in v2: - fix another previously missed directory location - prefix script with './' since '.' is usually not in $PATH couldn't determine the appropriate subsystem/tree and corresponding custodian for this patch, who'd pick it up and thus is to Cc: to? Simon, can you take it?