Re: FBR: Fix for new-updates-sync

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Better wording of the comment:

diff --git a/roles/bodhi2/backend/files/new-updates-sync b/roles/bodhi2/backend/files/new-updates-sync
index 3619e6c..ae78b45 100755
--- a/roles/bodhi2/backend/files/new-updates-sync
+++ b/roles/bodhi2/backend/files/new-updates-sync
@@ -367,10 +367,14 @@ def determine_last_link(release, repo):
     if repo == 'rawhide':
         source_path = os.path.join(RAWHIDESOURCE,
                                    RELEASES[release]['repos'][repo]['from'])
+        #Since latest-Fedora-Rawhide is a symlink pointing to just the
+        #compose dir rather than its full path, we need the absolute path
+        #of the compose rather than relative path
+        target = os.path.realpath(source_path)
     else:
         source_path = os.path.join(SOURCE,
                                    RELEASES[release]['repos'][repo]['from'])
-    target = os.readlink(source_path)
+        target = os.readlink(source_path)
     logger.info('Release %s, repo %s, target %s', release, repo, target)
     RELEASES[release]['repos'][repo]['from'] = target
     return target


On Tue, Aug 28, 2018 at 12:51 PM Mohan Boddu <mboddu@xxxxxxxxxx> wrote:
diff --git a/roles/bodhi2/backend/files/new-updates-sync b/roles/bodhi2/backend/files/new-updates-sync
index 3619e6c..5f00b29 100755
--- a/roles/bodhi2/backend/files/new-updates-sync
+++ b/roles/bodhi2/backend/files/new-updates-sync
@@ -367,10 +367,13 @@ def determine_last_link(release, repo):
     if repo == 'rawhide':
         source_path = os.path.join(RAWHIDESOURCE,
                                    RELEASES[release]['repos'][repo]['from'])
+        #Since latest-Fedora-Rawhide is not a symlink
+        #we need the absolute path
+        target = os.path.realpath(source_path)
     else:
         source_path = os.path.join(SOURCE,
                                    RELEASES[release]['repos'][repo]['from'])
-    target = os.readlink(source_path)
+        target = os.readlink(source_path)
     logger.info('Release %s, repo %s, target %s', release, repo, target)
     RELEASES[release]['repos'][repo]['from'] = target
     return target

_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux