Rename "taskgraph" -> "gecko_taskgraph"
Categories
(Firefox Build System :: Task Configuration, task)
Tracking
(firefox95 fixed)
| Tracking | Status | |
|---|---|---|
| firefox95 | --- | fixed |
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(2 files, 1 obsolete file)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
471.70 KB,
patch
|
Details | Diff | Splinter Review |
We currently have two taskgraph modules, one in mozilla-central and one standalone one that is used by everything outside of m-c.
Keeping these code bases in sync is a huge burden, so the plan is for the m-c version to start depending on the standalone one (and become just another project that consumes it alongside mobile repos and the like).
This means there will be a module name collision as they are currently both called taskgraph. To avoid confusion, let's rename the mozilla-central version to gecko_taskgraph. This follows the mobile convention of naming the project specific logic <project>_taskgraph (and also the convention of using gecko as the project label in things like the trust domain).
| Assignee | ||
Comment 1•4 years ago
|
||
For a long time two copies of the 'taskgraph' module have existed in parallel.
We've attempted to keep them in sync, but over time they have diverged and the
maintenance burden has increased.
In order to reduce this burden, we'd like to re-join the two code bases. The
canonical repo will be the one that lives outside of mozilla-central, and this
module will depend on it. Since they both have the same module name (taskgraph)
we need to rename the version in mozilla-central to avoid collisions.
Other consumers of 'taskgraph' (like mobile repos) have standardized on
'<project>_taskgraph' as their module names. So replicating that here as well.
| Assignee | ||
Comment 2•4 years ago
|
||
Splitting this out to avoid polluting prior diff.
Depends on D127118
Updated•4 years ago
|
| Assignee | ||
Comment 3•4 years ago
|
||
Phabricator is having issues with this diff for some reason. Will ask sheriffs to land this diff manually.
Comment 5•4 years ago
|
||
| bugherder | ||
Description
•