DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] devtools: fix incorrect shell assignment
@ 2019-03-04  9:57 David Marchand
  2019-03-04 10:12 ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: David Marchand @ 2019-03-04  9:57 UTC (permalink / raw)
  To: dev; +Cc: thomas, stable

No space needed here:
./devtools/checkpatches.sh: line 65: res: command not found

Fixes: dafc04c15174 ("devtools: fix return of forbidden addition checks")
Cc: stable@dpdk.org
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 devtools/checkpatches.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/devtools/checkpatches.sh b/devtools/checkpatches.sh
index 3b03b7e..02d1c30 100755
--- a/devtools/checkpatches.sh
+++ b/devtools/checkpatches.sh
@@ -62,7 +62,7 @@ check_forbidden_additions() { # <patch>
 		-v RET_ON_FAIL=1 \
 		-v MESSAGE='Using explicit .svg extension instead of .*' \
 		-f $(dirname $(readlink -e $0))/check-forbidden-tokens.awk \
-		"$1" || res = 1
+		"$1" || res=1
 
 	return $res
 }
-- 
1.8.3.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] devtools: fix incorrect shell assignment
  2019-03-04  9:57 [dpdk-dev] [PATCH] devtools: fix incorrect shell assignment David Marchand
@ 2019-03-04 10:12 ` Thomas Monjalon
  2019-03-04 10:17   ` David Marchand
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2019-03-04 10:12 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, stable

04/03/2019 10:57, David Marchand:
> No space needed here:
> ./devtools/checkpatches.sh: line 65: res: command not found
> 
> Fixes: dafc04c15174 ("devtools: fix return of forbidden addition checks")
> Cc: stable@dpdk.org
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Thank you
It is already submitted:
	https://patches.dpdk.org/patch/50727/
It is part of a patchset, waiting for a new version (for codespell feature).

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] devtools: fix incorrect shell assignment
  2019-03-04 10:12 ` Thomas Monjalon
@ 2019-03-04 10:17   ` David Marchand
  0 siblings, 0 replies; 3+ messages in thread
From: David Marchand @ 2019-03-04 10:17 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, dpdk stable

On Mon, Mar 4, 2019 at 11:13 AM Thomas Monjalon <thomas@monjalon.net> wrote:

> 04/03/2019 10:57, David Marchand:
> > No space needed here:
> > ./devtools/checkpatches.sh: line 65: res: command not found
> >
> > Fixes: dafc04c15174 ("devtools: fix return of forbidden addition checks")
> > Cc: stable@dpdk.org
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
>
> Thank you
> It is already submitted:
>         https://patches.dpdk.org/patch/50727/
> It is part of a patchset, waiting for a new version (for codespell
> feature).
>

Indeed, thanks.


-- 
David Marchand

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-03-04 10:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-04  9:57 [dpdk-dev] [PATCH] devtools: fix incorrect shell assignment David Marchand
2019-03-04 10:12 ` Thomas Monjalon
2019-03-04 10:17   ` David Marchand

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).