Closed
Bug 743280
Opened 13 years ago
Closed 13 years ago
Makefile.in update to use 680246 - file batch #2 - cosmetic patch edits moved to a separate bug
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla14
People
(Reporter: joey, Assigned: joey)
References
Details
Attachments
(1 file)
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #739710 +++
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → joey
Assignee | ||
Comment 1•13 years ago
|
||
Unrelated logic edits removed from patch for bug 739710 .
Declare local vars to hold common path values and use them to shorten the line length of several target commands.
replace "rm -f" with the standard make macro $(RM).
Decorate if conditionals with #{ & }# so editors can display block context. Continue the trend started by ipc/ makefiles.
Attachment #613339 -
Flags: review?(ted.mielczarek)
Comment 2•13 years ago
|
||
Comment on attachment 613339 [details] [diff] [review]
cosmetic patch edits removed from bug 739710 submitted under a new bug.
Review of attachment 613339 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks, that made both reviews much simpler!
Attachment #613339 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Try job: https://tbpl.mozilla.org/?tree=Try&rev=41b9b90e7202
win32 failure unrelated, win opt passed
Command ['ssh', '-o', 'IdentityFile=~/.ssh/trybld_dsa', 'trybld@stage.mozilla.org', 'mktemp -d'] returned non-zero exit code: 255
make.exe[1]: Leaving directory `/e/builds/moz2_slave/try-w32-dbg/build/obj-firefox/browser/installer'
Permission denied (publickey).
make.exe[1]: *** [upload] Error 2
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 4•13 years ago
|
||
Status: NEW → ASSIGNED
Keywords: checkin-needed
Target Milestone: --- → mozilla14
Version: unspecified → Trunk
Comment 5•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•