Revamp/update how SeaMonkey uses tooltool.

RESOLVED FIXED

Status

defect
--
critical
RESOLVED FIXED
4 years ago
9 months ago

People

(Reporter: ewong, Assigned: ewong)

Tracking

Trunk
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(9 attachments, 34 obsolete attachments)

23.07 KB, patch
Details | Diff | Splinter Review
11.50 KB, patch
Details | Diff | Splinter Review
4.96 KB, patch
Details | Diff | Splinter Review
9.22 KB, patch
philip.chee
: review+
Details | Diff | Splinter Review
53.29 KB, patch
Details | Diff | Splinter Review
2.95 KB, patch
Details | Diff | Splinter Review
8.81 KB, patch
ewong
: review+
Details | Diff | Splinter Review
1009 bytes, patch
Details | Diff | Splinter Review
9.13 KB, patch
philip.chee
: review+
Details | Diff | Splinter Review
Assignee

Description

4 years ago
Bug 1190256 stems from us not having the same tooltool configuration as
Moco-proper.

The inclusion of gtk3/setup.sh at http://hg.mozilla.org/comm-central/file/eace0c64cb12/build/unix/mozconfig.gtk#l25 so what's choking us.

Moco-proper's tooltool gtk3.tar.xz package has the setup.sh.  Ours
don't.  It's pointless to somehow 'fudge' it by copying our setup.sh to
gtk3/.  Somehow during the build, it deletes the setup.sh.

So, since we *need* to update our tooltool setup, might as well do it
now, especially since bug 1184951 is looming in the forefront as well.

Plan of attack:

1) Get the updated tooltool.py file and push them to the slaves.
2) Fix our buildbotcustom tooltool code.
3) *preliminary scan* implies that we won't need to do much in terms
   of modifying our mozconfigs.  

#1 is simple.

#2 is a little more complicated in that we have a bunch of manually
applied patches(without a proper review, call 'em hacks) (which 'fixes'
our previously busted releng (mock.. I'm looking at you).
Assignee

Comment 1

4 years ago
(In reply to Edmund Wong (:ewong) from comment #0)
> Bug 1190256 stems from us not having the same tooltool configuration as
> Moco-proper.
> 
> The inclusion of gtk3/setup.sh at
> http://hg.mozilla.org/comm-central/file/eace0c64cb12/build/unix/mozconfig.
> gtk#l25 so what's choking us.
> 
> Moco-proper's tooltool gtk3.tar.xz package has the setup.sh.  Ours
> don't.  It's pointless to somehow 'fudge' it by copying our setup.sh to
> gtk3/.  Somehow during the build, it deletes the setup.sh.
> 
> So, since we *need* to update our tooltool setup, might as well do it
> now, especially since bug 1184951 is looming in the forefront as well.
> 
> Plan of attack:
> 
> 1) Get the updated tooltool.py file and push them to the slaves.
> 2) Fix our buildbotcustom tooltool code.
> 3) *preliminary scan* implies that we won't need to do much in terms
>    of modifying our mozconfigs.  
> 

While the mozconfigs 'might' not need changing, then tooltool-manifests
do.
Assignee

Updated

4 years ago
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Assignee

Updated

4 years ago
Blocks: 1190256
Assignee

Comment 2

4 years ago
Posted patch buildbotcustom changes (v1) (obsolete) — Splinter Review
Attachment #8643114 - Flags: review?(bugspam.Callek)
Assignee

Comment 3

4 years ago
Posted patch buildbot-configs changes (v1) (obsolete) — Splinter Review
Attachment #8643115 - Flags: review?(bugspam.Callek)
Assignee

Comment 4

4 years ago
Posted patch c-c manifest changes (v1) (obsolete) — Splinter Review
Attachment #8643117 - Flags: review?(bugspam.Callek)
Assignee

Comment 5

4 years ago
Posted patch c-a manifest changes (v1) (obsolete) — Splinter Review
Attachment #8643119 - Flags: review?(bugspam.Callek)
Assignee

Comment 6

4 years ago
Posted patch c-b manifest changes (v1) (obsolete) — Splinter Review
Attachment #8643120 - Flags: review?(bugspam.Callek)
Assignee

Comment 7

4 years ago
Posted patch c-r manifest changes (v1) (obsolete) — Splinter Review
Attachment #8643121 - Flags: review?(bugspam.Callek)
Assignee

Comment 8

4 years ago
Those are the WIP patches.

It's my belief that c-c is currently the only tree that requires the
changes.  The others can wait for 1) train movement, or 2) uplift.
[train movement would render the c-a -> c-r patches irrelevant.)
Assignee

Comment 9

4 years ago
Addendum: I pro-actively added the Win64 for our future Win64 builds.
Assignee

Comment 10

4 years ago
Posted patch c-c manifests changes (v2) (obsolete) — Splinter Review
Fixed some nits and removed sccache.* mentions.
Attachment #8643117 - Attachment is obsolete: true
Attachment #8643117 - Flags: review?(bugspam.Callek)
Attachment #8643422 - Flags: review?(bugspam.Callek)
Assignee

Comment 11

4 years ago
Posted patch c-c manifests changes (v3) (obsolete) — Splinter Review
Forgot to remove the extra ,
Attachment #8643422 - Attachment is obsolete: true
Attachment #8643422 - Flags: review?(bugspam.Callek)
Attachment #8643424 - Flags: review?(bugspam.Callek)
Assignee

Comment 12

4 years ago
Attachment #8643114 - Attachment is obsolete: true
Attachment #8643114 - Flags: review?(bugspam.Callek)
Attachment #8649171 - Flags: review?(bugspam.Callek)
Assignee

Comment 13

4 years ago
Posted patch buildbotcustom patch (v3) (obsolete) — Splinter Review
Attachment #8649171 - Attachment is obsolete: true
Attachment #8649171 - Flags: review?(bugspam.Callek)
Attachment #8649723 - Flags: review?(bugspam.Callek)
Assignee

