Merge ~nacc/git-ubuntu:lp1731513-fix-cache-symlink-to-parent-dir into git-ubuntu:master

Proposed by Nish Aravamudan
Status: Merged
Merged at revision: 6e7088f206c8c3cdcd9d0efe2de3028711fadd4a
Proposed branch: ~nacc/git-ubuntu:lp1731513-fix-cache-symlink-to-parent-dir
Merge into: git-ubuntu:master
Diff against target: 28 lines (+6/-4)
1 file modified
gitubuntu/build.py (+6/-4)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
git-ubuntu developers Pending
Review via email: mp+333556@code.launchpad.net

Description of the change

Make jenkins happy.

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:6e7088f206c8c3cdcd9d0efe2de3028711fadd4a
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/206/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Style Check
    SUCCESS: Unit Tests
    SUCCESS: Integration Tests
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/206/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/gitubuntu/build.py b/gitubuntu/build.py
2index 8905ccb..072d3e7 100644
3--- a/gitubuntu/build.py
4+++ b/gitubuntu/build.py
5@@ -198,9 +198,9 @@ def _symlink_paths_into_parent_dir(paths):
6 unwind_unlink_paths = []
7 for path in paths:
8 try:
9- old_path = _symlink_into_parent_dir(path)
10- if old_path is not None:
11- unwind_unlink_paths.append(old_path)
12+ new_path = _symlink_into_parent_dir(path)
13+ if new_path is not None:
14+ unwind_unlink_paths.append(new_path)
15 except:
16 for path in unwind_unlink_paths:
17 os.unlink(path)
18@@ -243,7 +243,9 @@ def fetch_orig_from_cache(changelog, source, dl_cache=None):
19 )
20 return None # XXX decide - see docstring
21
22- return dsc.all_tarball_paths
23+ orig_paths = dsc.all_tarball_paths
24+ _symlink_paths_into_parent_dir(orig_paths)
25+ return orig_paths
26
27
28 def fetch_orig_from_pristine_tar(changelog, source, repo):

Subscribers

People subscribed via source and target branches