107796 - Add SOAP to installers

Status: VERIFIED FIXED

(Core :: XML, defect)

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

After the rewrite and some bake time we can add the new SOAP to installers.

Comment 1

[Heikki Toivonen (remove -bugzilla when emailing directly)]

rayw

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Tentatively scheduling for 0.9.7.

Comment 3

[Ray Whitmer]

Target milestone changed.

Comment 4

[Ray Whitmer]

Attachment: Add xmlsoap.xpt to the packager in 5 files (everywhere xmlextras was).

Now that SOAP has become part of the default build, it needs to be packaged. 
Here is the patch to do that.

Comment 5

[vidur (gone)]

Comment on attachment 70820 [details] [diff] [review]
Add xmlsoap.xpt to the packager in 5 files (everywhere xmlextras was).

r/sr=vidur

Comment 6

[Ray Whitmer]

Attachment: Output of windows wizard building of installer.

Comment 7

[Ray Whitmer]

Attachment: Error output of windows wizard building of installer

I do not believe that my changes caused the errors, but I clearly need to have
someone look at it who understands installer better than I do.

Comment 8

[Daniel Veditz [:dveditz]]

Comment on attachment 70820 [details] [diff] [review]
Add xmlsoap.xpt to the packager in 5 files (everywhere xmlextras was).

r=dveditz
Looks like the right thing

Comment 9

[Daniel Veditz [:dveditz]]

The missing component warnings are noise from people who forget to take obsolete
files out. The build team files bugs on this from time to time because the bogus
warnings get in the way of figuring out when the build has broken. For example,
it appears you didn't build security and that's the kind of thing the build team
does want warnings about.

The zip error looks like it's calling a different zip program than the one that
worked earlier. I'm guessing we should change the script to just pick up
whatever's on your path now that zip has long been a standard requirement.
Either copy your cygwin zip into your moztools\bin directory, or edit line 375
of mozilla/xpinstall/packager/windows/makeall.pl to remove the path from the zip
command.

Comment 10

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 70820 [details] [diff] [review]
Add xmlsoap.xpt to the packager in 5 files (everywhere xmlextras was).

a=dbaron for 0.9.9

Comment 11

[Ray Whitmer]

This patch was checked in, thanks to all who helped.

Comment 12

[Rakesh Mishra]

Changing Q/A contact

Comment 13

[Rakesh Mishra]

verified the patch on trunk build 2002-06-07-04-trunk.
marking verified