Checkbox Provider - Base

Merge ~bladernr/plainbox-provider-checkbox:fix-boot-mode-test into plainbox-provider-checkbox:master

  1. Git
  2. lp:~bladernr/plainbox-provider-checkbox
  3. fix-boot-mode-test
  4. Merge into master
Proposed by Jeff Lane 
Status: Merged
Approved by: Jeff Lane 
Approved revision: 366e3bec91fafca36f906bbd1e616728f6d01ac7
Merged at revision: 0dfd39891cb4099086cd282517fc13371d3635ef
Proposed branch: ~bladernr/plainbox-provider-checkbox:fix-boot-mode-test
Merge into: plainbox-provider-checkbox:master
Diff against target: 13 lines (+1/-1)
1 file modified
bin/boot_mode_test (+1/-1)
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Rod Smith Approve
Review via email: mp+316985@code.launchpad.net

Description of the change

fixes boot_mode_test to account for recent changes in libraries.

To post a comment you must log in.
Revision history for this message
Rod Smith (rodsmith) wrote :

Works for me.

review: Approve
Revision history for this message
Jeff Lane  (bladernr) wrote :

self-approve to clear the remaining item

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/bin/boot_mode_test b/bin/boot_mode_test
2index 330bb2b..53e4fac 100755
3--- a/bin/boot_mode_test
4+++ b/bin/boot_mode_test
5@@ -38,7 +38,7 @@ def version_check(check):
6 return code from requested check if the installed version is 16.04 or
7 newer.
8 """
9- installed_version = int(linux_distribution()[1].split()[0])
10+ installed_version = int(linux_distribution()[1].split('.')[0])
11 if installed_version < 16:
12 logging.info("This system appears to be older than 16.04 LTS so this "
13 "will not block a certification in progress.")

Subscribers

People subscribed via source and target branches