Merge lp:~liuyq0307/lava-dispatcher/fix-853657 into lp:lava-dispatcher

Proposed by Yongqin Liu
Status: Merged
Merged at revision: 115
Proposed branch: lp:~liuyq0307/lava-dispatcher/fix-853657
Merge into: lp:lava-dispatcher
Diff against target: 14 lines (+3/-1)
1 file modified
lava_dispatcher/utils.py (+3/-1)
To merge this branch: bzr merge lp:~liuyq0307/lava-dispatcher/fix-853657
Reviewer Review Type Date Requested Status
Paul Larson (community) Approve
Review via email: mp+75955@code.launchpad.net

Description of the change

fix bug of 853657 that only the first image is cached.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lava_dispatcher/utils.py'
--- lava_dispatcher/utils.py 2011-09-15 07:02:08 +0000
+++ lava_dispatcher/utils.py 2011-09-19 06:39:24 +0000
@@ -53,7 +53,9 @@
53 else:53 else:
54 file_location = download(url, path)54 file_location = download(url, path)
55 try:55 try:
56 os.makedirs(os.path.dirname(cache_loc))56 cache_dir = os.path.dirname(cache_loc)
57 if not os.path.exists(cache_dir):
58 os.makedirs(cache_dir)
57 os.link(file_location, cache_loc)59 os.link(file_location, cache_loc)
58 except OSError, err:60 except OSError, err:
59 #errno 18 is Invalid cross-device link61 #errno 18 is Invalid cross-device link

Subscribers

People subscribed via source and target branches