Closed Bug 573722 Opened 13 years ago Closed 11 years ago

Add --disable-auto-deps to nightly mozconfigs


(Release Engineering :: General, defect, P5)



(Not tracked)



(Reporter: catlee, Assigned: coop)


(Whiteboard: [mozconfig])


(2 files)

Spin-off from bug 573199.

For builds that are always full clobbers, like nightly builds and try builds, disabling auto deps has a significant speed win on windows.

We should figure out how to get our nightly builds building with this option.  Slightly complicated by the fact that the nightly builds share the same mozconfigs as the regular depend builds.
Priority: -- → P5
catlee - is this still needed?
Whiteboard: [mozconfig]
cc'ing joey so he can give it a review
(In reply to Mike Taylor [:bear] from comment #1)
> catlee - is this still needed?

Doubtful that its *needed* but certainly helpful.

I bet even without the depend-nightly distinction in mozconfigs yet, this would help us in release mozconfigs. (if done there, be sure to add to the release whitelist)
perhaps we could start by putting some indication in the environment that the build is a nightly build rather than a depend build?
Assignee: nobody → coop
Component: Release Engineering → Release Engineering: Automation (General)
QA Contact: release → catlee
Attachment #624533 - Flags: review?(bear)
Attachment #624533 - Flags: review?(bear) → review+
This patch checks for the ENV var set by the buildbotcustom patch in attachment 624533 [details] [diff] [review].
Attachment #624541 - Flags: review?(joey)
Comment on attachment 624541 [details] [diff] [review]
Set --disable-auto-deps for Windows nightlies
Attachment #624541 - Flags: checked-in?
Attachment #624541 - Flags: checked-in?
Comment on attachment 624541 [details] [diff] [review]
Set --disable-auto-deps for Windows nightlies

Review of attachment 624541 [details] [diff] [review]:

## For known full-clobber builds on Windows

Are there any other variables available that could be used to test nightly & clobber ?
If nighties are able to support dependency builds later testing only $IS_NIGHTLY would always assume clobber.

Minor but I would indent ac_add_options as a visual cue that it belongs to the conditional.
Attachment #624541 - Flags: review?(joey) → review+
Comment on attachment 624541 [details] [diff] [review]
Set --disable-auto-deps for Windows nightlies
Attachment #624541 - Flags: checked-in+
Windows builds are green on inbound. This will ride the trains.
Closed: 11 years ago
Resolution: --- → FIXED
Product: → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.