xpinstall/packager/makefile and pkgcp.pl updates

VERIFIED FIXED

Status

VERIFIED FIXED
16 years ago
14 years ago

People

(Reporter: mkaply, Assigned: mkaply)

Tracking

Trunk
Other
OS/2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

These are the changes to get OS/2 using the Makefile in xpinstall/packager.

I'll explain in the diff
Created attachment 103759 [details] [diff] [review]
Makefile updates

Pretty standard stuff.

OS/2 changes for packaging.

Change some parameters, as well as what is used for STRIP. also add code to
copy the C runtime so it gets packaged.

Also, we are a ZIP by default.

For VACPP, we change the TARGET_OS, since we default to os2-emx.
Created attachment 103760 [details] [diff] [review]
pkgcp.pl updates

This one is a little stranger.

I added back support for using os2 when invoking pkgcp.pl, but only at the top
level. The reason I did this is because we have a PERL bug where the current
directory is getting changed arbitrarily. My fix was to mangle the paths we
pass to Packager.pm to be fully qualified. We only do this in the OS/2 case,
and then we change the OS to Unix.
Comment on attachment 103759 [details] [diff] [review]
Makefile updates

r=cls
Attachment #103759 - Flags: review+
Comment on attachment 103760 [details] [diff] [review]
pkgcp.pl updates

r=cls
Attachment #103760 - Flags: review+

Updated

16 years ago
QA Contact: bugzilla → ktrina
a=dbaron for trunk checkin for both patches.
Fixes checked in.

I made a small change to the pkgcp.pl one - I only prepend the path if the
second character is not already a :

This allows fully qualified paths to work.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 7

16 years ago
Verified code fix
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.