Checkbox Provider - Base

Merge ~pwlars/plainbox-provider-checkbox:ipmctl-amd64-only into plainbox-provider-checkbox:master

  1. Git
  2. lp:~pwlars/plainbox-provider-checkbox
  3. ipmctl-amd64-only
  4. Merge into master
Proposed by Paul Larson
Status: Merged
Approved by: Paul Larson
Approved revision: 59d3c2b97f9050c4fde7dd602a6c173cda9e8f10
Merged at revision: e0e3b2a162bffa21851ad0aca96f6b57aa11a499
Proposed branch: ~pwlars/plainbox-provider-checkbox:ipmctl-amd64-only
Merge into: plainbox-provider-checkbox:master
Diff against target: 10 lines (+1/-1)
1 file modified
units/nvdimm/packaging.pxu (+1/-1)
Reviewer Review Type Date Requested Status
Sylvain Pineau Approve
Review via email: mp+380695@code.launchpad.net

Description of the change

Checkbox deb package installation is currently broken on everything except for amd64, because it depends on ipmctl (which only seems to exist on amd64)

I can't find a precedent for this in other providers, so hopefully this will work.

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

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/nvdimm/packaging.pxu b/units/nvdimm/packaging.pxu
2index fe9cfb2..8b048c5 100644
3--- a/units/nvdimm/packaging.pxu
4+++ b/units/nvdimm/packaging.pxu
5@@ -1,4 +1,4 @@
6 # nvdimms require ipmctl for health check jobs
7 unit: packaging meta-data
8 os-id: debian
9-Depends: ipmctl
10+Depends: ipmctl [amd64]

Subscribers

People subscribed via source and target branches