Eric Blake wrote: > According to Eric Blake on 12/2/2008 6:03 PM: >> $ grep AC_STATE_SAVE\( tests/testsuite | wc -l >> 591 > >> Of which, many instances are inside generated configure.ac files where the >> testsuite then runs the configure script multiple times. So shaving a >> process here shaves more than 1000 forks off the total testsuite time, >> with a noticeable impact on cygwin. > > I went ahead and applied the patch, after further testing that Solaris and > BSD sed behaved correctly.
Indeed, they shouldn't have problems because their line length limit is ~4000 (giving a maximum length of ~2000 bytes for files in the directory, which is safe). However, note that the script is emulating uniq, not sort -u. I don't know if this is safe? Paolo
