[SeaMonkey, Debug] (Windows) Mailnews packaged tests are all T-FAIL due to missing dll(s)

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Build Config
--
blocker
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

({fixed-seamonkey2.0.4})

Trunk
seamonkey2.1a1
x86
Windows 2000
fixed-seamonkey2.0.4
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

8 years ago
I downloaded a build and tried to run the xpcshell tests:
the (first) tests pass if I manually dismiss the (repetitive) dialog complaining about missing msgbsutl.dll.

KaiRo, could you confirm this is what is happening on our tinderboxes too?

***

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1267645769.1267651527.18808.gz
WINNT 5.2 comm-central-trunk leak test build on 2010/03/03 11:49:29
{
make package-compare

+bin/msgbsutl.dll
}

It looks like we may be missing to package other dlls too, which might explain some unexpected warnings/errors I already noticed in the tests logs...

I assume we never hit this failure before because we never ran debug+packaged tests.
Flags: in-testsuite-

Comment 1

8 years ago
(In reply to comment #0)
> KaiRo, could you confirm this is what is happening on our tinderboxes too?

How could I?

> It looks like we may be missing to package other dlls too, which might explain
> some unexpected warnings/errors I already noticed in the tests logs...

The question is if those belong into release packages or if those might possibly belong into the test packages, and if so, how they get there.

Where does msgbsutl.dll come from? It's completely possible that we're still missing some DLLs in the shared packages, I'm happy if we can solves those!
(Assignee)

Updated

8 years ago
Depends on: 534408
(Assignee)

Comment 2

8 years ago
http://mxr.mozilla.org/comm-central/source/mailnews/base/util/Makefile.in?mark=47-53#44
{
47 ifndef MOZ_STATIC_MAIL_BUILD
48 LIBRARY_NAME    = msgbaseutil
50 SHORT_LIBNAME   = msgbsutl
51 else
52 LIBRARY_NAME    = msgbsutl_s
53 endif
}

All platforms misses this file on Debug.
(Only Windows is hung by its dialog, it seems.)
(Assignee)

Comment 3

8 years ago
Created attachment 430224 [details] [diff] [review]
(Av1) Package "msgbaseutil" library
[Checkin: Comment 6]
Attachment #430224 - Flags: review?(kairo)
(Assignee)

Comment 4

8 years ago
I tried to briefly run the application itself:
*Does not complain about msgbsutl.dll at all :-|
*Lists some other mailnews dlls it fails to load: probably matches the non-packaged ones ;->

One step at a time...

Comment 5

8 years ago
Comment on attachment 430224 [details] [diff] [review]
(Av1) Package "msgbaseutil" library
[Checkin: Comment 6]

>From: Serge Gautherie <sgautherie.bz@free.fr>
>
>Bug 550018 - [SeaMonkey 2.1, Windows] Packaged tests are all T-FAIL due to missing dll(s);
>(Av1) Package "msgbaseutil" library.
>
>diff --git a/suite/installer/package-manifest.in b/suite/installer/package-manifest.in
>--- a/suite/installer/package-manifest.in
>+++ b/suite/installer/package-manifest.in
>@@ -747,25 +747,33 @@ bin/libfreebl_32int64_3.so
> #endif
> 
> ; [OS/2]
> #ifdef XP_OS2
> @BINPATH@/MozSounds.cmd
> #endif
> 
> [mail]
>+; MailNews component binaries
> #ifndef MOZ_STATIC_BUILD
>-; MailNews component binaries (shared build only)
> @BINPATH@/components/@DLL_PREFIX@import@DLL_SUFFIX@
> @BINPATH@/components/@DLL_PREFIX@mail@DLL_SUFFIX@
> #ifdef XP_WIN32
> @BINPATH@/components/msgMapi.dll
> #endif
> @BINPATH@/components/@DLL_PREFIX@msgsmime@DLL_SUFFIX@
> #endif
>+#ifndef MOZ_STATIC_MAIL_BUILD
>+#ifdef WIN_OR_OS2
>+; To become |#ifdef XP_OS2| in bug 534408.
>+@BINPATH@/msgbsutl.dll
>+#else
>+@BINPATH@/@DLL_PREFIX@msgbaseutil@DLL_SUFFIX@
>+#endif
>+#endif
> 
> ; MailNews component xpts
> @BINPATH@/components/addrbook.xpt
> @BINPATH@/components/fts3tok.xpt
> @BINPATH@/components/import.xpt
> @BINPATH@/components/impComm4xMail.xpt
> @BINPATH@/components/mailview.xpt
> @BINPATH@/components/mime.xpt
>diff --git a/suite/installer/removed-files.in b/suite/installer/removed-files.in
>--- a/suite/installer/removed-files.in
>+++ b/suite/installer/removed-files.in
>@@ -249,16 +249,25 @@ components/tkautoc.dll
> components/tkitcmps.dll
> components/windowsproxy.dll
> components/xpautoc.dll
> components/xpc3250.dll
> components/xppref32.dll
> #endif
> ; MOZ_STATIC_BUILD
> #endif
>+#ifdef MOZ_STATIC_MAIL_BUILD
>+#ifdef WIN_OR_OS2
>+; To become |#ifdef XP_OS2| in bug 534408.
>+; At that time, copy these entries into the following |#ifdef XP_WIN| block. (Or just don't.!.)
>+msgbsutl.dll
>+#else
>+@DLL_PREFIX@msgbaseutil@DLL_SUFFIX@
>+#endif
>+#endif
> #ifdef XP_WIN
> components/appshell.dll
> components/cmdlines.dll
> components/gkparser.dll
> components/perms.dll
> components/spellchk.dll
> components/strgcmps.dll
> components/tkautoc.dll
Attachment #430224 - Flags: review?(kairo) → review+
(Assignee)

Updated

8 years ago
Blocks: 534408
No longer depends on: 534408
(Assignee)

Comment 6

8 years ago
Comment on attachment 430224 [details] [diff] [review]
(Av1) Package "msgbaseutil" library
[Checkin: Comment 6]


http://hg.mozilla.org/comm-central/rev/81d4998ae662
Attachment #430224 - Attachment description: (Av1) Package "msgbaseutil" library → (Av1) Package "msgbaseutil" library [Checkin: Comment 6]
(Assignee)

Comment 7

8 years ago
I downloaded the first build:
on my local Windows 2000, xpcshell tests now start fine :-)