Comment 14

4 years ago
Posted file tooltool.py (obsolete) —
This is the tooltool.py file that is required to be put in all our
slaves.
Assignee

Updated

3 years ago
Blocks: 1260099
Assignee

Comment 15

3 years ago
Attachment #8643119 - Attachment is obsolete: true
Attachment #8643119 - Flags: review?(bugspam.Callek)
Attachment #8735695 - Flags: review?(bugspam.Callek)
Assignee

Comment 16

3 years ago
Attachment #8735695 - Attachment is obsolete: true
Attachment #8735695 - Flags: review?(bugspam.Callek)
Attachment #8735702 - Flags: review?(bugspam.Callek)
Assignee

Comment 17

3 years ago
Attachment #8643120 - Attachment is obsolete: true
Attachment #8643120 - Flags: review?(bugspam.Callek)
Attachment #8735703 - Flags: review?(bugspam.Callek)
Assignee

Comment 18

3 years ago
Attachment #8643121 - Attachment is obsolete: true
Attachment #8643121 - Flags: review?(bugspam.Callek)
Attachment #8735704 - Flags: review?(bugspam.Callek)
Assignee

Comment 19

3 years ago
Attachment #8735702 - Attachment is obsolete: true
Attachment #8735702 - Flags: review?(bugspam.Callek)
Attachment #8735711 - Flags: review?(bugspam.Callek)
Assignee

Comment 20

3 years ago
Attachment #8643424 - Attachment is obsolete: true
Attachment #8643424 - Flags: review?(bugspam.Callek)
Attachment #8735714 - Flags: review?(bugspam.Callek)
Assignee

Comment 21

3 years ago
Attachment #8735703 - Attachment is obsolete: true
Attachment #8735703 - Flags: review?(bugspam.Callek)
Attachment #8735715 - Flags: review?(bugspam.Callek)
Assignee

Comment 22

3 years ago
Attachment #8735704 - Attachment is obsolete: true
Attachment #8735704 - Flags: review?(bugspam.Callek)
Attachment #8735717 - Flags: review?(bugspam.Callek)
Assignee

Comment 23

3 years ago
Attachment #8735753 - Flags: review?(bugspam.Callek)
Assignee

Comment 24

3 years ago
Attachment #8643115 - Attachment is obsolete: true
Attachment #8643115 - Flags: review?(bugspam.Callek)
Assignee

Comment 25

3 years ago
With regards to the tooltool.token puppet and config changes, it's the
start of using the public api (which I believe, the tooltool.token
at this point, is optional).
Assignee

Comment 26

3 years ago
Attachment #8649723 - Attachment is obsolete: true
Attachment #8649723 - Flags: review?(bugspam.Callek)
Attachment #8736128 - Flags: review?(bugspam.Callek)
Assignee

Comment 27

3 years ago
Attachment #8735753 - Attachment is obsolete: true
Attachment #8735753 - Flags: review?(bugspam.Callek)
Attachment #8736130 - Flags: review?(bugspam.Callek)
Assignee

Comment 28

3 years ago
Assignee

Updated

3 years ago
Attachment #8736128 - Attachment is obsolete: true
Attachment #8736128 - Flags: review?(bugspam.Callek)
Assignee

Comment 29

3 years ago
Attachment #8736135 - Attachment is obsolete: true
Assignee

Comment 30

3 years ago
Attachment #8735754 - Attachment is obsolete: true
Attachment #8736142 - Flags: review?(bugspam.Callek)
Assignee

Comment 31

3 years ago
Attachment #8736130 - Attachment is obsolete: true
Attachment #8736130 - Flags: review?(bugspam.Callek)
Attachment #8736159 - Flags: review?(bugspam.Callek)
Assignee

Comment 32

3 years ago
Comment on attachment 8736159 [details] [diff] [review]
[puppet] support tooltool tokens (v3) [checked-in]

Pushed to puppet: (default)
https://hg.mozilla.org/SeaMonkey/puppet/rev/e28e041616db

Merged to production:
 https://hg.mozilla.org/SeaMonkey/puppet/rev/9efd7daf6f24
Attachment #8736159 - Attachment description: [puppet] support tooltool tokens (v3) → [puppet] support tooltool tokens (v3) [checked-in]

Comment 33

3 years ago
Comment on attachment 8735714 [details] [diff] [review]
[c-c] tooltool manifest changes (v4)

> # HG changeset patch
> # User Edmund Wong <ewong@pw-wspx.org>
> # Parent  709e8de02a1540e270af41bbb608b321e13f6a06
> Bug 1190773 - comm-central tooltool manifest changes

> +++ b/suite/config/tooltool-manifests/linux32/clang.manifest

> -"clang_version": "r170890"
> +"clang_version": "r247539"
Bug 1254451 - Better document toolchain versions in tooltool manifests.
"version": "clang 3.8.0/r247539"
Also please fix linux64/clang.manifest etc

> +"size": 105219872,
> +"digest": "aa8de2fa535d0667e079019c475c631ea008f1bb5228505510867255b4d9c30663e2c97e579220a575a5887aa3bcf250021b50f76b90c2fa8c65a7aa19270066",
OK.

>  "algorithm": "sha512",
> +"filename": "clang.tar.bz2",
Bug 1182727 - Part 17: Fix some incorrect file names in tooltool mnifests
"filename": "clang.tar.xz"
Also please fix /macosx64/releng.manifest etc

> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/linux32/releng.manifest
>  "size": 80458572,
>  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
This only gets you gcc 4.7.3
Bug 1175546 - Update GCC to 4.8.5
"version": "gcc 4.8.5",
"size": 81065660,
"digest": "db26f498ab56a3b5c65d7cda290cbb74174af9f2d021ca9c158f53b0382924ccf5ed9638d41eef449434aa9383a9113994d9729d9dd910321d1f35f9411eae38",
Also please fix up linux64/releng.manifest

