Bug 506493 (C192ConfSync)

Resync' configure.in between comm-central (= comm-1.9.2) and mozilla-central

RESOLVED FIXED in Thunderbird 3.1b2

Status

defect
--
trivial
RESOLVED FIXED
10 years ago
7 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Blocks 1 bug, {meta})

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(14 attachments, 6 obsolete attachments)

1.69 KB, patch
Callek
: review+
Callek
: feedback+
Details | Diff | Splinter Review
1.63 KB, patch
Callek
: review+
Details | Diff | Splinter Review
1.31 KB, patch
Callek
: review+
Details | Diff | Splinter Review
1.04 KB, patch
Callek
: review+
Details | Diff | Splinter Review
1.02 KB, patch
Callek
: review+
Details | Diff | Splinter Review
1.88 KB, patch
Callek
: review+
Details | Diff | Splinter Review
2.75 KB, patch
standard8
: review+
Details | Diff | Splinter Review
797 bytes, patch
Callek
: review+
Details | Diff | Splinter Review
1.35 KB, patch
Callek
: review+
Details | Diff | Splinter Review
1.03 KB, patch
Callek
: review+
Details | Diff | Splinter Review
1.15 KB, patch
Callek
: review+
Details | Diff | Splinter Review
4.49 KB, patch
Callek
: review+
Details | Diff | Splinter Review
3.16 KB, patch
Callek
: review+
Details | Diff | Splinter Review
2.52 KB, patch
Callek
: review+
Details | Diff | Splinter Review
Assignee

Description

10 years ago
[To be done after branching to m-c from m-1.9.1.]
Assignee

Updated

10 years ago
Depends on: 508760
Assignee

Updated

10 years ago
Depends on: 508156
Assignee

Updated

10 years ago
Blocks: 514215
Assignee

Updated

10 years ago
Depends on: 534408
Assignee

Updated

10 years ago
Depends on: 535231
Assignee

Updated

10 years ago
Depends on: 537874
Assignee

Updated

10 years ago
Depends on: 538747
Assignee

Updated

10 years ago
Target Milestone: Future → ---
Assignee

Updated

10 years ago
Depends on: 535469
Assignee

Updated

10 years ago
Depends on: 540470
Depends on: 538837

Updated

10 years ago
Depends on: 536678
Assignee

Updated

10 years ago
Blocks: 512490
Assignee

Updated

10 years ago
Depends on: 541991
Assignee

Updated

10 years ago
Depends on: 542283
Assignee

Updated

10 years ago
Depends on: 542291
Assignee

Updated

10 years ago
Depends on: 542297
Assignee

Updated

10 years ago
Alias: CcConfSync
Assignee

Updated

10 years ago
Depends on: 542692
Assignee

Updated

10 years ago
Depends on: 542767
Assignee

Updated

10 years ago
Depends on: 542770
Assignee

Updated

10 years ago
Depends on: 542779
Assignee

Updated

10 years ago
Depends on: 542786
Assignee

Updated

10 years ago
Depends on: 543159
Assignee

Updated

10 years ago
Depends on: 543212
Assignee

Updated

10 years ago
Depends on: 543230
Assignee

Updated

10 years ago
Depends on: 543380
Assignee

Updated

10 years ago
Depends on: 543382
Assignee

Updated

10 years ago
Depends on: 543893
Assignee

Updated

10 years ago
Depends on: 544172
Assignee

Updated

10 years ago
Depends on: 545055
Assignee

Updated

10 years ago
Depends on: 545057
Assignee

Updated

10 years ago
Depends on: 545058
Assignee

Updated

10 years ago
Depends on: 543294
Assignee

Updated

10 years ago
Depends on: 545133
Assignee

Updated

10 years ago
Depends on: 543295
Assignee

Updated

10 years ago
Depends on: 543296
Assignee

Updated

10 years ago
Depends on: 545313
Assignee

Updated

10 years ago
Depends on: 545320
Assignee

Updated

10 years ago
Depends on: 545343
Assignee

Updated

10 years ago
No longer depends on: 545133
Assignee

Updated

10 years ago
Depends on: 509147
Assignee

Updated

10 years ago
Depends on: 545594
Assignee

Updated

