Static build on OS/2

VERIFIED FIXED in mozilla0.9.3

Status

SeaMonkey
Build Config
P3
normal
VERIFIED FIXED
17 years ago
13 years ago

People

(Reporter: jhp (no longer active), Assigned: cls)

Tracking

Trunk
mozilla0.9.3
x86
OS/2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
Build static build on OS/2.  Based on the linux code from 
bug 46775.
(Reporter)

Updated

17 years ago
Depends on: 46775
(Reporter)

Comment 1

17 years ago
Created attachment 37947 [details] [diff] [review]
patch for os/2 changes
(Reporter)

Comment 2

17 years ago
With the changes from the above patch, as well as those in 
bugs 84153 and 83878, the os/2 static build successfully 
comes up.  The only problem (happens on os/2 but not on 
linux) is this assertion:

Assertion: nsLocaleModule: Module already created: 
'gModule==NULL', file 
intl/locale/src/os2/nsLocaleModule.cpp, line 329

Still haven't figured out why this happens.

Updated

17 years ago
Priority: -- → P3
Target Milestone: --- → mozilla0.9.3
(Assignee)

Comment 3

17 years ago
Javier, could you update the patch against the latest static build branch,
STATIC_BUILD_20010612_BRANCH ?  Waterson's pushing for getting the main static
build changes landed on Monday and I'd like to get these changes in as well.  
Status: NEW → ASSIGNED
Target Milestone: mozilla0.9.3 → mozilla0.9.2
(Reporter)

Comment 4

17 years ago
Chris:
I'm running into some problems.  The new build-list.pl perl 
file fails on OS/2.  I get this error: "grep: Bad file 
number".  Any idea why this is?  I reverted to the old file 
in order to complete the build.  I'll post a patch as soon 
as I can, but it probably won't be until tuesday.
(Assignee)

Comment 5

17 years ago
build-list.pl was changed to use the output of the grep -c command rather than
the return value of grep.  Not sure why you would be getting that error.  Can
you remove the @ from the cmdline that runs build-list so we can see what
command it is actually trying to use?
(Reporter)

Comment 6

17 years ago
Well, I'm not sure exactly what you want, but here is the 
command as it gets called:

D:/EMX/bin/perl.exe D:/mozilla/config/build-list.pl 
../../config/final-link-libs mozjs

And it fails on the line:

open(RES, "grep -c '^$entry\$' $file |") or $err = $!;

Not sure exactly why, but it works correctly if I remove 
"or $err = $!" from the end of the line.  Any idea about 
what is going on?

Also, does this check to see if the module is already 
written in the file?  Maybe it is due to my hack, but if I 
remove the end of the line, I can write multiple instances 
of an entry to the file.
(Reporter)

Comment 7

17 years ago
Created attachment 39143 [details] [diff] [review]
updated patch
(Reporter)

Comment 8

17 years ago
Latest patch is updated to the 
STATIC_BUILD_20010612_BRANCH.  Build-list.pl problems are 
still unresolved.
(Assignee)

Comment 9

17 years ago
Waterson, could you take a look at the non-OS2 specific portions of the patch? 
Mkaply, could you take a look at the OS2 specific portions of the patch?

Comment 10

17 years ago
r=mkaply on the OS/2 stuff.

Comment 11

17 years ago
r=waterson
(Reporter)

Comment 12

17 years ago
cls:
Any input on the build-list.pl problem?
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.2 → mozilla0.9.3
(Reporter)

Comment 13

17 years ago
Finally solved our build-list.pl problem.  The perl distribution on 
OS/2 emulates flock, which was causing our problems.  By setting 
USE_PERL_FLOCK=0, we fall back on the flock in EMX (nonfunctional, 
just a stub; same a passing -l to build-list.pl).  Also needed to use 
the sh.exe shell that comes with perl.  Will soon update instructions 
at www.mozilla.org/ports/os2.

Everything is ready to go now.
(Assignee)

Comment 14

17 years ago
The updated patch has been checked in.  I think we can work around the flock
problem by using config/mozLock.pm instead of perl's builtin flock().  I keep
meaning to switch the scripts that use locking over to it but never find the
time....
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Reporter)

Comment 15

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