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):

https://codereview.adblockplus.org/29329056/

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:2 Changed on 10/12/2015 at 03:50:26 PM by kzar

  • Description modified (diff)

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

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from kzar.
 
Note: See TracTickets for help on using tickets.