656806 - Install a clang-3.0 snapshot on Linux, OSX build machines

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[(no longer active)]

We need clang builds on Linux as well.  Rafael told me that we need to use a clang 3.0 snapshot in order to compile Firefox on Linux.  I'm sure he can point us to the correct snapshot.

I'm not sure if I need to file another bug in order to set up the clang builds or if this bug can cover both tasks.

Comment 1

[Chris AtLee [:catlee]]

Need instructions for which snapshot to use, where to get it, how to build it, etc.

Comment 2

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

I can give instructions now, but the existing bots on OS X already found some issues I am working on. Would you mind if I fixed those first? I should have something this week, with that we can deploy a single version on all platforms.

Comment 3

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

The following should do it. Is it too much work to also install this on OS X 10.5 and 10.6? I have fixed the initial issues that were found and it would be really nice to try the same version on the 3 platforms.

If this is automated, I would be happy to write the CLs if you point me to the documentation.

-------------------
REPO_REV=131621

svn co -r $REPO_REV http://llvm.org/svn/llvm-project/llvm/trunk
svn co -r $REPO_REV http://llvm.org/svn/llvm-project/cfe/trunk

cd llvm/tools/clang
ln -s ../../clang
cd ../../..
mkdir build
mkdir inst
DIR=$(pwd)
cd build
../llvm/configure --enable-optimized --prefix="$DIR/inst"
make -j8
make install
------------


Thanks!

Comment 4

[Chris AtLee [:catlee]]

hopefully I can get to this tomorrow.

Comment 5

[Chris AtLee [:catlee]]

/tools/clang-3.0 has this build on the following slaves:

linux64-ix-slave02, moz2-linux-slave10, moz2-darwin10-slave02, moz2-darwin9-slave68

I don't think I'll have time to package this up into dmgs/rpms, test and deploy it.  Back into the pool!

Comment 6

[Chris AtLee [:catlee]]

Attachment: untested manifests

it's important to include svn revision in the package version somewhere, since I imagine we'll be updating this in the future.

Comment 7

[Chris AtLee [:catlee]]

dmgs for mac are on staging-puppet:/N/staging/darwin{9,10}-i386/build/DMGs

Comment 8

[Chris AtLee [:catlee]]

Attachment: partially testing manifests

works on darwin9,darwin10

Comment 9

[Chris AtLee [:catlee]]

Attachment: Install clang snapshot on build machines

Comment 10

[Chris AtLee [:catlee]]

Attachment: rpm spec for clang

Comment 11

[Rail Aliiev [:rail]]

Comment on attachment 536424 [details] [diff] [review]
rpm spec for clang

>+%define toplevel_dir clang-3.0-r132336

What about
%define toplevel_dir %{name}-%{version}-%{release}

>+%define install_dir /tools/%{toplevel_dir}

Do you really want to install it to /tools/%{toplevel_dir}? I think, that /tools/%{name}-%{version} would be cleaner and you won't need to update symlinks in the future after every revision update.

>+%define REPO_REV 132336

Move it to the top and use for Revision:
Revision: r%{REPO_REV}

Otherwise good.

Comment 12

[Chris AtLee [:catlee]]

Attachment: rpm spec for clang

Comment 13

[Chris AtLee [:catlee]]

Attachment: clang snapshot puppet manifests

Comment 14

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 536622 [details] [diff] [review]
clang snapshot puppet manifests

I don't think you meant to include this hunk:

+node "linux64-slave02.build.sjc1.mozilla.com" inherits "centos5-x86_64-build" {
+    include staging-buildslave, vm
+}

since the box is both dead and mis-named :(

Without that hunk, r+

Comment 15

[Chris AtLee [:catlee]]

This is now being deployed to the build machines. It is available at /tools/clang-3.0/bin/clang