[bug #44768] gdiffmk(1) is not Solaris 10 /bin/sh compatible - PATCH provided

[Peter Bray]

Follow-up Comment #16, bug #44768 (project groff):

Mike (& Werner),

As you suggested the script still does not run on Solaris 10
natively. See attached test-suite.log-Solaris10-sh-sed-diff.

Reasons:
  - Solaris 10 /bin/sh does not support test's -ef test
  - GNU sed is required, 
  - GNU diff is required for reasons mentioned in usage

Changing the script to #! /bin/bash fixes the script complaint about
test's -ef option. /bin/bash is standard on Solaris 10 (version
3.00.16 on Sun Solaris 10 Update 8 and version 3.2.51 on Oracle
Solaris 10 update 11) .

I have GNU sed compiled locally as gsed, and GNU diff as gdiff,
neither of these are normally available on Solaris 10.

If I make these changes to the build script (./gdiffmk) than re-run
"gmake check", I get a clean result

Changing gsed back to sed results in the attached
test-suite.log-Solaris10-sed-GNU-diff-bash, which shows that it is
more than the failure to parse one expression that Solaris sed(1) can
not handle. As per my previous suggestion, "Developed with GNU sed(1),
other implementations not verified"

The only remaining changes that could be applied are the previously
mentioned @BASH@, @DIFF@, @SED@ substitutions at build time. But I
think Mike can consider the script complete from his point of view.

Regards,
Peter


(file #33777, file #33778)
    _______________________________________________________

Additional Item Attachment:

File name: test-suite.log-Solaris10-sh-sed-diff Size:2 KB
File name: test-suite.log-Solaris10-sed-GNU-diff-bash Size:2 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?44768>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/