Re: FBR : To fix rawhide sync location

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

 



On 08/29/2018 07:45 AM, Mohan Boddu wrote:
> Yeah, good catch by Dusty.
> 
> Hopefully, this is the right thing to do:
> 
> diff --git a/roles/bodhi2/backend/files/new-updates-sync
> b/roles/bodhi2/backend/files/new-updates-sync
> index e625f50..468d269 100755
> --- a/roles/bodhi2/backend/files/new-updates-sync
> +++ b/roles/bodhi2/backend/files/new-updates-sync
> @@ -344,7 +344,10 @@ def sync_single_repo(release, repo):
>      for archdef in RELEASES[release]['repos'][repo]['to']:
>          for arch in archdef['arches']:
>              destarch = 'SRPMS' if arch == 'source' else arch
> -            dest_path = os.path.join(archdef['dest'], destarch)
> +            if repo == 'rawhide':
> +                dest_path = os.path.join(archdef['dest'], destarch, 'os')
> +            else:
> +                dest_path = os.path.join(archdef['dest'], destarch)
>              results.append(sync_single_repo_arch(release, repo, arch,
> dest_path))
> 
>      stats = collect_stats(results)

I guess a reluctant +1.... but overall I'd prefer to just go back to the
nightly syncing rawhide instead of doing it here.

I know it's nice to have one place to change, but it seems pretty odd to
have rawhide wait for a script on a totally different machine with a
different function to run before it can complete.

kevin

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
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