Opened on 10/12/2015 at 03:29:01 PM
Closed on 10/20/2015 at 09:42:22 AM
#3194 closed change (fixed)
Support multiple sources for dependencies using a new syntax
Reported by: | kzar | Assignee: | kzar |
---|---|---|---|
Priority: | P4 | Milestone: | |
Module: | Automation | Keywords: | ensure_dependencies.py, dependencies |
Cc: | sebastian, trev | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by kzar)
Background
Our dependency management tool ensure_dependencies.py reads a project's
dependencies file and fetches all the dependencies listed therein. For each
dependency in the file a repository source is specified, which can either be an
absolute URL or a relative path to be appended to the _root repository URL.
When a relative path is specified the preferred version control system (VCS)
(the one used to clone the parent repository) is used where possible. When an
absolute URL is specified however the VCS used is tied to the URL.
As it's not possible to specify multiple sources for a dependency this means
that you are stuck with using relative paths if you want to support multiple
VCS tools. If you use an absolute URL for a dependency the other VCS can no
longer be used for it.
What to change
Allow the specification of multiple sources for a dependency. Do this by
supporting a new syntax for dependencies:
directory = [vcs:][<url>@]<rev> [vcs:][<url>@]<rev>
However, if the first source for a dependency does not contain the VCS part it
should be interpreted as being a URL without revision. This is so that we are
backwards compatible with the existing syntax:
directory = url [vcs:][<url>@]<rev>
(Of course there can be as many source parts per dependency as required.)
Attachments (0)
Change History (5)
comment:1 Changed on 10/12/2015 at 03:48:40 PM by kzar
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:3 Changed on 10/15/2015 at 03:34:11 PM by kzar
- Description modified (diff)
- Summary changed from Support multiple sources for dependencies to Support multiple sources for dependencies using a new syntax
comment:4 Changed on 10/19/2015 at 06:22:51 AM by sebastian
- Priority changed from Unknown to P4
- Ready set
comment:5 Changed on 10/20/2015 at 09:42:22 AM by kzar
- Resolution set to fixed
- Status changed from reviewing to closed
https://hg.adblockplus.org/buildtools/rev/04a0940f55a5