295060 - NSS test cipher.sh doesn't run in some MKS shells

Status: RESOLVED FIXED

(NSS :: Test, defect, P2)

Description

[Nelson Bolyard (seldom reads bugmail)]

As reported in bug 193055, some versions of MKS shell have a problem
with the construct
  cat somefile | while read var1 var2 var3; do .....; done

The afflicted shells only read the first line of piped input.
The fix is to change all those constructs to instead look like this:
  while read var1 var2 var3; do .....; done < somefile

That has already been done to most, but not all, NSS test scripts.  
The shell scripts in nss/test/cipher need this treatment.  
Patch forthcoming.

Comment 1

[Nelson Bolyard (seldom reads bugmail)]

Taking bug (I meant to assign it to myself)

Comment 2

[Nelson Bolyard (seldom reads bugmail)]

Attachment: patch v1

Wan-Teh, please review.

Comment 3

[Wan-Teh Chang]

Comment on attachment 184197 [details] [diff] [review]
patch v1

r=wtc.

Comment 4

[Nelson Bolyard (seldom reads bugmail)]

Thanks for the review Wah-Teh.

Checking in cipher.sh;       new revision: 1.9; previous revision: 1.8
Checking in performance.sh;  new revision: 1.3; previous revision: 1.2