639369 - [OS/2] Update packaging for mozilla-central and comm-central

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Rich Walsh]

These patches accommodate changes in our build environment:
- add *.xqs to the files copied to the packaging directory
- set the 'X' option for zip.exe to prevent extended attributes from being packaged
- run lxlite only once with all possible parameters specified and let it recurse through the packaging tree looking for *.exe and *.dll

The icon corruption that occurs when exe's are compressed is now known to be caused by overly large icon files.  This issue should be addressed in a separate bug as it doesn't directly involve packaging.

BTW.. please correct the Product and Component selections if they are incorrect.

Comment 1

[Rich Walsh]

Attachment: update OS/2 packaging for mozilla-central

Comment 2

[Rich Walsh]

Attachment: update OS/2 packaging for comm-central

Comment 3

[Walter Meinl]

Comment on attachment 517299 [details] [diff] [review]
update OS/2 packaging for mozilla-central

All ifdefed for OS/2 or in OS/2 only files. Risk is more or less zero for tier_1 platforms

Comment 4

[Daniel Veditz [:dveditz]]

Comment on attachment 517299 [details] [diff] [review]
update OS/2 packaging for mozilla-central

Approved for the mozilla2.0 repository, a=dveditz for release-drivers

Comment 5

[Walter Meinl]

(In reply to comment #2)
> Created attachment 517301 [details] [diff] [review]
> update OS/2 packaging for comm-central

Callek, how can I get approval for c-c for this patch? I couldn't find a proper flag?

Comment 6

[Walter Meinl]

Comment on attachment 517301 [details] [diff] [review]
update OS/2 packaging for comm-central

(In reply to comment #5)
> (In reply to comment #2)
> > Created attachment 517301 [details] [diff] [review]
> > update OS/2 packaging for comm-central
> 
> Callek, how can I get approval for c-c for this patch? I couldn't find a proper
> flag?

maybe feedback will be helpful?

Comment 7

[Justin Wood (:Callek)]

Comment on attachment 517301 [details] [diff] [review]
update OS/2 packaging for comm-central

rs=me, you don't need approval for comm-central. See the SeaMonkey2.1 tree and Mirimar tree on tinderbox for tree status.

(Orange on SeaMonkey is ignorable, red is not)

If this needs to land in comm-1.9.2 or comm-1.9.1 please request approval.

Comment 8

[(no longer active)]

The patch doesn't apply cleanly on trunk any more.

Comment 9

[Walter Meinl]

Attachment: m-c patch updated to apply to trunk

Sorry for the trouble, the older m-c patch is still needed for 2.0

Comment 10

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/047b8f3d5e67

Comment 11

[Philip Chee]

Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/2fffeac3941c

Leaving open for Mozilla-2.0

Comment 12

[Daniel Veditz [:dveditz]]

resolved fixed on trunk, mozilla-2.0 status is tracked in the "status2.0" custom field (which should be set to ".1-fixed" when this lands today.

It is landing today, right? code-freeze for "Macaw" is tomorrow.

Comment 13

[Justin Wood (:Callek)]

(In reply to comment #12)
> It is landing today, right? code-freeze for "Macaw" is tomorrow.

If it hasn't landed by 10pm EDT I'll land for the OS/2 folks when I get home tonight. That way it makes macaw freeze

Comment 14

[(no longer active)]

http://hg.mozilla.org/releases/mozilla-2.0/rev/eade0686131a

Comment 15

[Walter Meinl]

(In reply to comment #14)
> http://hg.mozilla.org/releases/mozilla-2.0/rev/eade0686131a

Thank you!

Comment 16

[Walter Meinl]

oops