413426 - script for running NSPR test can check for proper environment on execution

Status: NEW

(NSPR :: NSPR, defect)

Description

[Ray Kiddy]

Attachment: a patch for the test script to check the env on Mac OS X

As Wan-Teh Chang pointed out to me in an e-mail, the NSPR tests will not run properly unless they are run via the runtests.sh command and, on Mac OS X, the proper environment variable needs to be set.

I have changes I would suggest for the runtests.sh script so that people do not have to remember these details themselves. The script can check, on Mac OS X, that the environment variable is properly set. Computers can remember details like this better than people can remember.

I know, by the way, that I am assuming that the variable is set to a single directory. Is this a problem? It would probably be more correct for the first test to check the environment and that first executable would point out the error. Would you prefer that approach?

Also, can the test scripts be put somewhere in the build product? This is how it is done in most other Mozilla projects.

Comment 1

[Wan-Teh Chang]

Comment on attachment 298387 [details] [diff] [review]
a patch for the test script to check the env on Mac OS X

Ray, thanks for the patch.

It is better to test for libnspr4.dylib instead.
You should also allow libnspr4.dylib in the directory
where the test programs are.  That's how I run the
tests -- I copy the *.dylib files to the directory
where the test programs reside.

Remove "relative" from the error message.

Comment 2

[Wan-Teh Chang]

Another approach is to just copy ../../dist/lib/*.dylib
to the current directory for the user.  We can also use
symlinks.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.