10 years ago
Depends on: 546065
Assignee

Updated

10 years ago
Depends on: 487956
Assignee

Updated

10 years ago
Depends on: 481584
Assignee

Updated

10 years ago
Depends on: 514787
Assignee

Updated

10 years ago
Depends on: 541966
Assignee

Updated

10 years ago
Depends on: 546177
Assignee

Updated

10 years ago
Depends on: 543297
Assignee

Updated

10 years ago
Depends on: 482277
Assignee

Updated

10 years ago
Depends on: 485405
Assignee

Updated

10 years ago
Depends on: 541203
Assignee

Updated

10 years ago
Depends on: 492026
Assignee

Updated

10 years ago
Depends on: 547518
Assignee

Updated

9 years ago
Depends on: 548210
Assignee

Updated

9 years ago
Depends on: 463417, 505289, 509249
Assignee

Updated

9 years ago
Depends on: 509179
Assignee

Updated

9 years ago
Depends on: 505739
Assignee

Updated

9 years ago
Depends on: 548670
Assignee

Updated

9 years ago
No longer depends on: 548670
Assignee

Updated

9 years ago
Depends on: 517559
Assignee

Updated

9 years ago
Depends on: 525331
Assignee

Updated

9 years ago
Depends on: 549129
Assignee

Updated

9 years ago
Depends on: 549293
Assignee

Updated

9 years ago
Depends on: 511791
Assignee

Updated

9 years ago
Depends on: 506845
Assignee

Updated

9 years ago
Depends on: 505504
Assignee

Updated

9 years ago
Depends on: 512504
Assignee

Comment 1

9 years ago
NB: I have no idea if we could remove more code...
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #430123 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 years ago
Depends on: 518107
Assignee

Updated

9 years ago
Depends on: 518641
Assignee

Updated

9 years ago
Depends on: 520224
Assignee

Updated

9 years ago
Depends on: 517557
Assignee

Updated

9 years ago
Depends on: 547375
Comment on attachment 430123 [details] [diff] [review]
(Av1) Copy bug 512504, Remove useless AC_DEFINE(HAVE_LIBDL), (m-1.9.2+)
[Checkin: Comment 50]

>-        AC_DEFINE(HAVE_LIBDL)))
>+        ))

Please explain what this change is all about/for.
Assignee

Updated

9 years ago
Depends on: 524139
Assignee

Updated

9 years ago
Depends on: 538753
Assignee

Updated

9 years ago
Depends on: 518401
Assignee

Updated

9 years ago
No longer depends on: 518401
Assignee

Updated

9 years ago
Depends on: 550657
Assignee

Updated

9 years ago
Depends on: 520359
Assignee

Updated

9 years ago
Depends on: 524554
Assignee

Updated

9 years ago
Depends on: 521750
Assignee

Updated

9 years ago
Depends on: 514519
Assignee

Updated

9 years ago
Depends on: 549246
Assignee

Comment 6

9 years ago
Attachment #431902 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 years ago
Depends on: 547915
Assignee

Updated

9 years ago
No longer depends on: 547915
Assignee

Updated

9 years ago
Depends on: 536567
Assignee

Updated

9 years ago
Depends on: NoC192SM
Assignee

Updated

9 years ago
Attachment #431902 - Flags: review?(bugspam.Callek) → review?(bugzilla)
Assignee

Updated

9 years ago
No longer depends on: 521750
Assignee

Updated

9 years ago
Depends on: 545755
Assignee

Updated

9 years ago
Depends on: 529319
Assignee

Comment 9

9 years ago
Attachment #432139 - Flags: review?(bugspam.Callek)
Comment on attachment 430123 [details] [diff] [review]
(Av1) Copy bug 512504, Remove useless AC_DEFINE(HAVE_LIBDL), (m-1.9.2+)
[Checkin: Comment 50]

serge, can you please file a bug on the AC_DEFINE(LIBDL) removal from m-c. or remind me in IRC later and I'll do it. I'd like to stay in sync, but this can land as-is.
Attachment #430123 - Flags: review?(bugspam.Callek) → review+
Comment on attachment 430939 [details] [diff] [review]
(Bv1) Copy (the useful part of) changeset db2c3e1c7d28
[Checkin: Comment 13]

