Closed Bug 547499 Opened 14 years ago Closed 14 years ago

Mac packaged builds are completely broken by not packaging libwidget_mac.dylib

Categories

(SeaMonkey :: Build Config, defect)

All
macOS
defect
Not set
major

Tracking

(Not tracked)

VERIFIED FIXED
seamonkey2.1a1

People

(Reporter: philor, Assigned: philor)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Fix (obsolete) — Splinter Review
Because widget: cocoa is filled with hate and nastiness, and it wants to make your life as miserable as it possibly can, it has LIBRARY_NAME = widget_mac and XPIDL_MODULE = widget_cocoa, so you need to package libwidget_mac.dylib and widget_cocoa.xpt, rather than the libwidget_cocoa.dylib and no extra xpt that you are currently packaging.
Flags: in-testsuite-
Attachment #428000 - Flags: review?(kairo)
Comment on attachment 428000 [details] [diff] [review]
Fix

>@@ -427,16 +427,19 @@ bin/components/@DLL_PREFIX@nkgnomevfs@DL
>+#ifdef XP_MAC_OSX

XP_MACOSX...
Ftr, Firefox:
bug 463605
http://hg.mozilla.org/mozilla-central/rev/0bd17bd1cbaf
{
+++ b/browser/installer/osx/packages-static
+@BINPATH@/components/widget.xpt
}

I wonder why c-c apps need to do more? (because not libxul (yet)?)
Attached patch No, really fixSplinter Review
Be less embarrassing if I hadn't still had the package-compare that was telling me I typoed that ifdef open.

comm-central doesn't have to do more because it's not libxul; it has to do more because it actually has make package-compare, so it knows what it isn't shipping. I'll poke at josh and try to make him admit whether the way Fx has been missing it for as long as it's had a manifest for Mac means it's useless, or means that Fx has been broken.
Attachment #428000 - Attachment is obsolete: true
Attachment #428028 - Flags: review?(kairo)
Attachment #428000 - Flags: review?(kairo)
(In reply to comment #3)
> I'll poke at josh and try to make him admit whether the way Fx has
> been missing it for as long as it's had a manifest for Mac means it's useless,
> or means that Fx has been broken.

Thanks: that was the other (implicit) part of my wondering ;->

NB: Would you know why FF doesn't use 'make package-compare'? It looks like a pity (which I've run into a few times).
Comment on attachment 428028 [details] [diff] [review]
No, really fix

Thanks a lot for looking into that!
Attachment #428028 - Flags: review?(kairo) → review+
http://hg.mozilla.org/comm-central/rev/b6e895490411
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
V.Fixed, per our packaged tests tinderboxes :-)
Severity: normal → major
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: