Merge ~sylvain-pineau/plainbox-provider-checkbox:fix-1736810 into plainbox-provider-checkbox:master

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: c1e2306ef0f5814a82bb7e86c3beed98ac8a1a98
Merged at revision: e144a971e0fc72cfbb497e993001ffa2958fb535
Proposed branch: ~sylvain-pineau/plainbox-provider-checkbox:fix-1736810
Merge into: plainbox-provider-checkbox:master
Diff against target: 13 lines (+0/-2)
1 file modified
units/submission/jobs.pxu (+0/-2)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+337365@code.launchpad.net

Description of the change

Fixes linked bug

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Tested on a system w/o dkms installed, the attachment is now:

{}

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/submission/jobs.pxu b/units/submission/jobs.pxu
2index 1565409..f6f73bc 100644
3--- a/units/submission/jobs.pxu
4+++ b/units/submission/jobs.pxu
5@@ -1,8 +1,6 @@
6 id: dkms_info_json
7 plugin: attachment
8 category_id: com.canonical.plainbox::info
9-requires:
10- package.name == 'dkms'
11 command:
12 dkms_info --format json | python3 -m plainbox dev parse dkms-info | \
13 jq '.dkms_info'

Subscribers

People subscribed via source and target branches