Mir

Merge lp:~kdub/mir/fix-1379478 into lp:mir

Proposed by Kevin DuBois
Status: Merged
Approved by: Daniel van Vugt
Approved revision: no longer in the source branch.
Merged at revision: 1976
Proposed branch: lp:~kdub/mir/fix-1379478
Merge into: lp:mir
Diff against target: 37 lines (+8/-2)
1 file modified
tests/unit-tests/test_shared_library_prober.cpp (+8/-2)
To merge this branch: bzr merge lp:~kdub/mir/fix-1379478
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Daniel van Vugt Approve
Colin Watson (community) Approve
Review via email: mp+237842@code.launchpad.net

Commit message

fix arm64 package by adding arm64 to the shared library prober test
fixes: lp: #1379478

Description of the change

fix arm64 package by adding arm64 to the shared library prober test
fixes: lp: #1379478

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) wrote :

I can confirm that this builds cleanly on am2, the nearest thing we so far have to an arm64 porter-box.

review: Approve
Revision history for this message
Kevin DuBois (kdub) wrote :

> I can confirm that this builds cleanly on am2, the nearest thing we so far
> have to an arm64 porter-box.

Thanks for checking! Haven't gotten sbuild to the point of being cooperative just yet for this arch.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

Failure due to "virtual memory exhausted: Cannot allocate memory". Rekicking.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Daniel van Vugt (vanvugt) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'tests/unit-tests/test_data/libarm64.so'
2Binary files tests/unit-tests/test_data/libarm64.so 1970-01-01 00:00:00 +0000 and tests/unit-tests/test_data/libarm64.so 2014-10-09 18:59:11 +0000 differ
3=== modified file 'tests/unit-tests/test_shared_library_prober.cpp'
4--- tests/unit-tests/test_shared_library_prober.cpp 2014-10-01 03:41:21 +0000
5+++ tests/unit-tests/test_shared_library_prober.cpp 2014-10-09 18:59:11 +0000
6@@ -161,6 +161,7 @@
7 EXPECT_CALL(report, loading_library(FilenameMatches("libamd64.so")));
8 EXPECT_CALL(report, loading_library(FilenameMatches("libarmhf.so")));
9 EXPECT_CALL(report, loading_library(FilenameMatches("libi386.so")));
10+ EXPECT_CALL(report, loading_library(FilenameMatches("libarm64.so")));
11
12 mir::libraries_for_path(library_path, report);
13 }
14@@ -170,7 +171,10 @@
15 using namespace testing;
16 NiceMock<MockSharedLibraryProberReport> report;
17
18- bool armhf_failed{false}, amd64_failed{false}, i386_failed{false};
19+ bool armhf_failed{false};
20+ bool amd64_failed{false};
21+ bool i386_failed{false};
22+ bool arm64_failed{false};
23
24 ON_CALL(report, loading_failed(FilenameMatches("libamd64.so"), _))
25 .WillByDefault(InvokeWithoutArgs([&amd64_failed]() { amd64_failed = true; }));
26@@ -178,8 +182,10 @@
27 .WillByDefault(InvokeWithoutArgs([&armhf_failed]() { armhf_failed = true; }));
28 ON_CALL(report, loading_failed(FilenameMatches("libi386.so"), _))
29 .WillByDefault(InvokeWithoutArgs([&i386_failed]() { i386_failed = true; }));
30+ ON_CALL(report, loading_failed(FilenameMatches("libarm64.so"), _))
31+ .WillByDefault(InvokeWithoutArgs([&arm64_failed]() { arm64_failed = true; }));
32
33 mir::libraries_for_path(library_path, report);
34
35- EXPECT_TRUE(i386_failed || amd64_failed || armhf_failed);
36+ EXPECT_TRUE(i386_failed || amd64_failed || armhf_failed || arm64_failed);
37 }

Subscribers

People subscribed via source and target branches