1208029 - Add some packages to the new desktop-build to support building clang

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[(no longer active)]

I need to install a couple of packages in the centos6-build docker image to support running build-clang.py in TaskCluster.

Comment 1

[(no longer active)]

Attachment: Add some packages to the new desktop-build docker image to support building clang

Comment 2

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

Comment on attachment 8665393 [details] [diff] [review]
Add some packages to the new desktop-build docker image to support building clang

Review of attachment 8665393 [details] [diff] [review]:
-----------------------------------------------------------------

This looks fine, but note that glibc-devel.i686 will be installed by bug 1189892, which anyway will conflict with this.  Also, we should uninstall subversion at the end, to avoid shipping it around in the image unnecessarily.

Let's plan to land a rebased version of this, with subversion uninstalled, right after bug 1189892.

Comment 3

[(not currently active) Ted Mielczarek]

Er, we need subversion installed to run the build-clang script, which means it needs to be in the final image.

Comment 4

[(no longer active)]

(In reply to Ted Mielczarek [:ted.mielczarek] from comment #3)
> Er, we need subversion installed to run the build-clang script, which means
> it needs to be in the final image.

Yeah, that's the whole point of this patch...

Comment 5

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

Comment on attachment 8665393 [details] [diff] [review]
Add some packages to the new desktop-build docker image to support building clang

Review of attachment 8665393 [details] [diff] [review]:
-----------------------------------------------------------------

/me smacks forehead

Comment 6

[(no longer active)]

So should I land this?

Comment 7

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

I rolled it into bug 1189892 attachment 8665909 [details] [diff] [review] which will hopefully land shortly.

Comment 8

[(no longer active)]

Thanks!

Comment 9

[(no longer active)]

The 0.1.4 version of the image doesn't have subversion installed.

Dustin, perhaps you didn't include that when you built the image?

Comment 10

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

Hm, I think I got hosed by some hg merge in bug 1189892 -- lines that were supposed to go came back, and lines that were supposed to stay left :(

Comment 11

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

Hm, the `install subversion` line was committed correctly -- I wonder if something funny went on with the image generation (maybe somehow using an older version of that image generated locally?)

I'll bump the VERSIONs, re-generate the images, verify svn is installed, docker push, and generate a patch with the new VERSIONs.

Sorry about that.  Getting automatic generation of docker images (bug 1132346) isn't quite on the critical path, but its absence sure is annoying.

Comment 12

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

[root@taskcluster-worker ~]# svn
Type 'svn help' for usage.

Comment 13

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

Attachment: MozReview Request: Bug 1208029: bump image version numbers to capture subversion install; r?ehsan

Bug 1208029: bump image version numbers to capture subversion install; r?ehsan

Comment 14

[(no longer active)]

Comment on attachment 8667864 [details]
MozReview Request: Bug 1208029: bump image version numbers to capture subversion install; r?ehsan

https://reviewboard.mozilla.org/r/20843/#review18727

Thanks for the quick fix!

Comment 15

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

both images are already docker-push'd

Comment 16

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/16b7f24721a1

Comment 17

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/16b7f24721a1