Merge ~jocave/plainbox-provider-checkbox:allow-newer-luks-versions into plainbox-provider-checkbox:master

Proposed by Jonathan Cave
Status: Merged
Approved by: Jonathan Cave
Approved revision: eb3be0772bfad07098d2817841f42d9e0cd3f461
Merged at revision: bca7f9c427f10e97e0bdd45ae4e38d954b542d84
Proposed branch: ~jocave/plainbox-provider-checkbox:allow-newer-luks-versions
Merge into: plainbox-provider-checkbox:master
Diff against target: 13 lines (+1/-1)
1 file modified
bin/fde_tests.py (+1/-1)
Reviewer Review Type Date Requested Status
Sheila Miguez (community) Approve
Review via email: mp+387289@code.launchpad.net

Description of the change

Support newer LUKS version on UC18 projects and newer. Simple to change to the final Regex. Should extent support up to LUKS9 !

To post a comment you must log in.
Revision history for this message
Sheila Miguez (codersquid) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/bin/fde_tests.py b/bin/fde_tests.py
2index fd9ea14..0e658d4 100755
3--- a/bin/fde_tests.py
4+++ b/bin/fde_tests.py
5@@ -93,7 +93,7 @@ def main():
6 print(cryptinfo, '\n')
7
8 # use the type as the final arbiter of success
9- regexp = re.compile(r'type:\ *LUKS1')
10+ regexp = re.compile(r'type:\ *LUKS\d$')
11 if regexp.search(cryptinfo):
12 print('Full Disk Encryption is operational on this device')
13 else:

Subscribers

People subscribed via source and target branches