Closed Bug 725429 Opened 9 years ago Closed 9 years ago

Clean up *makefiles.sh, Feb 2012 edition

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla13

People

(Reporter: emorley, Assigned: emorley)

Details

(Whiteboard: [qa-])

Attachments

(1 file, 1 obsolete file)

Patch v1.1
17.67 KB, patch
Details | Diff | Splinter Review 
Another month, another round of forgotten *makefiles.sh changes.

Using an updated version of my script from bug 696498...

# Duplicate:

dom/locales/Makefile
netwerk/locales/Makefile
security/manager/locales/Makefile
toolkit/locales/Makefile
tools/update-packaging/Makefile


# Invalid:

toolkit/components/startup/tests/Makefile


# Omitted:

accessible/tests/mochitest/pivot/Makefile
accessible/tests/mochitest/textcaret/Makefile
b2g/branding/official/content/Makefile
b2g/branding/official/locales/Makefile
b2g/branding/unofficial/content/Makefile
b2g/branding/unofficial/locales/Makefile
browser/base/content/test/newtab/Makefile
browser/components/thumbnails/Makefile
browser/components/thumbnails/test/Makefile
browser/devtools/tilt/Makefile
browser/devtools/tilt/test/Makefile
build/mobile/robocop/Makefile
content/base/test/websocket_hybi/Makefile
dom/indexedDB/test/unit/Makefile
dom/network/Makefile
dom/network/interfaces/Makefile
dom/network/src/Makefile
dom/network/tests/Makefile
dom/power/Makefile
dom/power/test/Makefile
dom/wifi/Makefile
gfx/graphite2/src/Makefile
layout/media/Makefile
mfbt/Makefile
mobile/android/branding/aurora/content/Makefile
mobile/android/branding/beta/content/Makefile
mobile/android/branding/nightly/content/Makefile
mobile/android/branding/official/content/Makefile
mobile/android/branding/unofficial/content/Makefile
mozglue/tests/Makefile
testing/mochitest/roboextender/Makefile
toolkit/components/startup/tests/browser/Makefile
toolkit/devtools/debugger/tests/Makefile
toolkit/mozapps/update/common/Makefile
toolkit/mozapps/update/test_svc/Makefile
Attached patch Patch v1 (obsolete) — — Splinter Review 
rs=build

Builds fine locally/doesn't trigger |set -o errexit|.
Attachment #595582 - Flags: review+
Whiteboard: [autoland-try]
Whiteboard: [autoland-try] → [autoland-in-queue]
Autoland Patchset:
	Patches: 595582
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/rev/a50dc892ca65
Try run started, revision a50dc892ca65. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=a50dc892ca65
Try run for a50dc892ca65 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=a50dc892ca65
Results (out of 211 total builds):
    success: 178
    warnings: 19
    failure: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/autolanduser@mozilla.com-a50dc892ca65
Whiteboard: [autoland-in-queue]
Attached patch Patch v1.1 — — Splinter Review 
Slight change to toolkit/mozapps/readstrings/Makefile and toolkit/mozapps/update/common/Makefile inclusion logic, post bug 715749.

Was pushing to try anyway, so included just in case:
https://tbpl.mozilla.org/?tree=Try&rev=138e2820a551
Attachment #595582 - Attachment is obsolete: true
https://hg.mozilla.org/integration/mozilla-inbound/rev/7bbb3087e971
Flags: in-testsuite-
Whiteboard: [qa-]
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/7bbb3087e971
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.