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
=== added file 'tests/unit-tests/test_data/libarm64.so'
0Binary 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 differ0Binary 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
=== modified file 'tests/unit-tests/test_shared_library_prober.cpp'
--- tests/unit-tests/test_shared_library_prober.cpp 2014-10-01 03:41:21 +0000
+++ tests/unit-tests/test_shared_library_prober.cpp 2014-10-09 18:59:11 +0000
@@ -161,6 +161,7 @@
161 EXPECT_CALL(report, loading_library(FilenameMatches("libamd64.so")));161 EXPECT_CALL(report, loading_library(FilenameMatches("libamd64.so")));
162 EXPECT_CALL(report, loading_library(FilenameMatches("libarmhf.so")));162 EXPECT_CALL(report, loading_library(FilenameMatches("libarmhf.so")));
163 EXPECT_CALL(report, loading_library(FilenameMatches("libi386.so")));163 EXPECT_CALL(report, loading_library(FilenameMatches("libi386.so")));
164 EXPECT_CALL(report, loading_library(FilenameMatches("libarm64.so")));
164165
165 mir::libraries_for_path(library_path, report);166 mir::libraries_for_path(library_path, report);
166}167}
@@ -170,7 +171,10 @@
170 using namespace testing;171 using namespace testing;
171 NiceMock<MockSharedLibraryProberReport> report;172 NiceMock<MockSharedLibraryProberReport> report;
172173
173 bool armhf_failed{false}, amd64_failed{false}, i386_failed{false};174 bool armhf_failed{false};
175 bool amd64_failed{false};
176 bool i386_failed{false};
177 bool arm64_failed{false};
174178
175 ON_CALL(report, loading_failed(FilenameMatches("libamd64.so"), _))179 ON_CALL(report, loading_failed(FilenameMatches("libamd64.so"), _))
176 .WillByDefault(InvokeWithoutArgs([&amd64_failed]() { amd64_failed = true; }));180 .WillByDefault(InvokeWithoutArgs([&amd64_failed]() { amd64_failed = true; }));
@@ -178,8 +182,10 @@
178 .WillByDefault(InvokeWithoutArgs([&armhf_failed]() { armhf_failed = true; }));182 .WillByDefault(InvokeWithoutArgs([&armhf_failed]() { armhf_failed = true; }));
179 ON_CALL(report, loading_failed(FilenameMatches("libi386.so"), _))183 ON_CALL(report, loading_failed(FilenameMatches("libi386.so"), _))
180 .WillByDefault(InvokeWithoutArgs([&i386_failed]() { i386_failed = true; }));184 .WillByDefault(InvokeWithoutArgs([&i386_failed]() { i386_failed = true; }));
185 ON_CALL(report, loading_failed(FilenameMatches("libarm64.so"), _))
186 .WillByDefault(InvokeWithoutArgs([&arm64_failed]() { arm64_failed = true; }));
181187
182 mir::libraries_for_path(library_path, report);188 mir::libraries_for_path(library_path, report);
183189
184 EXPECT_TRUE(i386_failed || amd64_failed || armhf_failed);190 EXPECT_TRUE(i386_failed || amd64_failed || armhf_failed || arm64_failed);
185}191}

Subscribers

People subscribed via source and target branches