Checkbox Provider - Base

Merge ~jocave/plainbox-provider-checkbox:debsums-exe-for-snaps into plainbox-provider-checkbox:master

Proposed by Jonathan Cave on 2021-10-12

Status:	Merged
Approved by:	Jonathan Cave on 2021-10-13
Approved revision:	bc0bd3c12f9c836cc4323dbb33e6081927cb55e1
Merged at revision:	58ea2751b3ab0fac24ffd89fd34cb50707c272e7
Proposed branch:	~jocave/plainbox-provider-checkbox:debsums-exe-for-snaps
Merge into:	plainbox-provider-checkbox:master
Diff against target:	13 lines (+1/-1) 1 file modified units/miscellanea/jobs.pxu (+1/-1)
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Maciej Kisielewski		2021-10-12	Approve on 2021-10-13
Review via email: mp+410076@code.launchpad.net

Modify depends for debsums job so that it can be used when checkbox is shipped via a snap and the application is shipped in the checkbox-core-snap.

Revision history for this message

Maciej Kisielewski (kissiel) wrote on 2021-10-13:

review: Approve

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

People subscribed via source and target branches

to all changes:

1	diff --git a/units/miscellanea/jobs.pxu b/units/miscellanea/jobs.pxu
2	index a1eab66..dc9a0ac 100644
3	--- a/units/miscellanea/jobs.pxu
4	+++ b/units/miscellanea/jobs.pxu
5	@@ -469,7 +469,7 @@ category_id: com.canonical.plainbox::miscellanea
6	estimated_duration: 60
7	id: miscellanea/debsums
8	user: root
9	-requires: package.name == 'debsums'
10	+requires: executable.name == 'debsums'
11	command: debsums -c
12	_summary: Check the MD5 sums of installed Debian packages
13	_description: Verify installed Debian package files against MD5 checksum lists from /var/lib/dpkg/info/*.md5sums.