Port |Bug 508861 - [electrolysis] Build/ship the C++ runtime with MSVC/jemalloc| to comm-central. (SeaMonkey packaging fix only)

RESOLVED FIXED in Thunderbird 3.1b2

Status

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

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

({fixed-seamonkey2.0.4})

Trunk
Thunderbird 3.1b2
x86
Windows 2000
fixed-seamonkey2.0.4
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mostly fixed by blocking bugs in TB 3.1a1])

Attachments

(2 attachments, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

8 years ago
Oh, got real confused trying to sort this one out :-<
Assignee: sgautherie.bz → bugzilla
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Depends on: 512763, 531283
Resolution: --- → FIXED
Whiteboard: [Fixed by blocking bugs]
(Assignee)

Comment 2

7 years ago
Created attachment 430849 [details] [diff] [review]
(Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+)
[Backout: Bug 552943]
Attachment #430849 - Flags: review?(gozer)
(Assignee)

Updated

7 years ago
Blocks: 506493
(Assignee)

Comment 3

7 years ago
Created attachment 431354 [details] [diff] [review]
(Bv1-SM) Remove mozcpp19.dll (on trunk too)
[Checkin: Comment 9]

Firefox doesn't seem to care but SeaMonkey does.
Attachment #431354 - Flags: review?(bugspam.Callek)
(Assignee)

Updated

7 years ago
Depends on: 546484
Comment on attachment 430849 [details] [diff] [review]
(Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+)
[Backout: Bug 552943]

Just cleaning up my unnecessary review request
Attachment #430849 - Flags: review?(gozer) → review-
Comment on attachment 431354 [details] [diff] [review]
(Bv1-SM) Remove mozcpp19.dll (on trunk too)
[Checkin: Comment 9]

Can you run me through this, I'm not exactly sure what this bug; or this patch is trying to solve. I don't see any indication this should be in removed-files on trunk at all.
(Assignee)

Comment 6

7 years ago
(In reply to comment #5)

> Can you run me through this, I'm not exactly sure what this bug; or this patch

The main purpose of this bug was fixed by blocking bugs.
The current patches are additional/missed points.

> is trying to solve. I don't see any indication this should be in removed-files
> on trunk at all.

FF doesn't care at all; TB cares on m-1.9.2- only; SM already has it on c-1.9.1.
Actually, I don't see why this file shouldn't be removed when it's not needed on trunk.
(Assignee)

Comment 7

7 years ago
Comment on attachment 430849 [details] [diff] [review]
(Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+)
[Backout: Bug 552943]

gozer,
why r- patch you're not looking at?
why refuse to look at a followup patch for a bug you r+?
Attachment #430849 - Flags: review- → review?(bugspam.Callek)

Updated

7 years ago
Attachment #431354 - Flags: review?(bugspam.Callek) → review+
Comment on attachment 431354 [details] [diff] [review]
(Bv1-SM) Remove mozcpp19.dll (on trunk too)
[Checkin: Comment 9]

To me, this actually can land; I don't see a huge deal with us needing this in removed files (one could expect that if we enable jemalloc/"MOZ_MEMORY" that we'd keep it. But theoretically this is fine to to fix for mail/* too (to remove on trunk as well), but that is not up to me.
(Assignee)

Comment 9

7 years ago
Comment on attachment 431354 [details] [diff] [review]
(Bv1-SM) Remove mozcpp19.dll (on trunk too)
[Checkin: Comment 9]


http://hg.mozilla.org/comm-central/rev/0f98de542f0b
Attachment #431354 - Attachment description: (Bv1-SM) Remove mozcpp19.dll (on trunk too) → (Bv1-SM) Remove mozcpp19.dll (on trunk too) [Checkin: Comment 9]
(Assignee)

Comment 10

7 years ago
Created attachment 432420 [details] [diff] [review]
(Cv1-SM-191) Always remove mozcpp19.dll, wrongly fixed in bug 512763
[Checkin: Comment 11]
Attachment #432420 - Flags: review?(kairo)
Attachment #432420 - Flags: approval-seamonkey2.0.4?
(Assignee)

Updated

7 years ago
Attachment #430849 - Attachment description: (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 521763 → (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763

Updated

7 years ago
Attachment #432420 - Flags: review?(kairo)
Attachment #432420 - Flags: review+
Attachment #432420 - Flags: approval-seamonkey2.0.4?
Attachment #432420 - Flags: approval-seamonkey2.0.4+
(Assignee)

Comment 11

7 years ago
Comment on attachment 432420 [details] [diff] [review]
(Cv1-SM-191) Always remove mozcpp19.dll, wrongly fixed in bug 512763
[Checkin: Comment 11]


http://hg.mozilla.org/releases/comm-1.9.1/rev/ee3cd6696010
Attachment #432420 - Attachment description: (Cv1-SM-191) Always remove mozcpp19.dll, wrongly fixed in bug 512763 → (Cv1-SM-191) Always remove mozcpp19.dll, wrongly fixed in bug 512763 [Checkin: Comment 11]
(Assignee)

Updated

7 years ago
Keywords: fixed-seamonkey2.0.4
(In reply to comment #6)
> Actually, I don't see why this file shouldn't be removed when it's not needed
> on trunk.

If its not needed, then a) why does SM stil try and package it, b) the configs explicitly define it as a default library?

Yes, I'm not sure why the tinderboxes aren't broken, but maybe that's because it you're packaging and then removing it.

Either way, what's happening here just looks wrong to me IMO (see also:

http://mxr.mozilla.org/comm-central/search?string=mozcpp19

)

Anyway, I didn't fix/implement this bug, so don't assign it to me.
Assignee: bugzilla → sgautherie.bz
(Assignee)

Comment 13

7 years ago
(In reply to comment #12)

> If its not needed, then a) why does SM stil try and package it, b) the configs
> explicitly define it as a default library?

mozcpp19.dll is needed |#ifdef MOZ_MEMORY| only and on trunk only.
Hence, it's not needed |#ifndef MOZ_MEMORY| on trunk and never needed on m-1.9.1.
NB: And SeaMonkey doesn't care about m-1.9.2, though it looks like my trunk patch would be fine on it too.

> Yes, I'm not sure why the tinderboxes aren't broken, but maybe that's because

I'm not sure which tinderboxes you would expect to be broken and in which case.

> it you're packaging and then removing it.

I'm not, am I?

> Either way, what's happening here just looks wrong to me IMO (see also:

What, specifically?

> Anyway, I didn't fix/implement this bug, so don't assign it to me.

I had assigned it to you because you had fixed bug 512763.
I agree I'd rather be the assignee now that I'm fixing some nits here ;-)
Attachment #430849 - Attachment description: (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763 → (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in
Attachment #430849 - Flags: review+
(In reply to comment #7)
> (From update of attachment 430849 [details] [diff] [review])
> gozer,
> why r- patch you're not looking at?

Sorry, I was trying to remove a review request for a patch I thought I was not relevant reviewer for.

> why refuse to look at a followup patch for a bug you r+?

Because I mistakenly took this patch/bug for another one. Apologies.
(In reply to comment #12)

The points you bring up actually tripped me up at first too...

> (In reply to comment #6)
> > Actually, I don't see why this file shouldn't be removed when it's not needed
> > on trunk.
> 
> If its not needed, then a) why does SM stil try and package it, b) the configs
> explicitly define it as a default library?

It is only not needed when MOZ_MEMORY is not set. So the bits you cite are also correct.

> Either way, what's happening here just looks wrong to me IMO (see also:
> 
> http://mxr.mozilla.org/comm-central/search?string=mozcpp19

It did look wrong to me too; and I did like 3 checks on things just to be sure I understood what the case was. its packaged ifdef MOZ_MEMORY; and removed ifndef MOZ_MEMORY.

As I said earlier once we do set MOZ_MEMORY I don't forsee us dropping it (so long as that config is supported by m-c anyway). So I'm not certain this is even really needed in removed-files. But even so, it is "correct" so did not stop the patch landing.
(In reply to comment #13)
> > it you're packaging and then removing it.
> 
> I'm not, am I?

No you're not, I was getting confused between ifdefs and different patches.
(Assignee)

Comment 17

7 years ago
Comment on attachment 430849 [details] [diff] [review]
(Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+)
[Backout: Bug 552943]


http://hg.mozilla.org/comm-central/rev/e990fa115c50
Attachment #430849 - Attachment description: (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in → (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+) [Checkin: Comment 17]
Attachment #430849 - Flags: review?(bugspam.Callek)
(Assignee)

Updated

7 years ago
Whiteboard: [Fixed by blocking bugs] → [mostly fixed by blocking bugs in TB 3.1a1]
Target Milestone: Thunderbird 3.1a1 → Thunderbird 3.1b2
Depends on: 552943
Reopening, attachment 430849 [details] [diff] [review] caused bug 552943 as that define is still needed on 1.9.2 branch.

So in reality, I think this is wontfix and bug 512763 didn't miss anything. Although due to the additional patches for packaging changes, maybe just changing the title would do.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 19

7 years ago
Comment on attachment 430849 [details] [diff] [review]
(Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+)
[Backout: Bug 552943]


Wow, I really got confused on this case: sorry :-(
Attachment #430849 - Attachment description: (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+) [Checkin: Comment 17] → (Av1) Remove AC_DEFINE(_STATIC_CPPLIB), missed in bug 512763, (1.9.2+) [Backout: Bug 552943]
Attachment #430849 - Attachment is obsolete: true
(Assignee)

Updated

7 years ago
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago7 years ago
Resolution: --- → FIXED
Summary: Port |Bug 508861 - [electrolysis] Build/ship the C++ runtime with MSVC/jemalloc| to comm-central → Port |Bug 508861 - [electrolysis] Build/ship the C++ runtime with MSVC/jemalloc| to comm-central. (SeaMonkey packaging fix only)
You need to log in before you can comment on or make changes to this bug.