Merge lp:~xnox/apport/no-to-keybox into lp:~apport-hackers/apport/trunk

Proposed by Dimitri John Ledkov
Status: Merged
Merged at revision: 3140
Proposed branch: lp:~xnox/apport/no-to-keybox
Merge into: lp:~apport-hackers/apport/trunk
Diff against target: 38 lines (+7/-10)
2 files modified
backends/packaging-apt-dpkg.py (+6/-10)
test/test_backend_apt_dpkg.py (+1/-0)
To merge this branch: bzr merge lp:~xnox/apport/no-to-keybox
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+323513@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

I've uploaded this to artful and then will merge it with upstream.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'backends/packaging-apt-dpkg.py'
--- backends/packaging-apt-dpkg.py 2017-01-25 19:34:48 +0000
+++ backends/packaging-apt-dpkg.py 2017-05-02 17:42:21 +0000
@@ -1450,16 +1450,12 @@
1450 apport.warning("Error: can't find signing_key_fingerprint at %s"1450 apport.warning("Error: can't find signing_key_fingerprint at %s"
1451 % ppa_archive_url)1451 % ppa_archive_url)
1452 continue1452 continue
1453 argv = ['gpg', '--no-options',1453 argv = ['apt-key', '--keyring',
1454 '--no-default-keyring',1454 os.path.join(trusted_d, '%s.gpg' % origin),
1455 '--no-auto-check-trustdb',1455 'adv', '--quiet',
1456 '--keyring',1456 '--keyserver', 'keyserver.ubuntu.com', '--recv-key',
1457 os.path.join(trusted_d,1457 signing_key_fingerprint]
1458 '%s.gpg' % origin),1458
1459 ]
1460 argv += ['--quiet', '--batch',
1461 '--keyserver', 'hkp://keyserver.ubuntu.com:80/',
1462 '--recv', signing_key_fingerprint]
1463 if subprocess.call(argv) != 0:1459 if subprocess.call(argv) != 0:
1464 apport.warning('Unable to import key for %s' %1460 apport.warning('Unable to import key for %s' %
1465 ppa_archive_url)1461 ppa_archive_url)
14661462
=== modified file 'test/test_backend_apt_dpkg.py'
--- test/test_backend_apt_dpkg.py 2017-01-26 23:53:27 +0000
+++ test/test_backend_apt_dpkg.py 2017-05-02 17:42:21 +0000
@@ -1112,6 +1112,7 @@
1112 # install GPG key for ddebs1112 # install GPG key for ddebs
1113 keyring_dir = os.path.join(self.configdir, 'Foonux 1.2', 'trusted.gpg.d')1113 keyring_dir = os.path.join(self.configdir, 'Foonux 1.2', 'trusted.gpg.d')
1114 os.makedirs(keyring_dir, exist_ok=True)1114 os.makedirs(keyring_dir, exist_ok=True)
1115 shutil.copy('/usr/share/keyrings/ubuntu-archive-keyring.gpg', keyring_dir)
1115 subprocess.check_call(['apt-key', '--keyring', os.path.join(keyring_dir, 'ddebs.ubuntu.com.gpg'),1116 subprocess.check_call(['apt-key', '--keyring', os.path.join(keyring_dir, 'ddebs.ubuntu.com.gpg'),
1116 'adv', '--quiet', '--keyserver', 'keyserver.ubuntu.com', '--recv-key', 'C8CAB6595FDFF622'],1117 'adv', '--quiet', '--keyserver', 'keyserver.ubuntu.com', '--recv-key', 'C8CAB6595FDFF622'],
1117 stdout=subprocess.DEVNULL)1118 stdout=subprocess.DEVNULL)

Subscribers

People subscribed via source and target branches