Bug 496236 (C191ConfSync)

[Tracking] Resync' configure.in between c-1.9.1 and m-1.9.1

RESOLVED FIXED

Status

MailNews Core
Build Config
--
trivial
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

({fixed-seamonkey2.0.4, meta})

Trunk
fixed-seamonkey2.0.4, meta
Dependency tree / graph
Bug Flags:
in-litmus -

Thunderbird Tracking Flags

(thunderbird3.0 .4-fixed)

Details

(Whiteboard: [ToDo: Ov2...], URL)

Attachments

(18 attachments, 4 obsolete attachments)

1.52 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
1.17 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
1.36 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
4.83 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
9.93 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
1.62 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
1.36 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
857 bytes, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
1.04 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
2.55 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
2.05 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
8.86 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
7.13 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
1.01 KB, patch
Robert Kaiser
: review+
Details | Diff | Splinter Review
18.58 KB, patch
Details | Diff | Splinter Review
889 bytes, patch
Callek
: review+
Details | Diff | Splinter Review
1.68 KB, patch
Callek
: review+
Details | Diff | Splinter Review
987 bytes, patch
standard8
: review+
Details | Diff | Splinter Review
Comment hidden (empty)
(Assignee)

Comment 1

8 years ago
Created attachment 381429 [details] [diff] [review]
(Av1) Bug 478297
[Checkin: Comment 2]
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #381429 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Keywords: meta

Updated

8 years ago
Attachment #381429 - Flags: review?(kairo) → review+
(Assignee)

Comment 2

8 years ago
Comment on attachment 381429 [details] [diff] [review]
(Av1) Bug 478297
[Checkin: Comment 2]


http://hg.mozilla.org/comm-central/rev/5afa025fbe2b
Attachment #381429 - Attachment description: (Av1) Bug 478297 → (Av1) Bug 478297 [Checkin: Comment 2]
(Assignee)

Updated

8 years ago
(Assignee)

Comment 3

8 years ago
Created attachment 390672 [details] [diff] [review]
(Bv1) Bug 481780, m-1.9.1 patch only
[Checkin: Comment 4]
Attachment #390672 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Blocks: 506493

Updated

8 years ago
Attachment #390672 - Flags: review?(kairo) → review+
(Assignee)

Comment 4

8 years ago
Comment on attachment 390672 [details] [diff] [review]
(Bv1) Bug 481780, m-1.9.1 patch only
[Checkin: Comment 4]


http://hg.mozilla.org/comm-central/rev/f1db251753d2
Attachment #390672 - Attachment description: (Bv1) Bug 481780, m-1.9.1 patch only → (Bv1) Bug 481780, m-1.9.1 patch only [Checkin: Comment 4]
(Assignee)

Updated

8 years ago
No longer depends on: 481926
(Assignee)

Comment 5

8 years ago
Created attachment 390764 [details] [diff] [review]
(Cv1) Bug 451621 + bug 469225
[Checkin: Comment 7]

Maybe you could give me a rs+ for the (simple) patches I'm doing right here?
Attachment #390764 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Depends on: 451621, 469225
Flags: in-litmus-
(Assignee)

Updated

8 years ago
No longer depends on: 469225
(Assignee)

Updated

8 years ago
Depends on: 469225

Updated

8 years ago
Attachment #390764 - Flags: review?(kairo) → review+

Comment 6

8 years ago
Comment on attachment 390764 [details] [diff] [review]
(Cv1) Bug 451621 + bug 469225
[Checkin: Comment 7]

I don't see too much use in this ARM support stuff for us, but in case it doesn't hurt to have it.

And no, I don't rubberstamp anything. But I'd prefer a larger patch covering multiple bugs and closing the bug report. I hate multi-patch bug reports without clear targets of when they are FIXED.
(Assignee)

Comment 7

8 years ago
Comment on attachment 390764 [details] [diff] [review]
(Cv1) Bug 451621 + bug 469225
[Checkin: Comment 7]


http://hg.mozilla.org/comm-central/rev/36cc638ec544
Attachment #390764 - Attachment description: (Cv1) Bug 451621 + bug 469225 → (Cv1) Bug 451621 + bug 469225 [Checkin: Comment 7]
(Assignee)

Comment 8

8 years ago
Created attachment 390873 [details] [diff] [review]
(Dv1) Bug 462908
[Checkin: Comment 12]
Attachment #390873 - Flags: review?(kairo)

