Merge lp:~gz/juju-ci-tools/tsv_project_relative into lp:juju-ci-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 1191
Proposed branch: lp:~gz/juju-ci-tools/tsv_project_relative
Merge into: lp:juju-ci-tools
Diff against target: 34 lines (+3/-3)
2 files modified
git_gate.py (+2/-2)
tests/test_git_gate.py (+1/-1)
To merge this branch: bzr merge lp:~gz/juju-ci-tools/tsv_project_relative
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+280378@code.launchpad.net

Description of the change

Support relative path of dependencies.tsv

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'git_gate.py'
2--- git_gate.py 2015-12-12 01:31:25 +0000
3+++ git_gate.py 2015-12-12 01:51:43 +0000
4@@ -84,7 +84,7 @@
5 print_now("Getting {} and dependencies using go".format(dep))
6 go("get", "-v", "-d", dep)
7 if args.tsv_path:
8- tsv_path = os.path.join(gopath, "src", args.tsv_path)
9+ tsv_path = os.path.join(gopath, "src", final_project, args.tsv_path)
10 print_now("Getting dependencies using godeps from {}".format(tsv_path))
11 godeps = SubcommandRunner(os.path.join(gopath, "bin", "godeps"), goenv)
12 godeps("-u", tsv_path)
13@@ -144,7 +144,7 @@
14 help="Go import path of package needed to for build or testing.")
15 dep_group.add_argument(
16 "--tsv-path",
17- help="Path to dependencies.tsv file for project relative to src dir.")
18+ help="Path to dependencies.tsv file relative to project dir.")
19 args = parser.parse_args(args)
20 if args.project_url is None and not args.go_get_all:
21 parser.error("Must supply either --project-url or --go-get-all")
22
23=== modified file 'tests/test_git_gate.py'
24--- tests/test_git_gate.py 2015-12-12 01:31:25 +0000
25+++ tests/test_git_gate.py 2015-12-12 01:51:43 +0000
26@@ -231,7 +231,7 @@
27 def test_tsv_test(self):
28 args = self.make_args("git.testing/project",
29 project_url="https://git.testing/project",
30- tsv_path="git.testing/project/dependencies.tsv")
31+ tsv_path="dependencies.tsv")
32 git_gate.go_test(args, "/tmp/fake")
33 self.assertEqual(self.actions, [
34 ('print', 'Getting and installing godeps'),

Subscribers

People subscribed via source and target branches