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

[Bertrand Garrigues]

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

Here is a new patch.

1. Diff

As Peter said, both GNU diff and Solaris diff support -D, but #ifndef / #endif
are treated differently (GNU adds a '!' in the #endif). The easiest way to
solve this problem seems to modify the regexp (currently either a '!' or a
'not' was mandatory).

2. test -ef

OK ls -i is not the solution, stat is not posix, so I proceeded as follows:
  - Test if the shell supports test -ef
  - Test if bash is available, and use it by defautl in gdiffmk (otherwise we
keep /bin/sh).
In gdiffmk, if both tests were not successful, the option to pass a third file
as the output is disabled (gdiffmk returns an error, make check will fail).

3. Sed

I don't think that ! are the problem, but the space after the exclamation mark
is clearly not correct.  I forgot to remove one space.

Peter, Mike, could you please test?

Regards,

Bertrand Garrigues


(file #33881)
    _______________________________________________________

Additional Item Attachment:

File name: bug_44768_patch_2.patch        Size:3 KB


    _______________________________________________________

Reply to this item at:

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

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