Let's see the tinderbox results before proceeding.
Summary: [SeaMonkey 2.1, Windows] Packaged tests are all T-FAIL due to missing dll(s) → [SeaMonkey, Debug] Packaged tests are all T-FAIL due to missing dll(s)
(Assignee)

Comment 8

8 years ago
Created attachment 430375 [details] [diff] [review]
(Bv1-191) Package "msgbaseutil" library
[Checkin: Comment 9+10]

Fix failure, Allow downgrading.
Attachment #430375 - Flags: review?(kairo)
Attachment #430375 - Flags: approval-seamonkey2.0.4?

Updated

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

Comment 9

8 years ago
Comment on attachment 430375 [details] [diff] [review]
(Bv1-191) Package "msgbaseutil" library
[Checkin: Comment 9+10]


http://hg.mozilla.org/releases/comm-1.9.1/rev/d1222b6a7831
Attachment #430375 - Attachment description: (Bv1-191) Package "msgbaseutil" library → (Bv1-191) Package "msgbaseutil" library [Checkin: Comment 9]
(Assignee)

Updated

8 years ago
Keywords: fixed-seamonkey2.0.4
Summary: [SeaMonkey, Debug] Packaged tests are all T-FAIL due to missing dll(s) → [SeaMonkey, Debug] (Windows) Packaged tests are all T-FAIL due to missing dll(s)
(Assignee)

Comment 10

8 years ago
Comment on attachment 430375 [details] [diff] [review]
(Bv1-191) Package "msgbaseutil" library
[Checkin: Comment 9+10]


http://hg.mozilla.org/releases/comm-1.9.1/rev/ba8f1ab41f81
(Cv1-191) Fix unedited pastes
Attachment #430375 - Attachment description: (Bv1-191) Package "msgbaseutil" library [Checkin: Comment 9] → (Bv1-191) Package "msgbaseutil" library [Checkin: Comment 9+10]
(Assignee)

Updated

8 years ago
Depends on: 550474
(Assignee)

Comment 11

8 years ago
(In reply to comment #7)

Patch A fixed Windows tinderboxes too :-)
Failures are +/- like on the other platforms.

I prepared a patch to package the other missing mailnews dlls, but it "depends" on bug 534408 reviews and bug 550474 answer...
Whiteboard: [blocked by comment 11 dependencies]
(Assignee)

Comment 12

8 years ago
Created attachment 430950 [details] [diff] [review]
(Dv1) Package the non-static mail librairies, Document obsolete short names block
[Checkin: Comment 14+16+17]

This takes account of bug 550474 decision.

I'll land it as is if you agree to later forget about the renamed dlls (which releases don't care for) [and give me a "rs=you" for that],
otherwise I'll wait for the renames in bug 534408.
Attachment #430950 - Flags: review?(kairo)