Ironically we are flipping the configure.in value of this to "true" but our confvars actually *all* set it to false. For simplicity sake of staying in sync, I'm all for this
Attachment #430939 - Flags: review?(bugspam.Callek) → review+
Attachment #431065 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 12

9 years ago
(In reply to comment #10)

I don't understand: c-c doesn't use that define, but m-c does...
Assignee

Comment 13

9 years ago
Comment on attachment 430939 [details] [diff] [review]
(Bv1) Copy (the useful part of) changeset db2c3e1c7d28
[Checkin: Comment 13]


http://hg.mozilla.org/comm-central/rev/42abd8a80877
Attachment #430939 - Attachment description: (Bv1) Copy (the useful part of) changeset db2c3e1c7d28 → (Bv1) Copy (the useful part of) changeset db2c3e1c7d28 [Checkin: Comment 13]
Assignee

Comment 14

9 years ago
Comment on attachment 431065 [details] [diff] [review]
(Cv1) Copy bug 520359 and bug 524554
[Checkin: Comment 14]


http://hg.mozilla.org/comm-central/rev/86c1a9c1f192
Attachment #431065 - Attachment description: (Cv1) Copy bug 520359 and bug 524554 → (Cv1) Copy bug 520359 and bug 524554 [Checkin: Comment 14]
Assignee

Updated

9 years ago
Depends on: 448515
(In reply to comment #12)
> (In reply to comment #10)
> 
> I don't understand: c-c doesn't use that define, but m-c does...

Answered  this in e-mail actually...
Comment on attachment 432023 [details] [diff] [review]
(Ev1-SM) Port bug 536567
[Checkin: Comment 17]

'we' should also kill this from mail/
Attachment #432023 - Flags: review?(bugspam.Callek) → review+
Attachment #432138 - Flags: review?(bugspam.Callek) → review+
Attachment #432139 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 17

9 years ago
Comment on attachment 432023 [details] [diff] [review]
(Ev1-SM) Port bug 536567
[Checkin: Comment 17]


http://hg.mozilla.org/comm-central/rev/89a8752677ff
Attachment #432023 - Attachment description: (Ev1-SM) Port bug 536567 → (Ev1-SM) Port bug 536567 [Checkin: Comment 17]
Assignee

Comment 18

9 years ago
Comment on attachment 432138 [details] [diff] [review]
(Fv1-192) Copy bug 545755
[Checkin: Comment 18]


http://hg.mozilla.org/comm-central/rev/9a24c92bb357
Attachment #432138 - Attachment description: (Fv1-192) Copy bug 545755 → (Fv1-192) Copy bug 545755 [Checkin: Comment 18]
Assignee

Comment 19

9 years ago
Comment on attachment 432139 [details] [diff] [review]
(Gv1) Copy bug 529319
[Checkin: Comment 19]


http://hg.mozilla.org/comm-central/rev/37e3f6432a81
Attachment #432139 - Attachment description: (Gv1) Copy bug 529319 → (Gv1) Copy bug 529319 [Checkin: Comment 19]
Comment on attachment 432418 [details] [diff] [review]
(Hv1-TB-SB) Port bug 536567
[Checkin: See comment 22]


>+# Bug 506493 - Resync' configure.in between c-c and m-c; (Ev1-SM) Port bug 536567.
>+# Bug 536567 - Store the value of the per-site last file upload directories inside the memory while private browsing is active
>+#ifndef MOZILLA_1_9_2_BRANCH
>+components/contentprefs.xpt
>+#endif

I don't think we need the additional comments here.

r=Standard8 with those removed.
Attachment #432418 - Flags: review?(bugzilla) → review+
Assignee

Comment 22

9 years ago
Comment on attachment 432418 [details] [diff] [review]
(Hv1-TB-SB) Port bug 536567
[Checkin: See comment 22]


http://hg.mozilla.org/comm-central/rev/4812530ad1f6
Hv1-TB-SB, with comment 21 suggestion(s).
Attachment #432418 - Attachment description: (Hv1-TB-SB) Port bug 536567 → (Hv1-TB-SB) Port bug 536567 [Checkin: See comment 22]
Assignee

Updated

9 years ago
Depends on: 554993
Assignee

Updated

9 years ago
No longer depends on: 554993
Comment on attachment 431902 [details] [diff] [review]
(Dv1) Port bug 549246 (1.9.2+)

>diff --git a/mail/installer/package-manifest.in b/mail/installer/package-manifest.in
...
>-#ifndef XP_MACOSX
> @BINPATH@/@MOZ_APP_NAME@
> #endif
>-#endif

This is wrong, we don't build @MOZ_APP_NAME@ for Thunderbird.

>+#ifdef XP_UNIX
> #ifdef UNIX_BUT_NOT_MAC
>+@BINPATH@/icons/*.png
>+#endif
> @BINPATH@/run-mozilla.sh
>-@BINPATH@/icons/*.png
> #endif

Urgh, this is confusing. If you want to do it that way, change #ifdef UNIX_BUT_NOT_MAC to #ifndef XP_MACOSX (ditto in the ones case later on).
Attachment #431902 - Flags: review?(bugzilla) → review-
Assignee

Comment 24

9 years ago
Dv1, with comment 23 suggestion(s).


> This is wrong, we don't build @MOZ_APP_NAME@ for Thunderbird.

Indeed, MacOSX package-compare doesn't report /seamonkey nor /thunderbird.
Yet, I wonder why Firefox would have it and not our apps?
Attachment #431902 - Attachment is obsolete: true
Attachment #435063 - Flags: review?(bugzilla)
Assignee

Updated

9 years ago
Depends on: 526302
Assignee

Updated

9 years ago
Depends on: 529327
Assignee

Comment 26

9 years ago
Attachment #435413 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 years ago
Depends on: 522440
Assignee

Updated

9 years ago
Depends on: 522157
Assignee

Updated

9 years ago
Attachment #435401 - Attachment description: (Iv1) Port bug 526302 → (Iv1) Copy bug 526302
Assignee

Updated

9 years ago
Attachment #435413 - Attachment description: (Jv1) Port bug 529327 → (Jv1) Copy bug 529327
Attachment #435401 - Attachment description: (Iv1) Copy bug 526302 → (Iv1) Copy bug 526302 (m-1.9.2+)
Attachment #435401 - Flags: review?(bugspam.Callek) → review+
Attachment #435413 - Flags: review?(bugspam.Callek) → review+
Comment on attachment 435417 [details] [diff] [review]
(Kv1) Copy bug 522157, Port bug 522440
[Checkin: See comment 31]

needs !1.9.2 wrapper
Attachment #435417 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 29

9 years ago
Comment on attachment 435401 [details] [diff] [review]
(Iv1) Copy bug 526302 (m-1.9.2+)
[Checkin: Comment 29]


http://hg.mozilla.org/comm-central/rev/c830226b1820
Attachment #435401 - Attachment description: (Iv1) Copy bug 526302 (m-1.9.2+) → (Iv1) Copy bug 526302 (m-1.9.2+) [Checkin: Comment 29]
Assignee

Comment 30

9 years ago
Comment on attachment 435413 [details] [diff] [review]
(Jv1) Copy bug 529327
[Checkin: Comment 30]


http://hg.mozilla.org/comm-central/rev/871b3d26feec
Attachment #435413 - Attachment description: (Jv1) Copy bug 529327 → (Jv1) Copy bug 529327 [Checkin: Comment 30]
Assignee

Comment 31

9 years ago
Comment on attachment 435417 [details] [diff] [review]
(Kv1) Copy bug 522157, Port bug 522440
[Checkin: See comment 31]


http://hg.mozilla.org/comm-central/rev/17581dee5a68
Kv1, with comment 28 suggestion(s).
Attachment #435417 - Attachment description: (Kv1) Copy bug 522157, Port bug 522440 → (Kv1) Copy bug 522157, Port bug 522440 [Checkin: See comment 31]
Assignee

Updated

9 years ago
Depends on: 518506
Assignee

Comment 32

9 years ago
Attachment #435472 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 years ago
Depends on: 458088
Assignee

Comment 33

9 years ago
Attachment #435475 - Flags: review?(bugspam.Callek)
Comment on attachment 435063 [details] [diff] [review]
(Dv2) Port bug 549246 (1.9.2+)

AFAICT this is exactly the same as Dv1 apart from the comment in the header...
Attachment #435063 - Flags: review?(bugzilla) → review-
Assignee

Updated

9 years ago
Depends on: 486782
Assignee

Comment 35

9 years ago
Please, confirm whether you previously removed the m-1.9.2 part on purpose or by mistake...
Attachment #435803 - Flags: review?(bugspam.Callek)
Assignee

Comment 36

9 years ago
Ov1, with local merge issue fixed.
Attachment #435803 - Attachment is obsolete: true
Attachment #435805 - Flags: review?(bugspam.Callek)
Attachment #435803 - Flags: review?(bugspam.Callek)
(In reply to comment #35)
> Please, confirm whether you previously removed the m-1.9.2 part on purpose or
> by mistake...

As you seem to have suspected, it was a mistake; thanks. (I don't think it hurt anyone, but its "safer" to remain in sync here, yes)
Assignee

Updated

9 years ago
Depends on: 555684
No longer depends on: 514787
Assignee

Updated

9 years ago
Depends on: 556269
No longer depends on: 555684
Assignee

Updated

9 years ago
Depends on: 557042
Assignee

Updated

9 years ago
Depends on: 543976
Assignee

Comment 38

9 years ago
MOZ_GFX_OPTIMIZE_MOBILE is tested in
/mozilla/widget/public/nsILookAndFeel.h which is included in
/mailnews/base/src/nsMessengerWinIntegration.cpp

MOZ_PLATFORM_MAEMO is tested in
/mozilla/modules/plugin/base/public/nsPluginNativeWindow.h which is included in
/mozilla/modules/plugin/base/public/nsIPluginHost.idl which is used in
/suite/browser/test/browser/browser_pluginnotification.js
Attachment #436891 - Flags: review?(bugspam.Callek)
Assignee

Updated

9 years ago
Depends on: 464394
Attachment #435472 - Flags: review?(bugspam.Callek) → review+
Comment on attachment 435475 [details] [diff] [review]
(Mv1) Copy bug 458088
[superseded by newer bugs]

I doubt this would work; given order of things, the new vars you are adding is AFTER where its used.
Attachment #435475 - Flags: review?(bugspam.Callek) → review-
Attachment #435805 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 40

9 years ago
Comment on attachment 435472 [details] [diff] [review]
(Lv1) Copy bug 518506
[Checkin: Comment 40]


http://hg.mozilla.org/comm-central/rev/2b335d50b7f5
Attachment #435472 - Attachment description: (Lv1) Copy bug 518506 → (Lv1) Copy bug 518506 [Checkin: Comment 40]
Assignee

Comment 41

9 years ago
Comment on attachment 435805 [details] [diff] [review]
(Ov1a) Port bug 486782, Restore m-1.9.2 part after bug 543081
[Checkin: Comment 41]


http://hg.mozilla.org/comm-central/rev/655ab6620778
Attachment #435805 - Attachment description: (Ov1a) Port bug 486782, Restore m-1.9.2 part after bug 543081 → (Ov1a) Port bug 486782, Restore m-1.9.2 part after bug 543081 [Checkin: Comment 41]
Assignee

Comment 42

9 years ago
Comment on attachment 435475 [details] [diff] [review]
(Mv1) Copy bug 458088
[superseded by newer bugs]


(In reply to comment #39)

Then, I eventually commented on bug 458088.
Yet, in this case, I'm interested in sync'ing only...
r? again!?
(In reply to comment #42)
> Then, I eventually commented on bug 458088.
> Yet, in this case, I'm interested in sync'ing only...
> r? again!?

Given my latest comments in that bug; r- stands for now, even if we are syncing I'd rather not checkin broken code. If you insist you can get KaiRo's rs+ to override me.
Attachment #436891 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 44

9 years ago
Comment on attachment 436891 [details] [diff] [review]
(Pv1) Port (the useful part of) bug 464394 and bug 543976
[Checkin: Comment 44]


http://hg.mozilla.org/comm-central/rev/912bb76e5065
Attachment #436891 - Attachment description: (Pv1) Port (the useful part of) bug 464394 and bug 543976 → (Pv1) Port (the useful part of) bug 464394 and bug 543976 [Checkin: Comment 44]
Assignee

Updated

9 years ago
Depends on: 554359
Assignee

Updated

9 years ago
Depends on: 520622
Assignee

Comment 45

9 years ago
Posted patch (Qv1) Port bug 520622 (obsolete) — Splinter Review
Please, double-check the |_pwd=$_pwd/mozilla| trick I used.
Attachment #439000 - Flags: review?(bugspam.Callek)
Assignee

Comment 46

9 years ago
Qv1, with |_topsrcdirwin=$_topsrcdirwin/mozilla| too.
Attachment #439000 - Attachment is obsolete: true
Attachment #439002 - Flags: review?(bugspam.Callek)
Attachment #439000 - Flags: review?(bugspam.Callek)
Comment on attachment 439002 [details] [diff] [review]
(Qv1a) Port bug 520622
[Checkin: See comment 48]

>+    _pwd=`pwd -W`
>+    _pwd=$_pwd/mozilla
>+    _topsrcdirwin=`cd \`dirname $0\`; pwd -W`
>+    _topsrcdirwin=$_topsrcdirwin/mozilla

Correct and works, but very minor nit, just append |/mozilla| right after the last ` though I won't block on this nit if you disagree.
Attachment #439002 - Flags: review?(bugspam.Callek) → review+
Assignee

Comment 48

9 years ago
Comment on attachment 439002 [details] [diff] [review]
(Qv1a) Port bug 520622
[Checkin: See comment 48]


http://hg.mozilla.org/comm-central/rev/4d451d246928
Qv1a, with comment 47 suggestion(s).
Attachment #439002 - Attachment description: (Qv1a) Port bug 520622 → (Qv1a) Port bug 520622 [Checkin: Comment 48]
Assignee

Updated

9 years ago
Attachment #439002 - Attachment description: (Qv1a) Port bug 520622 [Checkin: Comment 48] → (Qv1a) Port bug 520622 [Checkin: See comment 48]
Assignee

Updated

9 years ago
Attachment #430123 - Flags: feedback?(bugspam.Callek)
Comment on attachment 430123 [details] [diff] [review]
(Av1) Copy bug 512504, Remove useless AC_DEFINE(HAVE_LIBDL), (m-1.9.2+)
[Checkin: Comment 50]

...Hmm you're right in that m-c (currently) needs this. But we don't so feel free to land this.
Attachment #430123 - Flags: feedback?(bugspam.Callek) → feedback+
Assignee

Comment 50

9 years ago
Comment on attachment 430123 [details] [diff] [review]
(Av1) Copy bug 512504, Remove useless AC_DEFINE(HAVE_LIBDL), (m-1.9.2+)
[Checkin: Comment 50]


http://hg.mozilla.org/comm-central/rev/3533388a3856
Attachment #430123 - Attachment description: (Av1) Copy bug 512504, Remove useless AC_DEFINE(HAVE_LIBDL) → (Av1) Copy bug 512504, Remove useless AC_DEFINE(HAVE_LIBDL), (m-1.9.2+) [Checkin: Comment 50]
Assignee

Updated

9 years ago
Assignee

Updated

9 years ago
No longer depends on: 549246

Updated

9 years ago
Depends on: 575179

Updated

9 years ago
Depends on: 575509

Updated

9 years ago
Depends on: 579704

Updated

9 years ago
Depends on: 601910

Updated

9 years ago
Depends on: 608670

Updated

9 years ago
Depends on: 608673
Assignee

Updated

9 years ago
Depends on: 613419
No longer depends on: 608670
Depends on: 613811
Depends on: 613814
Assignee

Comment 51

8 years ago
Dv2, with more of comment 23 suggestion(s).


(In reply to Serge Gautherie (:sgautherie) from comment #24)
> > This is wrong, we don't build @MOZ_APP_NAME@ for Thunderbird.
> 
> Indeed, MacOSX package-compare doesn't report /seamonkey nor /thunderbird.
> Yet, I wonder why Firefox would have it and not our apps?

I found that Thunderbird has now done that part in bug 668869.
I marked that bug as "TB2SM".


(In reply to Mark Banner (:standard8) from comment #34)
> AFAICT this is exactly the same as Dv1 apart from the comment in the
> header...

No, it had s/UNIX_BUT_NOT_MAC/XP_MACOSX/g per your comment 23.


I added SeaMonkey removed-files.in update too.
Attachment #435063 - Attachment is obsolete: true
Attachment #583424 - Flags: review?(mbanner)
Attachment #583424 - Flags: review?(bugspam.Callek)
Comment on attachment 583424 [details] [diff] [review]
(Dv3) Port '/icons/...' and 'run-mozilla.sh' parts of bug 549246
[superseded by newer bugs]

>Bug 549246 - Go back to shipping run-mozilla.sh and the "firefox" script on OS X

Actually, now I look at it again, I don't think we should follow this bug. We no longer ship the "thunderbird" script based on "mozilla.in" - the "thunderbird" file on Mac is now an actual executable, so from a shipping perspective, there is no incentive to ship run-mozilla.sh.

The build system still requires it, but I'm wondering how much of that is just historical.

I think instead we should look at seeing if we can avoid shipping run-mozilla.sh on OS X. If we can, then we may even be able to avoid shipping it on Linux, once we ship the executable there as well instead of the mozilla.in script (FF already does this).

I've pushed a proof of concept patch to try server, that I'll file a bug for if it succeeds - if it doesn't I'll come back and re-think this review (hence not changing the request for now).
Assignee

Updated

8 years ago
Depends on: 549246
Comment on attachment 583424 [details] [diff] [review]
(Dv3) Port '/icons/...' and 'run-mozilla.sh' parts of bug 549246
[superseded by newer bugs]

Ok, I've confirmed my suspicions enough that I've raised bug 715089 on not shipping run-mozilla.sh for Firefox on Mac. I haven't got time to complete it at the moment, but I definitely don't want to start shipping it for TB, and I see no reason to for SM.

The icons stuff I don't really care about as a result and I think we should just leave that alone in the TB instance at least.
Attachment #583424 - Flags: review?(mbanner)
Attachment #583424 - Flags: review?(bugspam.Callek)
Attachment #583424 - Flags: review-
Assignee

Updated

8 years ago
Depends on: 715089
Assignee

Comment 54

8 years ago
Comment on attachment 435475 [details] [diff] [review]
(Mv1) Copy bug 458088
[superseded by newer bugs]

WINCE and --disable-vista-sdk-requirements supports have since been removed (from m-c).
Attachment #435475 - Attachment is obsolete: true
Assignee

Updated

8 years ago
No longer depends on: 458088
Assignee

Updated

7 years ago
No longer depends on: 549246
Assignee

Comment 55

7 years ago
Comment on attachment 583424 [details] [diff] [review]
(Dv3) Port '/icons/...' and 'run-mozilla.sh' parts of bug 549246
[superseded by newer bugs]

(In reply to Mark Banner (:standard8) from comment #53)

> Ok, I've confirmed my suspicions enough that I've raised bug 715089 on not
> shipping run-mozilla.sh for Firefox on Mac. I haven't got time to complete
> it at the moment, but I definitely don't want to start shipping it for TB,
> and I see no reason to for SM.

TB and SM parts dropped, per bug 715089.

> The icons stuff I don't really care about as a result and I think we should
> just leave that alone in the TB instance at least.

TB part dropped.
SM part moved to bug 730254.
Attachment #583424 - Attachment description: (Dv3) Port '/icons/...' and 'run-mozilla.sh' parts of bug 549246 → (Dv3) Port '/icons/...' and 'run-mozilla.sh' parts of bug 549246 [superseded by newer bugs]
Attachment #583424 - Attachment is obsolete: true
Assignee

Updated

7 years ago
No longer depends on: 715089
Assignee

Updated

7 years ago
Attachment #435475 - Attachment description: (Mv1) Copy bug 458088 → (Mv1) Copy bug 458088 [superseded by newer bugs]
Assignee

Updated

7 years ago
Alias: CcConfSync → C192ConfSync
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Summary: Resync' configure.in between c-c and m-c → Resync' configure.in between comm-central (= comm-1.9.2) and mozilla-central
Target Milestone: --- → Thunderbird 3.1b2
You need to log in before you can comment on or make changes to this bug.