On Thu, 2012-08-02 at 11:07 +0800, Mike Qiu wrote: > Currently,in test case dacapo,it put the cache dir,which used to put > the download files in it, down to self.bindir. > Now Chang the cache dir to parent dir of self.srcdir,that is to > say by default is client/tmp/dacapo/cache
Looks good to me, applied. Just remember to put a more descriptive summary line: client.tests.dacapo: Move cache dir to parent dir of self.srcdir I've fixed this commit message, but next patches, please pay attention to it. > Signed-off-by: Mike Qiu <[email protected]> > --- > client/tests/dacapo/dacapo.py | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/client/tests/dacapo/dacapo.py b/client/tests/dacapo/dacapo.py > index 2348fca..a744911 100644 > --- a/client/tests/dacapo/dacapo.py > +++ b/client/tests/dacapo/dacapo.py > @@ -43,7 +43,7 @@ class dacapo(test.test): > def run_once(self, test='antlr', config='./dacapo.cfg', jvm='default'): > cfg = config_loader(cfg=config, tmpdir=self.tmpdir, > raise_errors=True) > self.test = test > - cachedir = os.path.join(self.bindir, 'cache') > + cachedir = os.path.join(os.path.dirname(self.srcdir), 'cache') > if not os.path.isdir(cachedir): > os.makedirs(cachedir) > _______________________________________________ Autotest-kernel mailing list [email protected] https://www.redhat.com/mailman/listinfo/autotest-kernel
