860235 - Install zlib.i686 and libstdc++.i686 in the android test mock environment

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

[Mike Hommey [:glandium]]

bug 859705 requires to run a binary that is result of the android build on the android test mock environment. Since the build environment is 32-bits, the binary is 32-bits, while the android test mock environment, aiui is 64-bits. Which leads that binary to fail to execute because its dependencies are not all there:

Running ldd /builds/tegra-314/test/build/tests/bin/host/szip
        linux-gate.so.1 =>  (0x008c0000)
        libz.so.1 => not found
        libstdc++.so.6 => not found
        libm.so.6 => /lib/libm.so.6 (0x004bd000)
        libgcc_s.so.1 => /lib/libgcc_s.so.1 (0x00a50000)
        libc.so.6 => /lib/libc.so.6 (0x00110000)
        /lib/ld-linux.so.2 (0x00404000)

Thus, please install the two missing libraries, zlib.i686 and libstdc++.i686

Comment 1

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

found in triage.

Comment 2

[Justin Wood (:Callek)]

Attachment: [puppet] v1

So, I'm not really a fan of throwing packages in this manifest, but I can't even come up with a valid package module name for this stuff, in a way I could have the foopy install it properly.

I'm open to suggestions on how to do this differently as well.

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 736169 [details] [diff] [review]
[puppet] v1

packages::ia32libs is exactly what you want ;)

Comment 4

[Justin Wood (:Callek)]

Attachment: [puppet] v2

As we all discussed in IRC yesterday, this is the plan as rail suggested it.

concerns raised were:
* (dustin raised:) possible mis-match of the .i686 libc with what the x86 mock is using, since the x64 repo snapshot is a few months newer than the x86 snapshot.
** Obviated by glandiums assertion that it is fine this way.

* (callek raised:) ia32libs module not being suitable for these inserts
** dustin and rail both think it is fair to use it, since it is for basic x86 needs on x64 machines. If we needed a wider swatch of libs we could/would use a different module/impl but this is cleanest for our needs now.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 736981 [details] [diff] [review]
[puppet] v2

Ship it!

Comment 6

[Justin Wood (:Callek)]

https://hg.mozilla.org/build/puppet/rev/4b4ec8f835f8

Should take no more than 15-20 minutes to fully deploy the way we have puppet setup on these machines.

Comment 7

[Justin Wood (:Callek)]

I backed this out on friday -- forgot to comment/reopen. We broke stuff, I'll attempt relanding when I have a clear head.

Comment 8

[Justin Wood (:Callek)]

relanded:

https://hg.mozilla.org/build/puppet/rev/637816f3c008
and the foopy init.pp as well :(
https://hg.mozilla.org/build/puppet/rev/5ab0f346e8e7