Updated

8 years ago
Attachment #390873 - Flags: review?(kairo) → review+

Comment 9

8 years ago
Comment on attachment 390873 [details] [diff] [review]
(Dv1) Bug 462908
[Checkin: Comment 12]

I wonder a bit how much of those vars about libs we don't build in comm-central but mozilla/ we really need, but for the moment, we seem to just have most of this in, so OK.
(Assignee)

Comment 10

8 years ago
(In reply to comment #6)
> I don't see too much use in this ARM support stuff for us, but in case it
> doesn't hurt to have it.

(In reply to comment #9)
> I wonder a bit how much of those vars about libs we don't build in comm-central
> but mozilla/ we really need, but for the moment, we seem to just have most of
> this in, so OK.

Indeed: I'm not trying to figure out what c-c might actually use or not,
I'm taking a "configure.in should be the same in c-c as in m-c, except where c-c needs something different" approach :-|
(Like what you seem to want in the buildbot repos.)

Still, it's not to late to stop me if you would prefer to do it the opposite way.

Comment 11

8 years ago
(In reply to comment #10)
> Still, it's not to late to stop me if you would prefer to do it the opposite
> way.

No, let's do the sync unless in those cases where we already did remove blocks of things from our build system (which are only a few so far). In some cases, we might have code that needs some of the flags, I don't have a good view of what we are checking in c-c code and what we'd never check there at all. If I had a lot of free time, I'd probably do some checking on that and cut the unneeded parts of configure and other build system files, but then I would have looked into all that porting myself as well ;-)
(Assignee)

Comment 12

8 years ago
Comment on attachment 390873 [details] [diff] [review]
(Dv1) Bug 462908
[Checkin: Comment 12]


http://hg.mozilla.org/comm-central/rev/48ed71f32871
Attachment #390873 - Attachment description: (Dv1) Bug 462908 → (Dv1) Bug 462908 [Checkin: Comment 12]
(Assignee)

Comment 13

8 years ago
Created attachment 392147 [details] [diff] [review]
(Ev1) Bug 476448 (m-1.9.1 patch)
[Checkin: Comment 14]
Attachment #392147 - Flags: review?(kairo)

Updated

8 years ago
Attachment #392147 - Flags: review?(kairo) → review+
(Assignee)

Comment 14

8 years ago
Comment on attachment 392147 [details] [diff] [review]
(Ev1) Bug 476448 (m-1.9.1 patch)
[Checkin: Comment 14]


http://hg.mozilla.org/comm-central/rev/ccaf6c38ce07
Attachment #392147 - Attachment description: (Ev1) Bug 476448 (m-1.9.1 patch) → (Ev1) Bug 476448 (m-1.9.1 patch) [Checkin: Comment 14]
(Assignee)

Comment 15

8 years ago
Created attachment 392513 [details] [diff] [review]
(Fv1) Changeset 18864:4611778ba22a
[Checkin: Comment 19]

From
{
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/4611778ba22a
Bug 97954: Don't cite Makefile variables in the value for LIBXUL_DIST. r=bsmedberg
}
Bug that bug doesn't quote it.
Attachment #392513 - Flags: review?(kairo)
(Assignee)

Comment 16

8 years ago
Created attachment 393366 [details] [diff] [review]
(Gv1) Bug 438397 (+ 2 unrelated nits)
Attachment #393366 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Depends on: 438397

Comment 17

8 years ago
Comment on attachment 392513 [details] [diff] [review]
(Fv1) Changeset 18864:4611778ba22a
[Checkin: Comment 19]

bug 97954 has a lot of checkin comments referring to it ;-)
Attachment #392513 - Flags: review?(kairo) → review+

Comment 18

8 years ago
Comment on attachment 393366 [details] [diff] [review]
(Gv1) Bug 438397 (+ 2 unrelated nits)

I don't see much use in introducing a flag that nothing besides XULRunner-specific files are using, as those get it in the Mozilla build system anyway.
And are those two nits fixed by any Mozilla checkins as well?
(Assignee)

Comment 19

8 years ago
Comment on attachment 392513 [details] [diff] [review]
(Fv1) Changeset 18864:4611778ba22a
[Checkin: Comment 19]


http://hg.mozilla.org/comm-central/rev/b6f5800cf03b
Attachment #392513 - Attachment description: (Fv1) Changeset 18864:4611778ba22a → (Fv1) Changeset 18864:4611778ba22a [Checkin: Comment 19]
(Assignee)

Comment 20

8 years ago
(In reply to comment #18)

> (From update of attachment 393366 [details] [diff] [review])
> I don't see much use in introducing a flag that nothing besides
> XULRunner-specific files are using, as those get it in the Mozilla build system
> anyway.

You're right about m-c/XulRunner.
/js/src/ has it (unused) too, but maybe this is required to be in sync'!?

Wrt c-c,
I'm still following my comment 10 idea, as it's the only way I can do it by comparing the files;
if you don't want unused code anymore, then you will have to use your new tool you emailed me about and/to tell me which (new code) bug to actually port.

> And are those two nits fixed by any Mozilla checkins as well?

They are synchronizations from current m-1.9.1:
I didn't check history wrt the blank line removal;
the indentation was checked in wrong in
http://hg.mozilla.org/comm-central/rev/32acc7b19dbe
(Assignee)

Comment 21

8 years ago
Created attachment 393437 [details] [diff] [review]
(Gv2) 2 random nits
[Checkin: Comment 22]

Per irc discussion: not porting bug 438397.
Attachment #393366 - Attachment is obsolete: true
Attachment #393437 - Flags: review?(kairo)
Attachment #393366 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
No longer depends on: 438397

Updated

8 years ago
Attachment #393437 - Flags: review?(kairo) → review+
(Assignee)

Comment 22

8 years ago
Comment on attachment 393437 [details] [diff] [review]
(Gv2) 2 random nits
[Checkin: Comment 22]


http://hg.mozilla.org/comm-central/rev/72019929e3f5
Attachment #393437 - Attachment description: (Gv2) 2 random nits → (Gv2) 2 random nits [Checkin: Comment 22]
(Assignee)

Comment 23

8 years ago
Created attachment 393493 [details] [diff] [review]
(Hv1) Bug 475064
[Checkin: Comment 25]
Attachment #393493 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Depends on: 475064

Comment 24

8 years ago
(In reply to comment #20)
> You're right about m-c/XulRunner.
> /js/src/ has it (unused) too, but maybe this is required to be in sync'!?

js/src/configure and ./configure don't need to be in sync.  js/src/configure just has a lot of crud that we haven't cleaned out yet.

The only parts of the main tree and js/src that are required to be in sync are those that are actually checked by the 'check' target in js/src/Makefile.in:

check-sync-dirs = $(PYTHON) $(srcdir)/config/check-sync-dirs.py
check::
	$(check-sync-dirs) $(srcdir)/config $(MOZ_SYNC_BUILD_FILES)/config
	$(check-sync-dirs) $(srcdir)/build $(MOZ_SYNC_BUILD_FILES)/build

Here, srcdir will be js/src, and $MOZ_SYNC_BUILD_FILES will be the top directory of the tree.  So ./js/src/config must be the same as ./config, and ./js/src/build must be the same as ./build.

Updated

8 years ago
Attachment #393493 - Flags: review?(kairo) → review+
(Assignee)

Comment 25

8 years ago
Comment on attachment 393493 [details] [diff] [review]
(Hv1) Bug 475064
[Checkin: Comment 25]


http://hg.mozilla.org/comm-central/rev/2754edb2bbd3
Attachment #393493 - Attachment description: (Hv1) Bug 475064 → (Hv1) Bug 475064 [Checkin: Comment 25]
(Assignee)

Comment 26

8 years ago
Created attachment 394273 [details] [diff] [review]
(Iv1) Bug 411726
[Checkin: Comment 27]
Attachment #394273 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Depends on: 411726

Updated

8 years ago
Attachment #394273 - Flags: review?(kairo) → review+
(Assignee)

Comment 27

8 years ago
Comment on attachment 394273 [details] [diff] [review]
(Iv1) Bug 411726
[Checkin: Comment 27]


http://hg.mozilla.org/comm-central/rev/32d7e909c526
Attachment #394273 - Attachment description: (Iv1) Bug 411726 → (Iv1) Bug 411726 [Checkin: Comment 27]
(Assignee)

Comment 28

8 years ago
Created attachment 394722 [details] [diff] [review]
(Jv1) Changeset 18865:97d4e97bb54a
[Checkin: Comment 29]

From
{
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/97d4e97bb54a
Bug 97954: Compute fully-substituted values of NSPR_CFLAGS and NSPR_LIBS. r=bsmedberg
}
Bug that bug doesn't quote it.

Per irc discussion with KaiRo, c-c doesn't have/want the "Recreating autoconf.mk with updated nspr-config output" block.
Attachment #394722 - Flags: review?(kairo)

Updated

8 years ago
Attachment #394722 - Flags: review?(kairo) → review+
(Assignee)

Comment 29

8 years ago
Comment on attachment 394722 [details] [diff] [review]
(Jv1) Changeset 18865:97d4e97bb54a
[Checkin: Comment 29]


http://hg.mozilla.org/comm-central/rev/e1f9e6f48cc0
Attachment #394722 - Attachment description: (Jv1) Changeset 18865:97d4e97bb54a → (Jv1) Changeset 18865:97d4e97bb54a [Checkin: Comment 29]
(Assignee)

Updated

8 years ago
Depends on: 463075
(Assignee)

Comment 30

8 years ago
Created attachment 395453 [details] [diff] [review]
(Kv1) Bug 463075
[Checkin: Comment 31]
Attachment #395453 - Flags: review?(kairo)

Updated

8 years ago
Attachment #395453 - Flags: review?(kairo) → review+
(Assignee)

Comment 31

8 years ago
Comment on attachment 395453 [details] [diff] [review]
(Kv1) Bug 463075
[Checkin: Comment 31]


http://hg.mozilla.org/comm-central/rev/3dde9c8d8ab2
Attachment #395453 - Attachment description: (Kv1) Bug 463075 → (Kv1) Bug 463075 [Checkin: Comment 31]
(Assignee)

Updated

8 years ago
Depends on: 422221
(Assignee)

Comment 32

8 years ago
Created attachment 395582 [details] [diff] [review]
(Lv1) (Bug 422221) Changeset 16526:a7b2f76a6ab8, also remove unused MOZ_PANGO_CFLAGS

From
{
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a7b2f76a6ab8
[dfb,422221] Gtk/DirectFB port core; patches from dfb tree, r=vlad
}
Though that bug doesn't explicitly quote it.


Things to double check:

MOZ_PANGO_CFLAGS:
c-c doesn't seem to care about it per
http://mxr.mozilla.org/comm-central/search?string=MOZ_PANGO_CFLAGS&case=on&find=%2Fconfig
so I remove it.

MOZ_PANGO_LIBS:
Only used in STATIC_EXTRA_LIBS per
http://mxr.mozilla.org/comm-central/search?string=MOZ_PANGO_LIBS&case=on&find=%2Fconfig
I'm not sure whether it can be removed too or not, so I left it...

MOZ_FIX_LINK_PATHS line:
You added |:/usr/lib:/usr/local/lib:/lib| in bug 450957.
I don't know whether we should keep or remove it now...
Attachment #395582 - Flags: review?(kairo)

Updated

8 years ago
Attachment #395582 - Flags: review?(kairo) → review-

Comment 33

8 years ago
Comment on attachment 395582 [details] [diff] [review]
(Lv1) (Bug 422221) Changeset 16526:a7b2f76a6ab8, also remove unused MOZ_PANGO_CFLAGS

>+AC_SUBST(MOZ_DFB)

>+    AC_SUBST(MOZ_PANGO)

Those two only make sense if they're available in autoconf.mk, so you need to port the changes to that file done in http://hg.mozilla.org/mozilla-central/rev/c07f8b188874 as well. Everything else looks fine, but r- for substituting those vars that don't exist there yet.
(Assignee)

Comment 34

8 years ago
Created attachment 395752 [details] [diff] [review]
(Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also remove unused MOZ_PANGO_CFLAGS
[Checkin: Comment 36]

Lv1, with comment 33 suggestion(s).

Damn these bugs with no patches nor checkin comments :-(
Note that other checkin happened after the one I ported :-|
Thanks to you new tracking tool ;-)
Attachment #395582 - Attachment is obsolete: true
Attachment #395752 - Flags: review?(kairo)

Comment 35

8 years ago
Comment on attachment 395752 [details] [diff] [review]
(Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also remove unused MOZ_PANGO_CFLAGS
[Checkin: Comment 36]

Yes, I also probably would have missed that one without my tool ;-)
Attachment #395752 - Flags: review?(kairo) → review+
(Assignee)

Updated

8 years ago
Attachment #395752 - Attachment description: (Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also remove unused MOZ_PANGO_CFLAGS → (Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also remove unused MOZ_PANGO_CFLAGS [Checkin: Comment 36]
(Assignee)

Comment 36

8 years ago
Comment on attachment 395752 [details] [diff] [review]
(Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also remove unused MOZ_PANGO_CFLAGS
[Checkin: Comment 36]


http://hg.mozilla.org/comm-central/rev/d349353f35aa
(Assignee)

Updated

8 years ago
Depends on: 485984, 445391
(Assignee)

Updated

8 years ago
Blocks: 511884
(Assignee)

Comment 37

8 years ago
Created attachment 395851 [details] [diff] [review]
(Mv1) Bug 445391 + (m-c) Bug 485984
[Checkin: See comment 38]

While there, also convert 3 "dnl" to "if+else+fi".

I filed bug 511884 to fix the UI, as a separate issue.
Attachment #395851 - Flags: review?(kairo)

Updated

8 years ago
Attachment #395851 - Flags: review?(kairo) → review+
(Assignee)

Comment 38

8 years ago
Comment on attachment 395851 [details] [diff] [review]
(Mv1) Bug 445391 + (m-c) Bug 485984
[Checkin: See comment 38]


http://hg.mozilla.org/comm-central/rev/c11e48d0bef1
+
http://hg.mozilla.org/comm-central/rev/32083046dee3
Bustage fix
Attachment #395851 - Attachment description: (Mv1) Bug 445391 + (m-c) Bug 485984 → (Mv1) Bug 445391 + (m-c) Bug 485984 [Checkin: See comment 38]
(Assignee)

Updated

8 years ago
Depends on: 499144
(Assignee)

Comment 39

8 years ago
Created attachment 396719 [details] [diff] [review]
(Nv1) Bug 499144
[Checkin: Comment 40]
Attachment #396719 - Flags: review?(kairo)

Updated

8 years ago
Attachment #396719 - Flags: review?(kairo) → review+
(Assignee)

Comment 40

8 years ago
Comment on attachment 396719 [details] [diff] [review]
(Nv1) Bug 499144
[Checkin: Comment 40]


http://hg.mozilla.org/comm-central/rev/8d39750eb0eb
Attachment #396719 - Attachment description: (Nv1) Bug 499144 → (Nv1) Bug 499144 [Checkin: Comment 40]
(Assignee)

Comment 41

8 years ago
Created attachment 396939 [details] [diff] [review]
(Ov1) Bug 447930
Attachment #396939 - Flags: review?(kairo)
(Assignee)

Updated

8 years ago
Depends on: 447930
(Assignee)

Updated

8 years ago
Blocks: 512490

Comment 42

8 years ago
Comment on attachment 396939 [details] [diff] [review]
(Ov1) Bug 447930

Please watch carefully if that actually might have the opposite effect of what it's supposed to do. The big question is if actually anything in _our_ configure script makes cairo-features.h even be rebuilt. We probably shouldn't, but not sure if we do. And if we don't we should not include this patch at all.
Yes, I know it's messy. And I only left this patch in my tool because I can't answer it right away.
r=me _only_ if i you can show that it doesn't have the opposite effect of what it should do.
Attachment #396939 - Flags: review?(kairo) → review+
(Assignee)

Comment 43

8 years ago
Created attachment 397455 [details] [diff] [review]
(Ov2) Remove useless CAIRO, MOZ_TREE_FREETYPE and PANGO configurations and a few more

Mark, (instead of a review) could you (just) submit this patch to Thunderbird Try Server?
(I'll ask KaiRo for review.)
Attachment #397455 - Flags: review?(bugzilla)
(Assignee)

Comment 44

8 years ago
(In reply to comment #42)
> (From update of attachment 396939 [details] [diff] [review])
> Yes, I know it's messy. And I only left this patch in my tool because I can't
> answer it right away.

Ah, that's why I ported that bug...
I checked that running |make configure| twice from c-c did not change the file: so this patch seems unneeded indeed.

(In reply to comment #43)
> Created an attachment (id=397455) [details]
> (Ov2) Remove useless CAIRO, MOZ_TREE_FREETYPE and PANGO configurations and a
> few more

This "opposite" (cleanup) patch reverts:
*all of '(Dv1) Bug 462908' :->
*most of '(Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also remove unused MOZ_PANGO_CFLAGS'.
Comment on attachment 397455 [details] [diff] [review]
(Ov2) Remove useless CAIRO, MOZ_TREE_FREETYPE and PANGO configurations and a few more

Done. It may take a few mins to show up. Serge, If you want access then follow the instructions here: https://wiki.mozilla.org/Thunderbird/Infrastructure/TryServer
Attachment #397455 - Flags: review?(bugzilla)
(Assignee)

Comment 46

8 years ago
(In reply to comment #45)
> (From update of attachment 397455 [details] [diff] [review])
> Done. It may take a few mins to show up. Serge, If you want access then follow

Thanks.
It failed to compile. I'm filling separate bugs to do one removal at a time ... then will see where that leads me to ;->

> the instructions here:
> https://wiki.mozilla.org/Thunderbird/Infrastructure/TryServer

I filed bug 513512.

Comment 47

8 years ago
Looks like quite a lot of work to port patches one by one, if those files are meant to be the same for 99% or so.
hg log --no-merges configure.in outputs a log file with 2480 lines, which means 496 changesets since March 2007.

Have you considered adding the Seamonkey/Thunderbird/Calendar bits back into mozilla-central's configure files? How much code would that be?

Or add more hooks to use app-specific files, which can live outside moz-central, but use moz-central's configure files?

Comment 48

8 years ago
(In reply to comment #47)
> Looks like quite a lot of work to port patches one by one, if those files are
> meant to be the same for 99% or so.

Actually less, but we haven't sorted out completely yet which parts we just can remove.

> hg log --no-merges configure.in outputs a log file with 2480 lines, which means
> 496 changesets since March 2007.

Not all need porting though, http://dev.seamonkey.at/?d=x&i=mozilla&m=c has a list of what needs to be ported or at least investigated to port.

> Have you considered adding the Seamonkey/Thunderbird/Calendar bits back into
> mozilla-central's configure files? How much code would that be?

A separate build system needs a separate configure - we have discussed this in length before creating the whole comm-central repository, and together with the platform build system owners we came to the conclusion that this is the way to go, even if it's not really pretty.

> Or add more hooks to use app-specific files, which can live outside
> moz-central, but use moz-central's configure files?

We're already doing this to a part, but I don't think it's the solution for the configure script itself. In any case, this bug is about getting thing in sync again between comm-central and 1.9.1, if you want to discuss the general approach, please do that in the newgroups / mailing lists and don't intermingle it with the actual work being done here.

This bug is large enough as it is (and I dislike that in any case), even without adding general discussion here when it belongs elsewhere.
(In reply to comment #48)
> This bug is large enough as it is (and I dislike that in any case), even
> without adding general discussion here when it belongs elsewhere.

The checkin comments also make it really difficult to find out at a glance what has actually been ported.
Depends on: 514381
(Assignee)

Updated

8 years ago
Blocks: 521624
(Assignee)

Updated

8 years ago
Depends on: 538747
(Assignee)

Updated

8 years ago
No longer blocks: 512490
(Assignee)

Updated

8 years ago
Depends on: 541966
(Assignee)

Updated

8 years ago
No longer depends on: 481780
Serge, since c-c has branched and both c-1.9.1 and m-1.9.1 are stable and not taking unneeded patches can we close this bug and just concentrate on c-c vs m-c[1.9.2] for now?
(Assignee)

Comment 51

8 years ago
(In reply to comment #50)

I am mostly working on c-c (bug 506493) now.
But there are probably still a few fixes to port from the list at
http://dev.seamonkey.at/?d=x&i=mozilla&m=c
for example,
so I prefer to leave this bug open ftb.
(Assignee)

Updated

8 years ago
Alias: C191ConfSync
Ok; lets file followups at least for anything more that needs 191 attention [so its easier to wontfix or approve- etc.]

For what its worth though, if there is _any_ risk in a conf change for 191 I'd vote against approval, unless it is a stability or security fix.
Summary: Resync' configure.in between c-c and m-1.9.1 → [Tracking] Resync' configure.in between c-1.9.1 and m-1.9.1
(Assignee)

Comment 53

8 years ago
(In reply to comment #34)
> Created an attachment (id=395752) [details]
> (Lv2) (Bug 422221) Changesets 16526:a7b2f76a6ab8 + 16527:c07f8b188874, also
> remove unused MOZ_PANGO_CFLAGS

Ftr, this implicitly ported bug 463887.
Depends on: 463887
(Assignee)

Updated

8 years ago
Depends on: 545055
(Assignee)

Updated

8 years ago
Depends on: 543294
(Assignee)

Updated

8 years ago
Depends on: 545133
(Assignee)

Updated

8 years ago
Depends on: 543296
(Assignee)

Updated

8 years ago
No longer depends on: 545133
(Assignee)

Updated

8 years ago
Depends on: 545594
(Assignee)

Updated

8 years ago
Depends on: 481584
(Assignee)

Comment 54

8 years ago
Comment on attachment 396939 [details] [diff] [review]
(Ov1) Bug 447930


(In reply to comment #44)
> I checked that running |make configure| twice from c-c did not change the file:
> so this patch seems unneeded indeed.

Confirming:
Now, I have a little more understanding of configure" and of what was written in bug 447930:
http://mxr.mozilla.org/comm-central/search?string=cairo-features.h&case=on
*cairo-features.h.in in listed in toolkit-makefiles.sh,
*toolkit-makefiles.sh is listed in allmakefiles.sh,
*allmakefiles.sh is called by '. ${srcdir}/allmakefiles.sh' in configure,
*and all this happens in mozilla-central (only).
*Hence, porting this to c-c is just useless.
Attachment #396939 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
No longer depends on: 447930
(Assignee)

Updated

8 years ago
No longer blocks: 521624
(Assignee)

Updated

8 years ago
Blocks: 521624
(Assignee)

Updated

8 years ago
Depends on: 547518
(Assignee)

Updated

8 years ago
Depends on: 548210
(Assignee)

Comment 55

8 years ago
Created attachment 429707 [details] [diff] [review]
(Pv1-191) Copy bug 481584

Moved from bug 481584.
Attachment #429707 - Flags: approval-thunderbird3.0.4?
(Assignee)

Updated

8 years ago
Whiteboard: [ToDo: Ov2...]
(Assignee)

Updated

8 years ago
Depends on: 481780
(Assignee)

Updated

8 years ago
Depends on: 478221
(Assignee)

Comment 56

8 years ago
Created attachment 430064 [details] [diff] [review]
(Qv1) Port bug 478221
[Checkin: Comment 57 & 65]

Needed by bug 474610 port.
Attachment #430064 - Flags: review?(bugspam.Callek)
(Assignee)

Updated

8 years ago
Blocks: 474610
Attachment #430064 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 57

8 years ago
Comment on attachment 430064 [details] [diff] [review]
(Qv1) Port bug 478221
[Checkin: Comment 57 & 65]


http://hg.mozilla.org/comm-central/rev/d1cc1d78e6dd


"approval-thunderbird3.0.4=?":
Zero risk, build config only.
Attachment #430064 - Attachment description: (Qv1) Port bug 478221 → (Qv1) Port bug 478221 [Checkin: Comment 57]
Attachment #430064 - Flags: approval-thunderbird3.0.4?
Attachment #429707 - Flags: review+
(Assignee)

Updated

8 years ago
Depends on: 521750
Attachment #429707 - Flags: approval-thunderbird3.0.4? → approval-thunderbird3.0.4-
Comment on attachment 429707 [details] [diff] [review]
(Pv1-191) Copy bug 481584

So now I look at it, I see no reason to take this on 1.9.1 as I doubt anyone's going to be doing much with Windows CE builds there.
Comment on attachment 430064 [details] [diff] [review]
(Qv1) Port bug 478221
[Checkin: Comment 57 & 65]

>+
>+include $(topsrcdir)/mozilla/toolkit/mozapps/installer/package-name.mk
>+

What does this change actually fix?

If the only reason for including it is for bug 474610, then please include it in the patch for that port.

Also, if you want approval on this bug, I'm warning you in advance that my requirement will be that 3.0.4 will be the last patches for this bug to go in on branch, otherwise it will be a nightmare to verify.
(Assignee)

Comment 60

8 years ago
Created attachment 432026 [details] [diff] [review]
(Rv1) Copy bug 521750
[Checkin: Comment 63 & 66]
Attachment #432026 - Flags: review?(bugspam.Callek)
(Assignee)

Comment 61

8 years ago
(In reply to comment #59)
> (From update of attachment 430064 [details] [diff] [review])
> >+
> >+include $(topsrcdir)/mozilla/toolkit/mozapps/installer/package-name.mk
> >+
> 
> What does this change actually fix?

At the very least, to avoid c-c to be bitten again by lacking this include, as it was with m-1.9.2.

> If the only reason for including it is for bug 474610, then please include it
> in the patch for that port.

Yes, that bug is what I have more specifically in mind.
"But" there is nothing "more" to port for it ;-)

> Also, if you want approval on this bug, I'm warning you in advance that my
> requirement will be that 3.0.4 will be the last patches for this bug to go in
> on branch, otherwise it will be a nightmare to verify.

Agreed.
(Assignee)

Updated

8 years ago
Depends on: 545755
(Assignee)

Comment 62

8 years ago
Created attachment 432137 [details] [diff] [review]
(Sv1-191) Copy bug 545755
[Checkin: Comment 67]
Attachment #432137 - Flags: review?(bugzilla)
Attachment #432137 - Flags: approval-thunderbird3.0.4?
Attachment #432026 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Updated

8 years ago
Attachment #429707 - Attachment is obsolete: true
(Assignee)

Comment 63

8 years ago
Comment on attachment 432026 [details] [diff] [review]
(Rv1) Copy bug 521750
[Checkin: Comment 63 & 66]


http://hg.mozilla.org/comm-central/rev/5f560aa814de


"approval-thunderbird3.0.4=?":
Needed with m-1.9.1.
Attachment #432026 - Attachment description: (Rv1) Copy bug 521750 → (Rv1) Copy bug 521750 [Checkin: Comment 63]
Attachment #432026 - Flags: approval-thunderbird3.0.4?
Comment on attachment 430064 [details] [diff] [review]
(Qv1) Port bug 478221
[Checkin: Comment 57 & 65]

a=Standard8 with the previously agreed provision that this bug is closed with the landing of these bugs.

Note that tree closes at 23:59 Pacific time tonight.
Attachment #430064 - Flags: approval-thunderbird3.0.4? → approval-thunderbird3.0.4+
Attachment #432026 - Flags: approval-thunderbird3.0.4? → approval-thunderbird3.0.4+
Attachment #432137 - Flags: review?(bugzilla)
Attachment #432137 - Flags: review+
Attachment #432137 - Flags: approval-thunderbird3.0.4?
Attachment #432137 - Flags: approval-thunderbird3.0.4+
(Assignee)

Updated

8 years ago
Attachment #430064 - Attachment description: (Qv1) Port bug 478221 [Checkin: Comment 57] → (Qv1) Port bug 478221 [Checkin: Comment 57 & 65]
(Assignee)

Comment 65

8 years ago
Comment on attachment 430064 [details] [diff] [review]
(Qv1) Port bug 478221
[Checkin: Comment 57 & 65]


http://hg.mozilla.org/releases/comm-1.9.1/rev/ded59344d902
(Assignee)

Comment 66

8 years ago
Comment on attachment 432026 [details] [diff] [review]
(Rv1) Copy bug 521750
[Checkin: Comment 63 & 66]


http://hg.mozilla.org/releases/comm-1.9.1/rev/517fd95ce33c
Attachment #432026 - Attachment description: (Rv1) Copy bug 521750 [Checkin: Comment 63] → (Rv1) Copy bug 521750 [Checkin: Comment 63 & 66]
(Assignee)

Comment 67

8 years ago
Comment on attachment 432137 [details] [diff] [review]
(Sv1-191) Copy bug 545755
[Checkin: Comment 67]


http://hg.mozilla.org/releases/comm-1.9.1/rev/f949a9f723ce
Attachment #432137 - Attachment description: (Sv1-191) Copy bug 545755 → (Sv1-191) Copy bug 545755 [Checkin: Comment 67]
(Assignee)

Updated

8 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
status-thunderbird3.0: --- → .4-fixed
Resolution: --- → FIXED
Verified fixed 3.0 on the basis that what is landed hasn't broken the builds.
Keywords: verified-thunderbird3.0

Updated

8 years ago
Keywords: fixed-seamonkey2.0.4
(Assignee)

Comment 69

8 years ago
Ftr, http://dev.seamonkey.at/?d=x&i=mozilla&m=c doesn't list any more (old) m-1.9.1 changeset to port atm :-)
(Assignee)

Updated

8 years ago
Depends on: 556409
You need to log in before you can comment on or make changes to this bug.