Closed Bug 550018 Opened 15 years ago Closed 14 years ago

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

Categories

(SeaMonkey :: Build Config, defect)

x86
Windows 2000
defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a1

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

(Keywords: fixed-seamonkey2.0.4)

Attachments

(3 files)

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-
(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!
Depends on: 534408
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.)
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 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+
Blocks: 534408
No longer depends on: 534408
Attachment #430224 - Attachment description: (Av1) Package "msgbaseutil" library → (Av1) Package "msgbaseutil" library [Checkin: Comment 6]
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)
Fix failure, Allow downgrading.
Attachment #430375 - Flags: review?(kairo)
Attachment #430375 - Flags: approval-seamonkey2.0.4?
Attachment #430375 - Flags: review?(kairo)
Attachment #430375 - Flags: review+
Attachment #430375 - Flags: approval-seamonkey2.0.4?
Attachment #430375 - Flags: approval-seamonkey2.0.4+
Attachment #430375 - Attachment description: (Bv1-191) Package "msgbaseutil" library → (Bv1-191) Package "msgbaseutil" library [Checkin: Comment 9]
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)
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]
Depends on: 550474
(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]
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 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+
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]
Whiteboard: [blocked by comment 11 dependencies] → [ToDo: bug 534408 (expected) renames, then c-1.9.1 backport]
(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.
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]
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]
Blocks: 542944
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]
We don't have all tests failing any more, so this is fixed, right?
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: