On 11/21/2012 11:01 PM, [email protected] wrote: > Hi! >> Currently, -s option only can run the test case in alltests by >> the PATTERN, but I think it's worth to be able to run all of >> the test cases under runtest/ by the PATTERN. >> For example, we can run test case like "cpuhotplug01" using >> ./runltp -s cpuhotplug01 >> with this patch. >> >> Signed-off-by: Wanlong Gao <[email protected]> >> --- >> runltp | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/runltp b/runltp >> index 04cc690..7d54f0d 100755 >> --- a/runltp >> +++ b/runltp >> @@ -651,7 +651,7 @@ main() >> if [ -n "$TAG_RESTRICT_STRING" ] >> then >> mv -f ${TMP}/alltests ${TMP}/alltests.orig >> - grep $TAG_RESTRICT_STRING ${TMP}/alltests.orig > ${TMP}/alltests >> #Not worth checking return codes for this case >> + grep $TAG_RESTRICT_STRING ${LTPROOT}/runtest/* | awk -F':' '{print $2}' >> | sort -u > ${TMP}/alltests #Not worth checking return codes for this case >> fi >> >> # Blacklist or skip tests if a SKIPFILE was specified with -S > > The runltp script is one of the places I'm still not familiar with, but > I'm rather woried of the 'sort -u'. This would change the order and > count of LTP cases executed and that is something that should be > discussed first.
Yeah, that's the thing, do you have any good idea about removing the repeated lines here? Thanks, Wanlong Gao > ------------------------------------------------------------------------------ Monitor your physical, virtual and cloud infrastructure from a single web console. Get in-depth insight into apps, servers, databases, vmware, SAP, cloud infrastructure, etc. Download 30-day Free Trial. Pricing starts from $795 for 25 servers or applications! http://p.sf.net/sfu/zoho_dev2dev_nov _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
