Thanks! Fixed the syntax error and pushed!

On 10/24/19 5:06 PM, Rick Elrod wrote:
> +1 after the syntax fix I mentioned inline.
> 
> -re
> 
> On 2019-10-24 15:56, Dusty Mabe wrote:
>> ---
>>   roles/bodhi2/backend/files/new-updates-sync | 10 ++++++----
>>   1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/roles/bodhi2/backend/files/new-updates-sync 
>> b/roles/bodhi2/backend/files/new-updates-sync
>> index d08c89301..9ad7628b8 100755
>> --- a/roles/bodhi2/backend/files/new-updates-sync
>> +++ b/roles/bodhi2/backend/files/new-updates-sync
>> @@ -25,8 +25,9 @@ RELEASES = {'f31': {'topic': 'fedora',
>>                       'modules': ['fedora', 'fedora-secondary'],
>>                       'repos': {'updates': {
>>                           'from': 'f31-updates',
>> -                        'ostrees': [{'ref': 
>> 'fedora/31/x86_64/updates/silverblue',
>> -                                     'dest': OSTREEDEST}],
>> +                        'ostrees': [{'ref': 
>> 'fedora/31/%(arch)s/updates/silverblue',
>> +                                     'dest': OSTREEDEST,
>> +                                     'arches': ['x86_64', 'ppc64le', 
>> 'aarch64']},
> 
> I think this is missing another ] at the end before the comma (to match 
> with the [ after 'ostrees':
> 
>>                           'to': [{'arches': ['x86_64', 'armhfp', 'aarch64', 
>> 'source'],
>>                                   'dest': os.path.join(FEDORADEST, '31', 
>> 'Everything')},
>>                                  {'arches': ['ppc64le', 's390x'],
>> @@ -34,8 +35,9 @@ RELEASES = {'f31': {'topic': 'fedora',
>>                                 ]},
>>                                 'updates-testing': {
>>                           'from': 'f31-updates-testing',
>> -                        'ostrees': [{'ref': 
>> 'fedora/31/x86_64/testing/silverblue',
>> -                                     'dest': OSTREEDEST}],
>> +                        'ostrees': [{'ref': 
>> 'fedora/31/%(arch)s/testing/silverblue',
>> +                                     'dest': OSTREEDEST,
>> +                                     'arches': ['x86_64', 'ppc64le', 
>> 'aarch64']},
> 
> Same here.
> 
>>                           'to': [{'arches': ['x86_64', 'aarch64', 'armhfp', 
>> 'source'],
>>                                   'dest': os.path.join(FEDORADEST, 
>> 'testing', '31', 'Everything')},
>>                                  {'arches': ['ppc64le', 's390x'],
>>
> _______________________________________________
> infrastructure mailing list -- [email protected]
> To unsubscribe send an email to [email protected]
> Fedora Code of Conduct: 
> https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: 
> https://lists.fedoraproject.org/archives/list/[email protected]
> 
_______________________________________________
infrastructure mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]

Reply via email to