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
1=== modified file 'backends/packaging-apt-dpkg.py'
2--- backends/packaging-apt-dpkg.py 2017-01-25 19:34:48 +0000
3+++ backends/packaging-apt-dpkg.py 2017-05-02 17:42:21 +0000
4@@ -1450,16 +1450,12 @@
5 apport.warning("Error: can't find signing_key_fingerprint at %s"
6 % ppa_archive_url)
7 continue
8- argv = ['gpg', '--no-options',
9- '--no-default-keyring',
10- '--no-auto-check-trustdb',
11- '--keyring',
12- os.path.join(trusted_d,
13- '%s.gpg' % origin),
14- ]
15- argv += ['--quiet', '--batch',
16- '--keyserver', 'hkp://keyserver.ubuntu.com:80/',
17- '--recv', signing_key_fingerprint]
18+ argv = ['apt-key', '--keyring',
19+ os.path.join(trusted_d, '%s.gpg' % origin),
20+ 'adv', '--quiet',
21+ '--keyserver', 'keyserver.ubuntu.com', '--recv-key',
22+ signing_key_fingerprint]
23+
24 if subprocess.call(argv) != 0:
25 apport.warning('Unable to import key for %s' %
26 ppa_archive_url)
27
28=== modified file 'test/test_backend_apt_dpkg.py'
29--- test/test_backend_apt_dpkg.py 2017-01-26 23:53:27 +0000
30+++ test/test_backend_apt_dpkg.py 2017-05-02 17:42:21 +0000
31@@ -1112,6 +1112,7 @@
32 # install GPG key for ddebs
33 keyring_dir = os.path.join(self.configdir, 'Foonux 1.2', 'trusted.gpg.d')
34 os.makedirs(keyring_dir, exist_ok=True)
35+ shutil.copy('/usr/share/keyrings/ubuntu-archive-keyring.gpg', keyring_dir)
36 subprocess.check_call(['apt-key', '--keyring', os.path.join(keyring_dir, 'ddebs.ubuntu.com.gpg'),
37 'adv', '--quiet', '--keyserver', 'keyserver.ubuntu.com', '--recv-key', 'C8CAB6595FDFF622'],
38 stdout=subprocess.DEVNULL)

Subscribers

People subscribed via source and target branches