public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: chrubis@suse.cz
To: Simon Xu <xu.simon@oracle.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH V4] rpc01: fix paths and add Makefile for datafiles
Date: Thu, 28 Nov 2013 11:20:56 +0100	[thread overview]
Message-ID: <20131128102056.GA19263@rei.Home> (raw)
In-Reply-To: <52971586.5090009@oracle.com>

Hi!
> > Ah, I guess there are at least two more places where tests install their
> > data files. I've looked around and found a few in testcases/bin/ :(
> >
> >> I think we better change all of them or do nothing.
> > Right. I want to have one single place to store test datafiles and
> > ideally to be easy to figure out which files belogs to which testcases.
> > Once this is defined we can fix all the testcases as well.
> >
> > What about this:
> >
> > 1. Testcases datafiles are stored in a TESTNAME_data directory
> >     where TESTNAME is name of the test or a group of test (if
> >     the files are used by more than one test)
> >
> > 2. The whole directory gets installed somewhere in the
> >     /opt/ltp/testcases/, be it testcases/bin/datafiles or
> >     testcases/datafiles.
> >
> > What do you think?
> 
> What about testcases/data/TESTNAME?

OK with me.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT 
organizations don't have a clear picture of how application performance 
affects their revenue. With AppDynamics, you get 100% visibility into your 
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  parent reply	other threads:[~2013-11-28 10:21 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-06  2:25 [LTP] [PATCH] rpc01: fix $TCbin and $TCtmp Simon Xu
2013-11-06  5:28 ` [LTP] [PATCH V2] rpc01: fix $TCbin and $TCtmp and data file names Simon Xu
2013-11-07 16:25   ` chrubis
     [not found]     ` <527C45FD.3090008@oracle.com>
2013-11-11 14:15       ` chrubis
2013-11-19  6:14         ` [LTP] [PATCH V4] rpc01: fix paths and add Makefile for datafiles Simon Xu
2013-11-25 13:39           ` Simon Xu
2013-11-26 14:49           ` chrubis
     [not found]             ` <52955FA1.7020909@oracle.com>
2013-11-27 11:49               ` chrubis
     [not found]                 ` <52971586.5090009@oracle.com>
2013-11-28 10:10                   ` [LTP] [PATCH V5] " Simon Xu
2013-12-02 17:59                     ` chrubis
2013-11-28 10:20                   ` chrubis [this message]
2013-11-08  6:17   ` [LTP] [PATCH V3] rpc01: fix $TCbin and $TCtmp " Simon Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20131128102056.GA19263@rei.Home \
    --to=chrubis@suse.cz \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=xu.simon@oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox