On 08/31/2011 09:20 PM, Dale Curtis wrote: > Includes the following changes: > > http://gerrit.chromium.org/gerrit/#change,7041 > http://gerrit.chromium.org/gerrit/#change,7047 > > The packaging fix yielded a 15% speed up in our testing. Depending on > the number of packages in a given test this could be higher or lower, > but always better :)
Awesome, applied: http://autotest.kernel.org/changeset/5540 Thank you very much! > > Signed-off-by: Dale Curtis<[email protected]> > --- > client/common_lib/base_packages.py | 7 ++++++- > 1 files changed, 6 insertions(+), 1 deletions(-) > > diff --git a/client/common_lib/base_packages.py > b/client/common_lib/base_packages.py > index cf71949..3d2acb6 100644 > --- a/client/common_lib/base_packages.py > +++ b/client/common_lib/base_packages.py > @@ -67,7 +67,9 @@ def repo_run_command(repo, cmd, ignore_status=False, > cd=True): > > > def create_directory(repo): > - _, remote_path = parse_ssh_path(repo) > + remote_path = repo > + if repo.startswith('ssh://'): > + _, remote_path = parse_ssh_path(repo) > repo_run_command(repo, 'mkdir -p %s' % remote_path, cd=False) > > > @@ -449,6 +451,9 @@ class BasePackageManager(object): > or not self.compare_checksum(dest_path, > fetcher.url)) > if need_to_fetch: > fetcher.fetch_pkg_file(pkg_name, dest_path) > + # update checksum so we won't refetch next time. > + if use_checksum: > + self.update_checksum(dest_path) > return > except (error.PackageFetchError, error.AutoservRunError): > # The package could not be found in this repo, continue > looking _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
