From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ltp-list-bounces@lists.sourceforge.net>
Received: from sog-mx-4.v43.ch3.sourceforge.com ([172.29.43.194]
	helo=mx.sourceforge.net)
	by sfs-ml-4.v29.ch3.sourceforge.com with esmtp (Exim 4.76)
	(envelope-from <metan@suse.de>) id 1UNNCr-00087c-BE
	for ltp-list@lists.sourceforge.net; Wed, 03 Apr 2013 12:54:25 +0000
Date: Wed, 3 Apr 2013 14:55:09 +0200
From: chrubis@suse.cz
Message-ID: <20130403125509.GD2002@rei>
References: <1364958179-5410-1-git-send-email-gaowanlong@cn.fujitsu.com>
	<20130403115218.GC2002@rei> <515C2398.4060906@cn.fujitsu.com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <515C2398.4060906@cn.fujitsu.com>
Subject: Re: [LTP] [PATCH] su01: fix the uncompleted string match
List-Id: Linux Test Project General Discussions
	<ltp-list.lists.sourceforge.net>
List-Unsubscribe: <https://lists.sourceforge.net/lists/listinfo/ltp-list>,
	<mailto:ltp-list-request@lists.sourceforge.net?subject=unsubscribe>
List-Archive: <http://sourceforge.net/mailarchive/forum.php?forum_name=ltp-list>
List-Post: <mailto:ltp-list@lists.sourceforge.net>
List-Help: <mailto:ltp-list-request@lists.sourceforge.net?subject=help>
List-Subscribe: <https://lists.sourceforge.net/lists/listinfo/ltp-list>,
	<mailto:ltp-list-request@lists.sourceforge.net?subject=subscribe>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: ltp-list-bounces@lists.sourceforge.net
To: Wanlong Gao <gaowanlong@cn.fujitsu.com>
Cc: Garrett Cooper <yanegomi@gmail.com>, LTP <ltp-list@lists.sourceforge.net>, Mike Frysinger <vapier@gentoo.org>

Hi!
> >> We find that sometimes "spawn /bin/su -l $USER1 -c whoami"
> >> may return two lines:
> >> "Attempting to create directory /home/su_usr2/perl5"
> >> "su_usr2"
> >> So the uncompleted "su_usr2" string match the first line,
> >> but we expect to match the second line here.
> >> So, fix it to match "su_usr2\r" instead.
> >>
> >> Reported-by: DAN LI <li.dan@cn.fujitsu.com>
> >> Signed-off-by: Wanlong Gao <gaowanlong@cn.fujitsu.com>
> > 
> > Looks good. What about making it even more strict with "^$USER\r"?
> 
> I'm afraid that "^" is not supported.

I've looked in the expect docs and the syntax for regular expressions
should be '-re "^$USER\r"' (beware not tested).

> > 
> >>      expect {
> >> -				"$USER1" { set i_am_correct 1 }
> >> +				"$USER1\r" { set i_am_correct 1 }
> >>      }
> >>    }
> >>  }
> > 
> 

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Minimize network downtime and maximize team effectiveness.
Reduce network management and security costs.Learn how to hire 
the most talented Cisco Certified professionals. Visit the 
Employer Resources Portal
http://www.cisco.com/web/learning/employer_resources/index.html
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list