567302 - Need YASM installed on Linux boxes

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Robert O'Callahan (:roc) (email my personal email if necessary)]

We need a NASM-syntax-compatible assembler on Linux to build VP8's assembly files. The best option is YASM. Please install it on the build machines.

Comment 1

[John Ford [:jhford] CET/CEST Berlin Time]

Are there specific version requirements for the YASM deployed?

Comment 2

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: yasm.spec

spec file.  derived from the fedora yasm package

Comment 3

[John Ford [:jhford] CET/CEST Berlin Time]

building rpms with
 mock -r epel-5-i386 rebuild yasm-1.0.1-1.fc12.src.rpm 
 mock -r epel-5-x86_64 rebuild yasm-1.0.1-1.fc12.src.rpm 
on a Fedora 12 machine.

Comment 4

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: source rpm

Mock didn't work, built on moz2-linux-slave17

Comment 5

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: yasm 1.0.1 for Cent OS 5.0 on i386

i386 package built on moz2-linux-slave17

Comment 6

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: yasm 1.0.1 for Cent OS 5.0 on x86_64

package built for centos 5.0 on x86_64 processor

Comment 7

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: puppet manifest changes

puppet manifest changes required to deploy yasm 1.0.1

If we decide to go with the epel provided yasm 0.7.1 we should be able to run
 yumdownloader yasm
on each platform and substitute the version number/arch in this patch.

Comment 8

[[e-mail] (:bhearsum)]

Comment on attachment 446753 [details] [diff] [review]
puppet manifest changes

Seems alright. Let's wait until we here back re: the version before we do any landing, though.

Comment 9

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Chris Pearce will confirm but I believe 1.0.1 is good.

Comment 10

[Chris Pearce [:cpearce (Not reading bugmail)]]

YASM 1.0.1 is what we need, thanks.

Comment 11

[[e-mail] (:bhearsum)]

Comment on attachment 446753 [details] [diff] [review]
puppet manifest changes

Still looks fine :P.
In the future, as long as you've tested the patch you can carry forward feedback+ as review+.

Comment 12

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 446753 [details] [diff] [review]
puppet manifest changes

http://hg.mozilla.org/build/puppet-manifests/rev/a4224c36a073

Comment 13

[John Ford [:jhford] CET/CEST Berlin Time]

This has been landed and files put in the correct location for production and staging puppet configurations.  This will be picked up as slaves reboot.  The RPMs on the *-puppet machines have not been checked into CVS.

On a 32bit slave:

[cltbld@moz2-linux-slave02 ~]$ yasm --version
yasm 1.0.1.2326
Compiled on May 21 2010.
Copyright (c) 2001-2010 Peter Johnson and other Yasm developers.
Run yasm --license for licensing overview and summary.
[cltbld@moz2-linux-slave02 ~]$ file `which yasm`
/usr/bin/yasm: ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV), for GNU/Linux 2.6.9, dynamically linked (uses shared libs), for GNU/Linux 2.6.9, stripped

On a 64bit slave:

[cltbld@moz2-linux64-slave03 ~]$ yasm --version
yasm 1.0.1.2326
Compiled on May 21 2010.
Copyright (c) 2001-2010 Peter Johnson and other Yasm developers.
Run yasm --license for licensing overview and summary.
[cltbld@moz2-linux64-slave03 ~]$ file `which yasm`
/usr/bin/yasm: ELF 64-bit LSB executable, AMD x86-64, version 1 (SYSV), for GNU/Linux 2.6.9, dynamically linked (uses shared libs), for GNU/Linux 2.6.9, stripped

Comment 14

[John Ford [:jhford] CET/CEST Berlin Time]

I have been able to run yasm on every slave that i sampled (at random)

Marking as FIXED.