Merge ~pwlars/checkbox-provider-phoronix/+git/packaging:vulkan-utils-tools-transition into ~checkbox-dev/checkbox-provider-phoronix/+git/packaging:master

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 3fa1809207e49012cd6dce120911e9138d6a53f8
Merged at revision: 939ab8998c40fddb96f5ab209ae6db40602b9825
Proposed branch: ~pwlars/checkbox-provider-phoronix/+git/packaging:vulkan-utils-tools-transition
Merge into: ~checkbox-dev/checkbox-provider-phoronix/+git/packaging:master
Diff against target: 13 lines (+1/-1)
1 file modified
debian/control (+1/-1)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+401622@code.launchpad.net

This proposal supersedes a proposal from 2021-04-14.

Description of the change

Not sure if this is the right way to proceed - it looks like in focal, vulkan-utils is a transitional package, which installs vulkan-tools. After that, it's just vulkan-tools (at least in hirsute) and vulkan-utils is no longer available.

So if we ever need to rebuild on anything before focal, this might break things. In which case I guess we'll need to maintain series-specific branches of this.

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

Let's try with:

vulkan-utils | vulkan-tools

review: Needs Fixing
Revision history for this message
Paul Larson (pwlars) wrote :

Fixed, thanks!

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/debian/control b/debian/control
2index 7ca628b..b6c6217 100644
3--- a/debian/control
4+++ b/debian/control
5@@ -35,7 +35,7 @@ Depends: apt-file,
6 opencl-headers,
7 python3,
8 unzip,
9- vulkan-utils,
10+ vulkan-utils | vulkan-tools,
11 xorg-dev
12 ${misc:Depends},
13 ${plainbox:Depends}

Subscribers

People subscribed via source and target branches