>  "algorithm": "sha512",
> +"filename": "gcc.tar.xz",
> +"unpack": true
OK

>  "size": 11179576,
>  "digest": "91567ce8e2bb8ab0ebc60c31e90731d88a1ea889fb71bcf55c735746a60fa7610b7e040ea3d8f727b6f692ae3ee703d6f3b30cdbd76fdf5617f77d9c38aa20ed",
Do we need Bug 1242641?
"size": 11189216,
"digest": "18bc52b0599b1308b667e282abb45f47597bfc98a5140cfcab8da71dacf89dd76d0dee22a04ce26fe7ad1f04e2d6596991f9e5b01fd2aaaab5542965f596b0e6",

>  "algorithm": "sha512",
> -"filename": "gtk3.tar.xz"
> +"filename": "gtk3.tar.xz",
"setup": "setup.sh", (Bug 124264) Also please fix up linux64/releng.manifest.

> +"unpack": true
OK.

>  "size": 167175,
>  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
>  "algorithm": "sha512",
> -"filename": "sccache.tar.bz2"
> +"filename": "sccache.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/linux64/clang.manifest
> +"clang_version": "r247539"
"version": "clang 3.8.0/r247539",

> +"size": 105219872,
> +"digest": "aa8de2fa535d0667e079019c475c631ea008f1bb5228505510867255b4d9c30663e2c97e579220a575a5887aa3bcf250021b50f76b90c2fa8c65a7aa19270066",
>  "algorithm": "sha512",
> +"filename": "clang.tar.bz2",
"filename": "clang.tar.xz"

> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/linux64/releng.manifest

>  "size": 80458572,
>  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
>  "algorithm": "sha512",
> +"filename": "gcc.tar.xz",
> +"unpack": true
Bug 1175546 - Update GCC to 4.8.5
"version": "gcc 4.8.5",
"size": 81065660,
"digest": "db26f498ab56a3b5c65d7cda290cbb74174af9f2d021ca9c158f53b0382924ccf5ed9638d41eef449434aa9383a9113994d9729d9dd910321d1f35f9411eae38",

>  "size": 12057960,
>  "digest": "6105d6432943141cffb40020dc5ba3a793650bdeb3af9bd5e56d3796c5f03df9962a73e521646cd71fbfb5e266c1e74716ad722fb6055589dfb7d35175bca89e",
Bug 1242641
"size": 12072532,
"digest": "3915f8ec396c56a8a92e6f9695b70f09ce9d1582359d1258e37e3fd43a143bc974410e4cfc27f500e095f34a8956206e0ebf799b7287f0f38def0d5e34ed71c9",

>  "algorithm": "sha512",
> -"filename": "gtk3.tar.xz"
> +"filename": "gtk3.tar.xz",
Bug 1242641
"setup": "setup.sh",

> +"unpack": true
OK.

>  "size": 167175,
>  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
>  "algorithm": "sha512",
> -"filename": "sccache.tar.bz2"
> +"filename": "sccache.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/macosx64/releng.manifest
> @@ -1,17 +1,12 @@
>  "clang_version": "r247539"
"version": "clang 3.8.0/r247539",

> +"size": 121393888,
> +"digest": "6ae4e651e545538e6de326a7fb8b44b6e6d0b3acdb6a969ecb2b6f63b9995bbad2111cabf044ba575464f17f1f948d78ec92ad3a6922a7bfdf3ad6b6b2cad050",
Where does this size/digest come from? I think this should be:
"size": 121389802,
"digest": "2be6b42cfa1e92de4b49a57123f54043fec2d3cf8385276516dc6aaed99c88768ac4aebd7ce2e007ab074163523da29223436a4d1aef82f0f750f08f1b14cd71",

>  "algorithm": "sha512",
> +"filename": "clang.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/win32/l10n.manifest

>  "size": 266240,
>  "digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
>  "algorithm": "sha512",
>  "filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win32/releng.manifest

>  "size": 266240,
>  "digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
>  "algorithm": "sha512",
>  "filename": "mozmake.exe"
OK.

Do we need entries for sccache, rust, and VS 2015u1?

> +++ b/suite/config/tooltool-manifests/win64/l10n.manifest

> +"size": 266240,
> +"digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
> +"algorithm": "sha512",
> +"filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win64/releng.manifest

> +"size": 266240,
> +"digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
> +"algorithm": "sha512",
> +"filename": "mozmake.exe"
OK.

Do we need entries for sccache, rust, and VS 2015u1?
Attachment #8735714 - Flags: review?(bugspam.Callek) → review-

Comment 34

