Last Comment Bug 868393 - Require alphabetical order when assigning to order-independent moz.build variables
: Require alphabetical order when assigning to order-independent moz.build vari...
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Build Config (show other bugs)
: unspecified
: All All
: -- normal (vote)
: Thunderbird 23.0
Assigned To: Mark Banner (:standard8)
:
Mentors:
Depends on: 863069
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-03 05:48 PDT by Mark Banner (:standard8)
Modified: 2013-05-07 02:09 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Reorder in mailnews/addrbook/public/moz.build (670 bytes, patch)
2013-05-03 05:48 PDT, Mark Banner (:standard8)
Pidgeot18: review+
Details | Diff | Splinter Review

Description Mark Banner (:standard8) 2013-05-03 05:48:37 PDT
Created attachment 745113 [details] [diff] [review]
Reorder in mailnews/addrbook/public/moz.build

+++ This bug was initially created as a clone of Bug #863069 +++

In preparation for bug 863069 landing, I tried running it against c-c. All I've found so far are some LDAP idl files are not in alphabetical order.

I'll probably push bug 863069 to try sometime before it lands to check windows & linux as well, but this should mean at least mac keeps going ;-)
Comment 1 Joshua Cranmer [:jcranmer] 2013-05-03 05:59:04 PDT
Comment on attachment 745113 [details] [diff] [review]
Reorder in mailnews/addrbook/public/moz.build

Review of attachment 745113 [details] [diff] [review]:
-----------------------------------------------------------------

Ironic that it's in reverse alphabetical order right now :-)
Comment 2 Mark Banner (:standard8) 2013-05-07 02:09:09 PDT
https://hg.mozilla.org/comm-central/rev/730e8799d959

Note You need to log in before you can comment on or make changes to this bug.