Open
Bug 1368168
Opened 7 years ago
Updated 2 years ago
`mach artifact toolchain` uncompresses existing downloaded toolchains unnecessarily
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(Not tracked)
NEW
People
(Reporter: froydnj, Unassigned)
References
Details
Ran into this when testing patches for bug 1364588. If you have downloaded a toolchain X with a `mach bootstrap` invocation, and the necessary toolchain hasn't changed, then re-invoking `mach bootstrap` will not re-download the toolchain (good) but will re-un-compress the downloaded toolchain to overwrite the existing one. I can see both good and bad sides to this, but this sort of behavior is decidedly less convenient than distribution package managers. If we're going to seriously consider downloading bits and pieces of the toolchains people need from taskcluster/tooltool, rather than using existing packages on their systems, we should make bootstrap do the minimum amount of work necessary here.
Comment 1•7 years ago
|
||
How do you know the extracted directory contains the right stuff?
Comment 2•7 years ago
|
||
We could save a manifest with hashes the first time we unpack and verify that. For me personally, it would be more helpful to have filenames under .mozbuild/toolchains so I can verify or remove bits manually without having to run things through tar.
Comment 3•7 years ago
|
||
The files don't have a useful filename because of tooltool cache compatibility (allows to reuse tooltool cache). The filenames will be more useful for files we take out of TC artifacts. That being said, the cache is also managed by the `mach artifact toolchain` command (it will remove old unused stuff), so you actually don't need to verify and remove bits manually.
Updated•7 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•