Re: FBR: Fix for new-updates-sync

[Mohan Boddu <mboddu@xxxxxxxxxx>]

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