public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH] Fix a wrong (reversed) success condition in echo01
@ 2009-12-01 22:16 Jiri Palecek
  0 siblings, 0 replies; 2+ messages in thread
From: Jiri Palecek @ 2009-12-01 22:16 UTC (permalink / raw)
  Cc: ltp-list

Hello,

this is the first of another series of small ltp fixes. I'm sorry I wasn't able to send them earlier.

This patch reverses the success check in echo01, obviously the checksums being equal is the right outcome.

Regards
    Jiri Palecek

Signed-off-by: Jiri Palecek <jpalecek@web.de>
---
 testcases/network/tcp_cmds/echo/echo01 |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/testcases/network/tcp_cmds/echo/echo01 b/testcases/network/tcp_cmds/echo/echo01
index eb798ae..2bc4621 100755
--- a/testcases/network/tcp_cmds/echo/echo01
+++ b/testcases/network/tcp_cmds/echo/echo01
@@ -95,7 +95,7 @@ do_test()
         while [ $NUM -lt $NUMPROCESSES ]
         do
             csum2=$(sum "$TCtmp/${ECHOFILE}${NUM}" | awk '{print $1}')
-            if [ "$csum1" = "$csum2" ]; then
+            if ! [ "$csum1" = "$csum2" ]; then
                 end_testcase "Checksums differ in loop $TST_COUNT"
             fi
             NUM=$(( $NUM + 1 ))
-- 
1.6.4.3




------------------------------------------------------------------------------
Join us December 9, 2009 for the Red Hat Virtual Experience,
a free event focused on virtualization and cloud computing. 
Attend in-depth sessions from your desk. Your couch. Anywhere.
http://p.sf.net/sfu/redhat-sfdev2dev
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

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

* Re: [LTP] [PATCH] Fix a wrong (reversed) success condition in echo01
       [not found] <4b159725.8d13f30a.7a77.1f9bSMTPIN_ADDED@mx.google.com>
@ 2009-12-01 23:15 ` Mike Frysinger
  0 siblings, 0 replies; 2+ messages in thread
From: Mike Frysinger @ 2009-12-01 23:15 UTC (permalink / raw)
  To: ltp-list; +Cc: Jiri Palecek


[-- Attachment #1.1: Type: Text/Plain, Size: 208 bytes --]

On Tuesday 01 December 2009 17:16:45 Jiri Palecek wrote:
> -            if [ "$csum1" = "$csum2" ]; then
> +            if ! [ "$csum1" = "$csum2" ]; then

please use [ x != z ] rather than ! [ x = z ]
-mike

[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 300 bytes --]

------------------------------------------------------------------------------
Join us December 9, 2009 for the Red Hat Virtual Experience,
a free event focused on virtualization and cloud computing. 
Attend in-depth sessions from your desk. Your couch. Anywhere.
http://p.sf.net/sfu/redhat-sfdev2dev

[-- Attachment #3: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

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

end of thread, other threads:[~2009-12-01 23:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <4b159725.8d13f30a.7a77.1f9bSMTPIN_ADDED@mx.google.com>
2009-12-01 23:15 ` [LTP] [PATCH] Fix a wrong (reversed) success condition in echo01 Mike Frysinger
2009-12-01 22:16 Jiri Palecek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox