Make use of UNIFIED_SOURCES in calendar

RESOLVED FIXED in 5.5

Status

Calendar
Build Config
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

unspecified

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
I just read https://wiki.mozilla.org/Platform/Porting_to_unified_sources and it looks like a good idea.

I've tried it locally and it seems to work. I can't do a try run right now since tip is broken and requires an m-c patch.
(Assignee)

Comment 1

a year ago
Created attachment 8812472 [details] [diff] [review]
Fix - v1
Attachment #8812472 - Flags: review?(aleth)

Comment 2

a year ago
Comment on attachment 8812472 [details] [diff] [review]
Fix - v1

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

That's nice, I hadn't come across that before!
Attachment #8812472 - Flags: review?(aleth) → review+

Comment 3

a year ago
I ran the script on c-c and got:

      46 ./mailnews/base/src
      45 ./mailnews/mime/src
      45 ./mailnews/addrbook/src
      44 ./db/mork/src
      24 ./mailnews/compose/src
      22 ./mailnews/local/src
      19 ./mailnews/imap/src
      18 ./mailnews/base/util
      15 ./mailnews/import/src
      13 ./ldap/xpcom/src
      12 ./mailnews/news/src
      11 ./mailnews/import/outlook/src
      11 ./mailnews/base/search/src
       9 ./mailnews/import/oexpress
       8 ./mailnews/db/msgdb/src
       8 ./calendar/base/backend/libical
       7 ./mailnews/mime/emitters
       7 ./mailnews/jsaccount/src
       7 ./mailnews/import/becky/src \o/ UNIFIED_SOURCES \o/
       7 ./mail/components/migration/src
       6 ./mailnews/mapi/mapihook/src
       6 ./mailnews/intl \o/ UNIFIED_SOURCES \o/
       4 ./suite/profile
       4 ./mailnews/import/winlivemail
       4 ./mailnews/extensions/smime/src
       4 ./mail/components/shell
       4 ./im/components/mintrayr
       3 ./suite/shell/src
       3 ./suite/profile/migration/src
       2 ./mailnews/mime/cthandlers/glue
       2 ./mailnews/import/vcard/src
       2 ./mailnews/import/text/src
       2 ./mailnews/import/applemail/src
       2 ./mailnews/extensions/fts3/src
       2 ./mail/components/search
       1 ./suite/feeds/src
       1 ./suite/build
       1 ./suite/app
       1 ./mailnews/mime/cthandlers/vcard
       1 ./mailnews/mime/cthandlers/pgpmime
       1 ./mailnews/mapi/mapiDll
       1 ./mailnews/import/build
       1 ./mailnews/extensions/mdn/src
       1 ./mailnews/extensions/mailviews/src
       1 ./mailnews/extensions/bayesian-spam-filter/src
       1 ./mailnews/build
       1 ./mail/components/search/wsenable
       1 ./mail/components/build
       1 ./mail/app
       1 ./im/app
       1 ./db/mork/build
       1 ./calendar/base/backend/libical/build

Looks like doing this for mailnews too might be useful.

Comment 4

a year ago
https://hg.mozilla.org/comm-central/rev/aa2c8b078e9cadd736deac46a5a26376048992be
Bug 1318878 - Make use of UNIFIED_SOURCES in calendar. r=aleth

Updated

a year ago
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → 5.5
You need to log in before you can comment on or make changes to this bug.