Merge ~canonical-kernel-team/+git/autotest-client-tests:magalilemes/fix-2032807 into ~canonical-kernel-team/+git/autotest-client-tests:master

Proposed by Magali Lemes do Sacramento
Status: Merged
Merge reported by: Po-Hsu Lin
Merged at revision: bfe89c3f663ce2f8b5dc1564e03146dd92bece4d
Proposed branch: ~canonical-kernel-team/+git/autotest-client-tests:magalilemes/fix-2032807
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 28 lines (+5/-5)
1 file modified
ubuntu_bpf/ubuntu_bpf.py (+5/-5)
Reviewer Review Type Date Requested Status
Po-Hsu Lin Approve
Review via email: mp+449761@code.launchpad.net

Description of the change

Tested on j:linux-aws-fips.

To post a comment you must log in.
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

+1
Sorry for the trouble.

review: Approve
Revision history for this message
Francis Ginther (fginther) wrote :

+1

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/ubuntu_bpf/ubuntu_bpf.py b/ubuntu_bpf/ubuntu_bpf.py
2index 516bc58..313e95e 100644
3--- a/ubuntu_bpf/ubuntu_bpf.py
4+++ b/ubuntu_bpf/ubuntu_bpf.py
5@@ -43,11 +43,6 @@ class ubuntu_bpf(test.test):
6 else:
7 pkgs.extend(['clang', 'llvm', 'lld'])
8
9- # Build header first (LP: #2031400)
10- if not self.series in ['trusty', 'xenial', 'bionic', 'focal']:
11- cmd = "make -C linux/ headers"
12- utils.system_output(cmd, retain_output=True)
13-
14 cmd = 'yes "" | DEBIAN_FRONTEND=noninteractive apt-get install --yes --force-yes ' + ' '.join(pkgs)
15 self.results = utils.system_output(cmd, retain_output=True)
16
17@@ -79,6 +74,11 @@ class ubuntu_bpf(test.test):
18 # Assist local testing by restoring the linux repo to vanilla.
19 self.extract()
20
21+ # Build header first (LP: #2031400)
22+ if not self.series in ['trusty', 'xenial', 'bionic', 'focal']:
23+ cmd = "make -C linux/ headers"
24+ utils.system_output(cmd, retain_output=True)
25+
26 # clean source tree so changes from debian.foo/reconstruct
27 # (e.g. deleting files) are applied
28 os.chdir('linux')

Subscribers

People subscribed via source and target branches

to all changes: