1318878 - Make use of UNIFIED_SOURCES in calendar

Status: RESOLVED FIXED

(Calendar :: Build Config, defect)

Description

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

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.

Comment 1

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v1

Comment 2

[aleth [:aleth]]

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!

Comment 3

[aleth [:aleth]]

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

[aleth [:aleth]]

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