Closed
Bug 615301
Opened 14 years ago
Closed 13 years ago
remove the build-tools package from opsi/puppet and blow away dir
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dustin, Assigned: dustin)
References
Details
Attachments
(2 files)
3.78 KB,
patch
|
bhearsum
:
review+
|
Details | Diff | Splinter Review |
1.14 KB,
patch
|
bhearsum
:
review+
|
Details | Diff | Splinter Review |
buildbot-tac.py is the only thing that is used in /tools/build-tools, which is installed by this package. This makes it pretty fiddly to upgrade the script, and there's no reason to build a package for a simple python script.
Bug 611149 will remove the need for this package, at which point we should remove the package and delete the directory it installs into.
Assignee | ||
Comment 1•14 years ago
|
||
Changing to depend on the dynamic slave alloc bug, in hopes that it will involve removing buildbot-tac.py.
Assignee | ||
Updated•14 years ago
|
Assignee | ||
Comment 2•13 years ago
|
||
I think this is ready to do now - win64 isn't in OPSI, so it doesn't block this bug.
No longer depends on: 629690
Assignee | ||
Comment 3•13 years ago
|
||
Ben, how confident are you that we don't use things under /tools/build-tools for other purposes? A check of the build-central MXR doesn't reveal anything obvious:
http://mxr.mozilla.org/build-central/search?string=build-tools
Attachment #542350 -
Flags: review?(bhearsum)
Assignee | ||
Comment 4•13 years ago
|
||
Huh, I'm not sure *where* this is installed with OPSI. I guess we can just let it die.
Assignee | ||
Comment 5•13 years ago
|
||
Attachment #542362 -
Flags: review?(bhearsum)
Comment 6•13 years ago
|
||
(In reply to comment #4)
> Huh, I'm not sure *where* this is installed with OPSI. I guess we can just
> let it die.
It's installed on the Windows build machines, to d:\tools:
D:\>dir tools
Volume in drive D is tools
Volume Serial Number is 048A-CE48
Directory of D:\tools
02/23/2010 03:12 PM <DIR> .
02/23/2010 03:12 PM <DIR> ..
02/23/2010 03:12 PM <DIR> .hg
02/23/2010 03:12 PM 3,633 .hgtags
02/23/2010 03:12 PM <DIR> breakpad
02/23/2010 03:12 PM <DIR> buildbot-helpers
02/23/2010 03:12 PM <DIR> buildfarm
02/23/2010 03:12 PM <DIR> cdmaker
02/23/2010 03:12 PM <DIR> clobberer
02/23/2010 03:12 PM <DIR> release
02/23/2010 03:12 PM <DIR> stage
1 File(s) 3,633 bytes
10 Dir(s) 18,198,835,200 bytes free
(In reply to comment #3)
> Created attachment 542350 [details] [diff] [review] [review]
> m615301-puppet-manifests-p1-r1.patch
>
> Ben, how confident are you that we don't use things under /tools/build-tools
> for other purposes? A check of the build-central MXR doesn't reveal
> anything obvious:
> http://mxr.mozilla.org/build-central/search?string=build-tools
I don't know of anything else on POSIX machines that uses /tools/build-tools. On Windows, the OPSI host key generator (http://hg.mozilla.org/build/opsi-package-sources/file/fcce4578d03f/hostkey-generator/CLIENT_DATA/hostkey.ins) uses d:\tools to reset the host key in fresh clones.
Updated•13 years ago
|
Attachment #542350 -
Flags: review?(bhearsum) → review+
Updated•13 years ago
|
Attachment #542362 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 7•13 years ago
|
||
Both are landed, and puppet servers restarted.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•