Merge lp:~fboudra/linaro-license-protection/create-latest-symlink-for-prebuilt-job-type into lp:~linaro-automation/linaro-license-protection/trunk

[Fathi Boudra]

Create latest symlink for prebuilt job type

[James Tunnicliffe (dooferlad)]

Thanks for this. Looks OK, but a test would be nice (tests/test_publish_to_snapshots.py)

review: Needs Fixing

[Fathi Boudra (fboudra)]

it's already tested in the existing tests

[James Tunnicliffe (dooferlad)]

Ah, OK then. If it wasn't tested before (clearly no tests have changed and I assume they still pass), then lets get this merged.

review: Approve

=== modified file 'scripts/publish_to_snapshots.py'
--- scripts/publish_to_snapshots.py	2012-12-05 14:35:12 +0000
+++ scripts/publish_to_snapshots.py	2013-01-02 00:01:22 +0000
@@ -208,7 +208,11 @@
 
         return build_dir_path, target_dir_path
 
-    def create_symlink(self, args, target_dir_path):
+    def create_symlink(self, args, build_dir_path, target_dir_path):
+        if args.job_type == "prebuilt":
+            for root, dirs, files in os.walk(build_dir_path):
+                for dir in dirs:
+                    target_dir_path = os.path.join(target_dir_path, dir)
         symlink_path = os.path.dirname(target_dir_path)
         if args.job_type == "android":
             symlink_path = os.path.join(symlink_path, "lastSuccessful")
@@ -332,11 +336,13 @@
                                   sanitize=args.staging)
 
             if (args.job_type == "android" or
+                args.job_type == "prebuilt" or
                 args.job_type == "ubuntu-hwpacks" or
                 args.job_type == "ubuntu-images" or
                 args.job_type == "ubuntu-restricted" or
                 args.job_type == "ubuntu-sysroots"):
-                ret = self.create_symlink(args, target_dir_path)
+                ret = self.create_symlink(
+                    args, build_dir_path, target_dir_path)
                 if ret != PASS:
                     return ret