126932 - NSPR tests don't run on OS/2

Status: RESOLVED FIXED

(NSPR :: NSPR, defect, P3)

Description

[Julien Pierre]

Right now they don't even build on OS/2.

Comment 1

[Mike Kaply [:mkaply]]

Attachment: get tests building

This is all that is needed - use $(LD) instead of $(LINK)

Comment 2

[Julien Pierre]

Now that the tests are building, we need to run runtests.ksh .
The standard mozilla OS/2 tools don't appear to include ksh. Mike, do 
you know which one we should use ?
Also, I think runtests.ksh will need some modifications for OS/2 in 
order to pick the correct tests to run.

Comment 3

[Mike Kaply [:mkaply]]

Attachment: fix to runtests.sh

runtests.sh should be used - it needs to be modified.

You also need printf and basename from gnusutils.

Comment 4

[Mike Kaply [:mkaply]]

Attachment: Fix for sh and attach testcase

Might as well put two fixes in - attach test is totally broke for os/2

Comment 5

[Mike Kaply [:mkaply]]

Attachment: Need direct.h for VACPP

No idea why this is working for other platforms.

tmocon.c says that #includes are needed for getcwd

Comment 6

[Wan-Teh Chang]

Michael, please update your patches against the current
NSPR client branch.

Comment 7

[Mike Kaply [:mkaply]]

Attachment: Current diff

Sorry, I didn't realize some of this went in.

This puts one more OS/2 in the .sh file and fixes an include problem in
thrpool_server.c

Once these changes are in, this bug is done.

Comment 8

[Wan-Teh Chang]

Patch checked into the NSPR TIP and client branch.