#657 protonmsg: include the arch in the headers of rpm sign messages
Merged 4 years ago by mikem. Opened 4 years ago by mikeb.

file modified
+2 -1
@@ -206,7 +206,8 @@ 

               'release': kws['build']['release'],

               'rpm_name': kws['rpm']['name'],

               'rpm_version': kws['rpm']['version'],

-              'rpm_release': kws['rpm']['release']}

+              'rpm_release': kws['rpm']['release'],

+              'rpm_arch': kws['rpm']['arch']}

      queue_msg(address, props, kws)

  

  def _prep_tag_msg(address, cbtype, kws):

@@ -115,12 +115,14 @@ 

                   'release': '1'}

          rpm = {'name': 'test-pkg-subpkg',

                 'version': '2.0',

-                'release': '2'}

+                'release': '2',

+                'arch': 'x86_64'}

          sigkey = 'a1b2c3d4'

          protonmsg.prep_rpm_sign('postRPMSign', sigkey=sigkey, sighash='fedcba9876543210',

                                  build=build, rpm=rpm)

          self.assertMsg('sign.rpm', type='RPMSign', sigkey=sigkey, rpm_name=rpm['name'],

                         rpm_version=rpm['version'], rpm_release=rpm['release'],

+                        rpm_arch='x86_64',

                         **build)

  

      def test_prep_rpm_sign_no_sigkey(self):
@@ -129,7 +131,8 @@ 

                   'release': '1'}

          rpm = {'name': 'test-pkg-subpkg',

                 'version': '2.0',

-                'release': '2'}

+                'release': '2',

+                'arch': 'x86_64'}

          sigkey = ''

          protonmsg.prep_rpm_sign('postRPMSign', sigkey=sigkey, sighash='fedcba9876543210',

                                  build=build, rpm=rpm)

Including the arch in the headers may be useful for filtering.

looks good :thumbsup:

Commit a4e4174 fixes this pull-request

Pull-Request has been merged by mikem@redhat.com

4 years ago