Bug 168737 - AutoInstaller should not skip the install step if the package is downloaded but not installed
Summary: AutoInstaller should not skip the install step if the package is downloaded b...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Carlos Alberto Lopez Perez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-22 12:43 PST by Carlos Alberto Lopez Perez
Modified: 2017-02-22 13:56 PST (History)
8 users (show)

See Also:


Attachments
Patch (2.13 KB, patch)
2017-02-22 12:50 PST, Carlos Alberto Lopez Perez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2017-02-22 12:43:05 PST
When installing a python library for the tests via AutoInstaller, there is an optional argument "should_refresh" that can be defined to instruct the installer to always re-download and re-install such package.


The issue is that "should_refresh" only checks if the source tarball is downloaded, but it ignores the fact that it can be already installed or not

This can happen if the user manually cleans the installation folder for the auto-installed python libraries or if the installation of the library failed previously for any reason.

So... what ends happening is that the module never gets installed.

This has bitten me, I was cleaning the automatically installed wpt-tools library at LayoutTests/imported/w3c/web-platform-tests/tools, but wiping the tools sub-directory never caused this library to be installed again.

The check to decide if the installation step should be skipped, it was only checking for the existence of the file defined at LayoutTests/imported/w3c/web-platform-tests/.tools.url rather than checking also for the target directory to exist.
Comment 1 Carlos Alberto Lopez Perez 2017-02-22 12:50:31 PST
Created attachment 302426 [details]
Patch
Comment 2 Ryosuke Niwa 2017-02-22 13:28:30 PST
Comment on attachment 302426 [details]
Patch

Looks sane.
Comment 3 WebKit Commit Bot 2017-02-22 13:56:50 PST
Comment on attachment 302426 [details]
Patch

Clearing flags on attachment: 302426

Committed r212852: <http://trac.webkit.org/changeset/212852>
Comment 4 WebKit Commit Bot 2017-02-22 13:56:55 PST
All reviewed patches have been landed.  Closing bug.