336521 - init.sh adds /bin and /usr/bin before ${DIST}/${OBJDIR}/bin to PATH

Status: RESOLVED FIXED

(NSS :: Test, defect)

Description

[Wan-Teh Chang]

The script tests/common/init.sh sets the PATH environment
variable so that the command-line tools such as certutil
under ${DIST}/${OBJDIR}/bin can be found.  However, it
adds the system directories /bin and /usr/bin before
${DIST}/${OBJDIR}/bin to PATH.  So, if your system has
a certutil installed in /usr/bin, our test scripts will
use that certutil instead of the one in the build tree.

Comment 1

[Wan-Teh Chang]

Attachment: Proposed patch

As the comment explains, the purpose of having /bin and /usr/bin
at the beginning of PATH is to ensure that a local perl (as opposed
to a perl installed in a network-mounted directory) is used.
Since we do not have any program named perl in our build tree,
it is safe to move ${DIST)/${OBJDIR}/bin and ${DIST}/${OBJDIR}/lib
before /bin and /usr/bin in PATH.

Comment 2

[Wan-Teh Chang]

I checked in the patch on the NSS trunk (3.12) and
NSS_3_11_BRANCH (3.11.2).