3 years ago
(In reply to Edmund Wong (:ewong) from comment #22)
> Created attachment 8735717 [details] [diff] [review]
> [c-r] tooltool manifest changes. (v4)
This appears to be identical to the comm-beta patch. Is this intentional?
Flags: needinfo?(ewong)

Comment 35

3 years ago
Comment on attachment 8735711 [details] [diff] [review]
[c-a] tooltool manifest changes (v4)

> # HG changeset patch
> # User Edmund Wong <ewong@pw-wspx.org>
> # Parent  aa7262925765e09922407bb43f0af40569f72ef1
> Bug 1190773 - comm-aurora manifest changes.

> +++ b/suite/config/tooltool-manifests/linux32/clang.manifest

> -"clang_version": "r170890"
> +"clang_version": "r247539"
"version": "clang 3.8.0/r247539"

> +"size": 105219872,
> +"digest": "aa8de2fa535d0667e079019c475c631ea008f1bb5228505510867255b4d9c30663e2c97e579220a575a5887aa3bcf250021b50f76b90c2fa8c65a7aa19270066",
>  "algorithm": "sha512",
>  "filename": "clang.tar.bz2"
"filename": "clang.tar.xz",

> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/linux32/releng.manifest
>  "size": 80458572,
>  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
>  "algorithm": "sha512",
> +"filename": "gcc.tar.xz",
> +"unpack": true
OK.

>  "size": 11179576,
>  "digest": "91567ce8e2bb8ab0ebc60c31e90731d88a1ea889fb71bcf55c735746a60fa7610b7e040ea3d8f727b6f692ae3ee703d6f3b30cdbd76fdf5617f77d9c38aa20ed",
"size": 11189216,
"digest": "18bc52b0599b1308b667e282abb45f47597bfc98a5140cfcab8da71dacf89dd76d0dee22a04ce26fe7ad1f04e2d6596991f9e5b01fd2aaaab5542965f596b0e6",

>  "algorithm": "sha512",
> +"filename": "gtk3.tar.xz",
"setup": "setup.sh",

> +"unpack": true,
OK.

>  "size": 167175,
>  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
>  "algorithm": "sha512",
> +"filename": "sccache.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/linux64/clang.manifest

> +"clang_version": "r247539"
"version": "clang 3.8.0/r247539"

> +"size": 105219872,
> +"digest": "aa8de2fa535d0667e079019c475c631ea008f1bb5228505510867255b4d9c30663e2c97e579220a575a5887aa3bcf250021b50f76b90c2fa8c65a7aa19270066",
"size": 106877168,
"digest": "1c50c6348eaf429ed59bb603cff63bcc1f870f59216dd3c234db5b1156cfd351d5ee7b820ec31be4d2661eb4213b2e0030e2ba2782b42905d1ec19c7f8bd322a",

>  "algorithm": "sha512",

> +"filename": "clang.tar.bz2",
"filename": "clang.tar.xz",

> +"unpack": true

> +++ b/suite/config/tooltool-manifests/linux64/releng.manifest

>  "size": 80458572,
>  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
>  "algorithm": "sha512",
> +"filename": "gcc.tar.xz",
> +"unpack": true
OK.

>  "size": 12057960,
>  "digest": "6105d6432943141cffb40020dc5ba3a793650bdeb3af9bd5e56d3796c5f03df9962a73e521646cd71fbfb5e266c1e74716ad722fb6055589dfb7d35175bca89e",
"size": 12072532,
"digest": "3915f8ec396c56a8a92e6f9695b70f09ce9d1582359d1258e37e3fd43a143bc974410e4cfc27f500e095f34a8956206e0ebf799b7287f0f38def0d5e34ed71c9",

>  "algorithm": "sha512",
> +"filename": "gtk3.tar.xz",
"setup": "setup.sh",

> +"unpack": true
OK.

>  "size": 167175,
>  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
>  "algorithm": "sha512",
> +"filename": "sccache.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/macosx64/clang.manifest

> +"clang_version": "r247539"
"version": "clang 3.8.0/r247539"

> +"size": 121393888,
> +"digest": "6ae4e651e545538e6de326a7fb8b44b6e6d0b3acdb6a969ecb2b6f63b9995bbad2111cabf044ba575464f17f1f948d78ec92ad3a6922a7bfdf3ad6b6b2cad050",
Where do these numbers come from?
"size": 121389802,
"digest": "2be6b42cfa1e92de4b49a57123f54043fec2d3cf8385276516dc6aaed99c88768ac4aebd7ce2e007ab074163523da29223436a4d1aef82f0f750f08f1b14cd71",

> +"algorithm": "sha512",
> +"filename": "clang.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/macosx64/releng.manifest

>  "clang_version": "r247539"
"version": "clang 3.8.0/r247539"

>  "size": 121389802,
>  "digest": "2be6b42cfa1e92de4b49a57123f54043fec2d3cf8385276516dc6aaed99c88768ac4aebd7ce2e007ab074163523da29223436a4d1aef82f0f750f08f1b14cd71",
>  "algorithm": "sha512",
> +"filename": "clang.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/win32/l10n.manifest
>  "size": 266240,
>  "digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
>  "algorithm": "sha512",
>  "filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win32/releng.manifest

>  "size": 266240,
>  "digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
>  "algorithm": "sha512",
>  "filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win64/l10n.manifest

> +"size": 266240,
> +"digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
> +"algorithm": "sha512",
> +"filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win64/releng.manifest

> +"size": 266240,
> +"digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
> +"algorithm": "sha512",
> +"filename": "mozmake.exe"
OK.
Attachment #8735711 - Flags: review?(bugspam.Callek) → review-
Assignee

Comment 37

3 years ago
(In reply to Philip Chee from comment #36)
> >  https://hg.mozilla.org/SeaMonkey/puppet/rev/9efd7daf6f24
> What is the difference between
> https://hg.mozilla.org/SeaMonkey/puppet/manifests/seamonkey-config.pp
> and
> https://hg.mozilla.org/build/puppet/manifests/seamonkey-config.pp

You don't need to review the puppet changes.  

the difference is that we no longer use build/puppet and we
strictly use SeaMonkey/puppet now.
Flags: needinfo?(ewong)
Assignee

Comment 38

3 years ago
(In reply to Philip Chee from comment #35)
> Comment on attachment 8735711 [details] [diff] [review]
> [c-a] tooltool manifest changes (v4)
> 
> > # HG changeset patch
> > # User Edmund Wong <ewong@pw-wspx.org>
> > # Parent  aa7262925765e09922407bb43f0af40569f72ef1
> > Bug 1190773 - comm-aurora manifest changes.
> 
> > +++ b/suite/config/tooltool-manifests/linux32/clang.manifest
> 
> > -"clang_version": "r170890"
> > +"clang_version": "r247539"
> "version": "clang 3.8.0/r247539"
> 
> > +"size": 105219872,
> > +"digest": "aa8de2fa535d0667e079019c475c631ea008f1bb5228505510867255b4d9c30663e2c97e579220a575a5887aa3bcf250021b50f76b90c2fa8c65a7aa19270066",
> >  "algorithm": "sha512",
> >  "filename": "clang.tar.bz2"
> "filename": "clang.tar.xz",
> 

the digest describes the bz2 file.  the xz requires
the digest and size to be changed as well. 

We might as well start using .xz. :)

I'll get an updated patch done.
Assignee

Updated

3 years ago
Duplicate of this bug: 1260099
Assignee

Comment 40

3 years ago
incidentally, those #s come from the tooltool repository and they represent
the digest and the size of the filename, which is stored in the tooltool
repository as a sha512 hash.
Assignee

Comment 41

3 years ago
We're supposed to move to VS2015u1 (or was it it u2?); but
our Win32 machines can't handle that (couldn't even have vs2013 installed on
them); but there'd be too many changes to include and the VS2015u1/2 change
needs its own bug since it requires tooltool manifest changes and
mozconfig changes.
Assignee

Comment 42

3 years ago
Attachment #8735714 - Attachment is obsolete: true
Attachment #8737452 - Flags: review?(philip.chee)

Comment 43

3 years ago
Comment on attachment 8737452 [details] [diff] [review]
[c-c] tooltool manifest changes (v5) [checked-in]

> +++ b/suite/config/tooltool-manifests/win32/releng.manifest
> -{
> -"size": 165226,
> -"digest": "79280f7595bc9e1613e05f8b2f0db3798ac739b96191e0f133e8ccd8ad149fedc84a1046e59863574189db28363a01712ae7b368ad1714e30ff88e7ebd5dad23",
> -"algorithm": "sha512",
> -"filename": "sccache.tar.bz2"
We don't need sccache here?
Attachment #8737452 - Flags: review?(philip.chee) → review+
Assignee

Comment 44

3 years ago
(In reply to Philip Chee from comment #43)
> Comment on attachment 8737452 [details] [diff] [review]
> [c-c] tooltool manifest changes (v5)
> 
> > +++ b/suite/config/tooltool-manifests/win32/releng.manifest
> > -{
> > -"size": 165226,
> > -"digest": "79280f7595bc9e1613e05f8b2f0db3798ac739b96191e0f133e8ccd8ad149fedc84a1046e59863574189db28363a01712ae7b368ad1714e30ff88e7ebd5dad23",
> > -"algorithm": "sha512",
> > -"filename": "sccache.tar.bz2"
> We don't need sccache here?

we don't use sccache on win32.
Assignee

Comment 45

3 years ago
Comment on attachment 8736141 [details] [diff] [review]
[custom] custom code changes (v4) [checked-in]

Pushed to buildbotcustom:
 https://hg.mozilla.org/build/buildbotcustom/rev/9df5790d4725
Attachment #8736141 - Attachment description: [custom] custom code changes (v4) → [custom] custom code changes (v4) [checked-in]
Attachment #8736141 - Flags: review?(bugspam.Callek)
Assignee

Comment 46

3 years ago
Comment on attachment 8736142 [details] [diff] [review]
[configs] Update configs to support new tooltool.py (v3) [checked-in]

Pushed to buildbot-configs:
https://hg.mozilla.org/build/buildbot-configs/rev/6cfd37e02e84
Attachment #8736142 - Attachment description: [configs] Update configs to support new tooltool.py (v3) → [configs] Update configs to support new tooltool.py (v3) [checked-in]
Assignee

Comment 47

3 years ago
Comment on attachment 8737452 [details] [diff] [review]
[c-c] tooltool manifest changes (v5) [checked-in]

Pushed to comm-central:
https://hg.mozilla.org/comm-central/rev/62a82225206d
Attachment #8737452 - Attachment description: [c-c] tooltool manifest changes (v5) → [c-c] tooltool manifest changes (v5) [checked-in]
Assignee

Comment 48

3 years ago
Attachment #8649877 - Attachment is obsolete: true
Attachment #8737683 - Flags: review?(bugspam.Callek)
Assignee

Comment 49

3 years ago
Attachment #8737683 - Attachment is obsolete: true
Attachment #8737683 - Flags: review?(bugspam.Callek)
Attachment #8737684 - Flags: review?(bugspam.Callek)
Assignee

Comment 50

3 years ago
Comment on attachment 8737684 [details] [diff] [review]
[puppet] update tooltool.py in packages. (v2) [checked-in]

Pushed to puppet:
https://hg.mozilla.org/SeaMonkey/puppet/rev/cdc5476c2064
Attachment #8737684 - Attachment description: [puppet] update tooltool.py in packages. (v2) → [puppet] update tooltool.py in packages. (v2) [checked-in]
Assignee

Comment 52

3 years ago
Comment on attachment 8737691 [details] [diff] [review]
[custom] fix for custom code. change ShellCommand to MockCommand. (v1) [checked-in]

Pushed to buildbotcustom:
https://hg.mozilla.org/build/buildbotcustom/rev/2418962d2be8
Attachment #8737691 - Attachment description: [custom] fix for custom code. change ShellCommand to MockCommand. (v1) → [custom] fix for custom code. change ShellCommand to MockCommand. (v1) [checked-in]
Assignee

Comment 53

3 years ago
Attachment #8737691 - Attachment is obsolete: true
Attachment #8737691 - Flags: review?(bugspam.Callek)
Attachment #8737692 - Flags: review?(bugspam.Callek)
Assignee

Comment 54

3 years ago
Comment on attachment 8737692 [details] [diff] [review]
[custom] fix for custom code. change ShellCommand to MockCommand. (v2) [checked-in]

Pushed to buildbotcustom:

(backout of previous patch):
https://hg.mozilla.org/build/buildbotcustom/rev/4b80bebf2018

Pushed:
https://hg.mozilla.org/build/buildbotcustom/rev/09fe2c1233bb
Attachment #8737692 - Attachment description: [custom] fix for custom code. change ShellCommand to MockCommand. (v2) → [custom] fix for custom code. change ShellCommand to MockCommand. (v2) [checked-in]
Assignee

Comment 56

3 years ago
Attachment #8737692 - Attachment is obsolete: true
Attachment #8737695 - Attachment is obsolete: true
Attachment #8737692 - Flags: review?(bugspam.Callek)
Assignee

Comment 57

3 years ago
Comment on attachment 8737696 [details] [diff] [review]
[custom] fix for custom code. change ShellCommand to MockCommand. (v3) [checked-in]

Pushed to buildbotcustom:

(backed out previous patch)
https://hg.mozilla.org/build/buildbotcustom/rev/937f5dfa6f68

(pushed)
https://hg.mozilla.org/build/buildbotcustom/rev/55aeadc24f9b
Attachment #8737696 - Attachment description: [custom] fix for custom code. change ShellCommand to MockCommand. (v3) → [custom] fix for custom code. change ShellCommand to MockCommand. (v3) [checked-in]
Assignee

Comment 58

3 years ago
Attachment #8735711 - Attachment is obsolete: true
Attachment #8738004 - Flags: review?(philip.chee)

Comment 59

3 years ago
Comment on attachment 8738004 [details] [diff] [review]
[c-a] tooltool manifest changes (v5)

Please fix clang.tar.bz2/clang.tar.xz before check-in
r=me for comm-aurora. a=me for CLOSED TREE

> +"version": "clang 3.8.0/r247539",
> +"size": 106877168,
> +"digest": "1c50c6348eaf429ed59bb603cff63bcc1f870f59216dd3c234db5b1156cfd351d5ee7b820ec31be4d2661eb4213b2e0030e2ba2782b42905d1ec19c7f8bd322a",
>  "algorithm": "sha512",
>  "filename": "clang.tar.bz2"
> +"unpack": true
This size/digest appears to be for "clang.tar.xz"

> +++ b/suite/config/tooltool-manifests/linux32/releng.manifest

>  "size": 80458572,
>  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
>  "algorithm": "sha512",
> -"filename": "gcc.tar.xz"
> +"filename": "gcc.tar.xz",
> +"unpack": true
OK.

> +"size": 11189216,
> +"digest": "18bc52b0599b1308b667e282abb45f47597bfc98a5140cfcab8da71dacf89dd76d0dee22a04ce26fe7ad1f04e2d6596991f9e5b01fd2aaaab5542965f596b0e6",
>  "algorithm": "sha512",
> +"filename": "gtk3.tar.xz",
> +"setup": "setup.sh",
> +"unpack": true,
OK.

>  "size": 167175,
>  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
>  "algorithm": "sha512",
> +"filename": "sccache.tar.bz2",
> +"unpack": true
OK

> +++ b/suite/config/tooltool-manifests/linux64/clang.manifest

> +"version": "clang 3.8.0/r247539",
> +"size": 106877168,
> +"digest": "1c50c6348eaf429ed59bb603cff63bcc1f870f59216dd3c234db5b1156cfd351d5ee7b820ec31be4d2661eb4213b2e0030e2ba2782b42905d1ec19c7f8bd322a",
>  "algorithm": "sha512",
> +"filename": "clang.tar.xz",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/linux64/releng.manifest
> +"version": "gcc 4.8.5",
>  "size": 80458572,
>  "digest": "e5101f9dee1e462f6cbd3897ea57eede41d23981825c7b20d91d23ab461875d54d3dfc24999aa58a31e8b01f49fb3140e05ffe5af2957ef1d1afb89fd0dfe1ad",
>  "algorithm": "sha512",
> +"filename": "gcc.tar.xz",
> +"unpack": true
OK.

>  "size": 12057960,
>  "digest": "6105d6432943141cffb40020dc5ba3a793650bdeb3af9bd5e56d3796c5f03df9962a73e521646cd71fbfb5e266c1e74716ad722fb6055589dfb7d35175bca89e",
>  "algorithm": "sha512",
> +"filename": "gtk3.tar.xz",
> +"setup": "setup.sh",
> +"unpack": true

Do we need to update gtk3 to this?
Bug 1242641 - Update gtk3 package in tooltool to specify the fontconfig path in the setup.sh
+"size": 12072532,
+"digest": "3915f8ec396c56a8a92e6f9695b70f09ce9d1582359d1258e37e3fd43a143bc974410e4cfc27f500e095f34a8956206e0ebf799b7287f0f38def0d5e34ed71c9",


>  "size": 167175,
>  "digest": "0b71a936edf5bd70cf274aaa5d7abc8f77fe8e7b5593a208f805cc9436fac646b9c4f0b43c2b10de63ff3da671497d35536077ecbc72dba7f8159a38b580f831",
>  "algorithm": "sha512",
> -"filename": "sccache.tar.bz2"
> +"filename": "sccache.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/macosx64/clang.manifest

> +"version": "clang 3.8.0/r247539",
> +"size": 121393888,
> +"digest": "6ae4e651e545538e6de326a7fb8b44b6e6d0b3acdb6a969ecb2b6f63b9995bbad2111cabf044ba575464f17f1f948d78ec92ad3a6922a7bfdf3ad6b6b2cad050",
> +"algorithm": "sha512",
> +"filename": "clang.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/macosx64/releng.manifest

> +"version": "clang 3.8.0/r247539",
>  "size": 121389802,
>  "digest": "2be6b42cfa1e92de4b49a57123f54043fec2d3cf8385276516dc6aaed99c88768ac4aebd7ce2e007ab074163523da29223436a4d1aef82f0f750f08f1b14cd71",
>  "algorithm": "sha512",
> -"filename": "clang.tar.bz2"
> +"filename": "clang.tar.bz2",
> +"unpack": true
OK.

> +++ b/suite/config/tooltool-manifests/win32/l10n.manifest
>  "size": 266240,
>  "digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
>  "algorithm": "sha512",
>  "filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win32/releng.manifest

>  "size": 266240,
>  "digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
>  "algorithm": "sha512",
>  "filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win64/l10n.manifest

> +"size": 266240,
> +"digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
> +"algorithm": "sha512",
> +"filename": "mozmake.exe"
OK.

> +++ b/suite/config/tooltool-manifests/win64/releng.manifest

> +"size": 266240,
> +"digest": "bb345b0e700ffab4d09436981f14b5de84da55a3f18a7f09ebc4364a4488acdeab8d46f447b12ac70f2da1444a68b8ce8b8675f0dae2ccf845e966d1df0f0869",
> +"algorithm": "sha512",
> +"filename": "mozmake.exe"
OK.
Attachment #8738004 - Flags: review?(philip.chee) → review+
Assignee

Comment 60

3 years ago
(In reply to Philip Chee from comment #59)
> Comment on attachment 8738004 [details] [diff] [review]
> [c-a] tooltool manifest changes (v5)
> 
> Please fix clang.tar.bz2/clang.tar.xz before check-in
> r=me for comm-aurora. a=me for CLOSED TREE
> 
> > +"version": "clang 3.8.0/r247539",
> > +"size": 106877168,
> > +"digest": "1c50c6348eaf429ed59bb603cff63bcc1f870f59216dd3c234db5b1156cfd351d5ee7b820ec31be4d2661eb4213b2e0030e2ba2782b42905d1ec19c7f8bd322a",
> >  "algorithm": "sha512",
> >  "filename": "clang.tar.bz2"
> > +"unpack": true
> This size/digest appears to be for "clang.tar.xz"
> 

My bad.  fixed.


> >  "size": 12057960,
> >  "digest": "6105d6432943141cffb40020dc5ba3a793650bdeb3af9bd5e56d3796c5f03df9962a73e521646cd71fbfb5e266c1e74716ad722fb6055589dfb7d35175bca89e",
> >  "algorithm": "sha512",
> > +"filename": "gtk3.tar.xz",
> > +"setup": "setup.sh",
> > +"unpack": true
> 
> Do we need to update gtk3 to this?
> Bug 1242641 - Update gtk3 package in tooltool to specify the fontconfig path
> in the setup.sh
> +"size": 12072532,
> +"digest":
> "3915f8ec396c56a8a92e6f9695b70f09ce9d1582359d1258e37e3fd43a143bc974410e4cfc27
> f500e095f34a8956206e0ebf799b7287f0f38def0d5e34ed71c9",
> 

Fixed.
Assignee

Comment 61

3 years ago
Attachment #8738004 - Attachment is obsolete: true
Attachment #8739285 - Flags: review+
Assignee

Comment 62

3 years ago
Comment on attachment 8739285 [details] [diff] [review]
[c-a] tooltool manifest changes (v6) [checked-in]

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:
Attachment #8739285 - Flags: approval-comm-aurora?
Assignee

Comment 63

3 years ago
(In reply to Edmund Wong (:ewong) from comment #62)
> Comment on attachment 8739285 [details] [diff] [review]
> [c-a] tooltool manifest changes (v6)
> 
> [Approval Request Comment]
> Regression caused by (bug #): 
> User impact if declined: cannot build in automation once the old tooltool repo is gone.
> Testing completed (on m-c, etc.): 
> Risk to taking this patch (and alternatives if risky):
> String changes made by this patch: none

Updated

3 years ago
Attachment #8739285 - Flags: approval-comm-aurora? → approval-comm-aurora+
Assignee

Comment 64

3 years ago
https://hg.mozilla.org/releases/comm-aurora/rev/c21a3b196cf4f771fc3083d52409a3099fb2a48e
Bug 1190773 - tooltool manifest changes to support new tooltool repo. r+a=RattyAway
Assignee

Comment 65

3 years ago
Comment on attachment 8739285 [details] [diff] [review]
[c-a] tooltool manifest changes (v6) [checked-in]

Pushed to comm-aurora:
https://hg.mozilla.org/releases/comm-aurora/rev/c21a3b196cf4
Assignee

Comment 68

3 years ago
Comment on attachment 8744771 [details] [diff] [review]
[c-a] tooltool manifest fix (v1) [checked-in]

Pushed to comm-aurora:
https://hg.mozilla.org/releases/comm-aurora/rev/54445fccee69
Assignee

Comment 69

3 years ago
Attachment #8735717 - Attachment is obsolete: true
Attachment #8735717 - Flags: review?(bugspam.Callek)
Attachment #8744813 - Flags: review?(philip.chee)
Assignee

Updated

3 years ago
Attachment #8739285 - Attachment description: [c-a] tooltool manifest changes (v6) → [c-a] tooltool manifest changes (v6) [checked-in]
Assignee

Updated

3 years ago
Attachment #8744771 - Attachment description: [c-a] tooltool manifest fix (v1) → [c-a] tooltool manifest fix (v1) [checked-in]
Assignee

Comment 70

3 years ago
(In reply to Philip Chee from comment #34)
> (In reply to Edmund Wong (:ewong) from comment #22)
> > Created attachment 8735717 [details] [diff] [review]
> > [c-r] tooltool manifest changes. (v4)
> This appears to be identical to the comm-beta patch. Is this intentional?

No.  Mistake on my part. Just fixed that.
Assignee

Comment 71

3 years ago
Comment on attachment 8735715 [details] [diff] [review]
[c-b] tooltool manifest changes (v4)

With the recent merge from c-a -> c-b, this patch is no longer needed.
Just need c-r to be reviewed.
Attachment #8735715 - Attachment is obsolete: true
Attachment #8735715 - Flags: review?(bugspam.Callek)

Comment 72

3 years ago
Comment on attachment 8744813 [details] [diff] [review]
[c-r] tooltool manifest changes(v5)

This doesn't apply to comm-release tip. It appears to come from a 2.40 build?
Attachment #8744813 - Flags: review?(philip.chee) → review-
Assignee

Comment 73

3 years ago
(In reply to Philip Chee from comment #72)
> Comment on attachment 8744813 [details] [diff] [review]
> [c-r] tooltool manifest changes(v5)
> 
> This doesn't apply to comm-release tip. It appears to come from a 2.40 build?

Ah, right.  ;/ Sorry.
Assignee

Comment 74

3 years ago
Attachment #8744813 - Attachment is obsolete: true
Attachment #8746900 - Flags: review?(philip.chee)
Assignee

Comment 75

3 years ago
the VS changes will be done in Bug 1092468. (mainly because we're not
ready to use VS2015u2 yet.)
Attachment #8746900 - Attachment is obsolete: true
Attachment #8746900 - Flags: review?(philip.chee)
Attachment #8746901 - Flags: review?(philip.chee)

Comment 76

3 years ago
Comment on attachment 8746901 [details] [diff] [review]
[c-r] tooltool manifest changes (v6)

r=me with the missing newlines fixed and sccache either restored or explained why we don't need this.

> suite/config/tooltool-manifests/linux32/releng.manifest

> +"version": "gcc 4.8.5",
> +"size": 81065660,
> +"digest": "db26f498ab56a3b5c65d7cda290cbb74174af9f2d021ca9c158f53b0382924ccf5ed9638d41eef449434aa9383a9113994d9729d9dd910321d1f35f9411eae38",
>  "algorithm": "sha512",
> -"filename": "setup.sh"
> +"filename": "gcc.tar.xz",
> +"unpack": "true"
I don't know if this makes a difference but in other parts of this file and in other manifests true is not "quoted"

> suite/config/tooltool-manifests/linux64/clang.manifest
Ah we don't use sccache here I remember.
> suite/config/tooltool-manifests/macosx64/releng.manifest
But we do here? What are the reasons we do or don't use sccache?

> suite/config/tooltool-manifests/win32/l10n.manifest
> -]
> +]
> \ No newline at end of file
Please add back all the missing NL at the ends of all the files you touched.

> suite/config/tooltool-manifests/win32/releng.manifest
> -"size": 165226,
> -"digest": "79280f7595bc9e1613e05f8b2f0db3798ac739b96191e0f133e8ccd8ad149fedc84a1046e59863574189db28363a01712ae7b368ad1714e30ff88e7ebd5dad23",
> -"algorithm": "sha512",
> -"filename": "sccache.tar.bz2"
What reason are we removing this? Thunderbird and Firefox both still have this.

> suite/config/tooltool-manifests/win64/l10n.manifest
> suite/config/tooltool-manifests/win64/releng.manifest
Missing NewLines?
Missing sccache?
Attachment #8746901 - Flags: review?(philip.chee) → review+
Assignee

Comment 77

3 years ago
Posted patch [c-r] tooltool manifests (v7) (obsolete) — Splinter Review
Attachment #8746901 - Attachment is obsolete: true
Attachment #8747583 - Flags: review?(philip.chee)
Assignee

Comment 78

3 years ago
Posted patch [c-r] tooltool manifests (v7) (obsolete) — Splinter Review
Attachment #8747583 - Attachment is obsolete: true
Attachment #8747583 - Flags: review?(philip.chee)
Attachment #8747586 - Flags: review?(philip.chee)

Comment 79

3 years ago
Comment on attachment 8747586 [details] [diff] [review]
[c-r] tooltool manifests (v7)

I don't know what happened but your linux32/releng.manifest and linux64/releng.manifest look totally broken.
Attachment #8747586 - Flags: review?(philip.chee) → review-

Comment 81

3 years ago
Comment on attachment 8748649 [details] [diff] [review]
[c-r] tooltool manifest changes (v7c) (for real) r+Ratty a+Ratty [checked-in]

r=Ratty a=Ratty for comm-release (with minor fixes
Attachment #8748649 - Attachment description: [c-r] tooltool manifest changes (v7) (for real) → [c-r] tooltool manifest changes (v7c) (for real) r+Ratty a+Ratty
Attachment #8748649 - Attachment filename: bug_1190773.diff → bug_1190773c-rV3.diff
Attachment #8748649 - Flags: review?(philip.chee)
Attachment #8748649 - Flags: review+
Attachment #8748649 - Flags: approval-comm-release+

Comment 82

3 years ago
> [c-r] tooltool manifest changes (v7c) (for real) r+Ratty a+Ratty
> r=Ratty a=Ratty for comm-release (with minor fixes)
http://hg.mozilla.org/releases/comm-release/rev/07a4595692ad

Updated

3 years ago
Attachment #8748649 - Attachment description: [c-r] tooltool manifest changes (v7c) (for real) r+Ratty a+Ratty → [c-r] tooltool manifest changes (v7c) (for real) r+Ratty a+Ratty [checked-in]
Assignee

Updated

3 years ago
Severity: normal → critical
Assignee

Comment 83

3 years ago
Apart from the post-land-review patches, should I consider this bug fixed?

Puppet, build-configs, buildbotcustom and the tooltool manifest changes
have all been checked in.

Am I missing anything?
Assignee

Updated

3 years ago
Blocks: 1092468
Assignee

Updated

9 months ago
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Assignee

Updated

9 months ago
Attachment #8737684 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 months ago
Attachment #8736141 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 months ago
Attachment #8736142 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 months ago
Attachment #8736159 - Flags: review?(bugspam.Callek)
You need to log in before you can comment on or make changes to this bug.