Comment 13

8 years ago
Comment on attachment 430950 [details] [diff] [review]
(Dv1) Package the non-static mail librairies, Document obsolete short names block
[Checkin: Comment 14+16+17]

I start to think it doesn't make sense to support non-static builds in removed-files, as we don't do releases or nightlies as shared builds any more, i.e. we never need to update from a static to a shared build, we only need to do it the other way round, i.e. from some older shared alphas or nightlies to the static builds we ship now.
r+ for now, though, but we might be able to clean up more in removed-files in the future.
Attachment #430950 - Flags: review?(kairo) → review+
(Assignee)

Comment 14

8 years ago
Comment on attachment 430950 [details] [diff] [review]
(Dv1) Package the non-static mail librairies, Document obsolete short names block
[Checkin: Comment 14+16+17]


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


(In reply to comment #13)
> we might be able to clean up more in removed-files in the future.

Indeed, that future is planned to be when SeaMonkey becomes libxul only (as Firefox is now).
Ftb, I assume the current situation is not very valuable, but I maintain it by "just in case" default :-|

Making the decision before is a possibility:
it might be as simple as adding an "non-static means no installer" error check in configure.in.!?.
Attachment #430950 - Attachment description: (Dv1) Package the non-static mail librairies, Document obsolete short names block → (Dv1) Package the non-static mail librairies, Document obsolete short names block [Checkin: Comment 14]
(Assignee)

Updated

8 years ago
Whiteboard: [blocked by comment 11 dependencies] → [ToDo: bug 534408 (expected) renames, then c-1.9.1 backport]

Comment 15

8 years ago
(In reply to comment #14)
> Making the decision before is a possibility:
> it might be as simple as adding an "non-static means no installer" error check
> in configure.in.!?.

Well, it's not that easy, as we need to allow creating packages from shared builds so we can run packaged tests on them (static doesn't build with --enable-tests, as you know).
We haven't *shipped* shared build for quite some time though (in nightlies or releases), and removed-files is only needed for updating existing installations (via installer or updater), which test boxes don't do, so we probably only need to maintain removed-files for static builds (right now, libxul will come in the future) and for legacy reasons have the files in that need to be removed from the times when we did ship shared builds.
Even that would already clean that file significantly, I think.
(Assignee)

Comment 16

8 years ago
Comment on attachment 430950 [details] [diff] [review]
(Dv1) Package the non-static mail librairies, Document obsolete short names block
[Checkin: Comment 14+16+17]


http://hg.mozilla.org/comm-central/rev/5c3256103b4a
(Ev1) Add missing 'components/' in patch Dv1.

(Damn... :-/)
Attachment #430950 - Attachment description: (Dv1) Package the non-static mail librairies, Document obsolete short names block [Checkin: Comment 14] → (Dv1) Package the non-static mail librairies, Document obsolete short names block [Checkin: Comment 14+16]
(Assignee)

Comment 17

8 years ago
Comment on attachment 430950 [details] [diff] [review]
(Dv1) Package the non-static mail librairies, Document obsolete short names block
[Checkin: Comment 14+16+17]


http://hg.mozilla.org/comm-central/rev/a8a7ef44ba1d
(Fv1) Add missing '#' in patch Dv1

(Damn me/2... :-/)
Attachment #430950 - Attachment description: (Dv1) Package the non-static mail librairies, Document obsolete short names block [Checkin: Comment 14+16] → (Dv1) Package the non-static mail librairies, Document obsolete short names block [Checkin: Comment 14+16+17]
(Assignee)

Updated

8 years ago
Blocks: 542944
(Assignee)

Comment 18

8 years ago
http://hg.mozilla.org/comm-central/rev/a566062b1c0b
(Gv1) Bug 534408 renamings.
(Assignee)

Updated

8 years ago
No longer blocks: 534408
Depends on: 534408
Summary: [SeaMonkey, Debug] (Windows) Packaged tests are all T-FAIL due to missing dll(s) → [SeaMonkey, Debug] (Windows) Mailnews packaged tests are all T-FAIL due to missing dll(s)
Whiteboard: [ToDo: bug 534408 (expected) renames, then c-1.9.1 backport] → [ToDo: very bug 534408 renamings, then c-1.9.1 backport]

Comment 19

7 years ago
We don't have all tests failing any more, so this is fixed, right?

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Whiteboard: [ToDo: very bug 534408 renamings, then c-1.9.1 backport]
You need to log in before you can comment on or make changes to this bug.