On Wed, Aug 3, 2016 at 12:21 PM, Jim Meyering <j...@meyering.net> wrote:
> On Wed, Aug 3, 2016 at 12:52 AM, Dagobert Michelsen <d...@opencsw.org> wrote:
>> Hi Jim,
>
> Thanks for more quick testing.
>
>>> Am 03.08.2016 um 05:00 schrieb Jim Meyering <j...@meyering.net>:
>>>
>>> Here's another snapshot.
> ...
>> Now I get on Solaris 10 x86:
>>
>>> gmake check-TESTS
>>> gmake[1]: Entering directory
>>> '/home/dam/mgar/pkg/diffutils/trunk/work/solaris10-i386/build-isa-pentium_pro/diffutils-3.3.57-a37c/tests'
>>> gmake[2]: Entering directory
>>> '/home/dam/mgar/pkg/diffutils/trunk/work/solaris10-i386/build-isa-pentium_pro/diffutils-3.3.57-a37c/tests'
>>> diff: missing operand after `diff'
>>> diff: Try `diff --help' for more information.
>>> diff3: missing operand after `diff3'
>>> diff3: Try `diff3 --help' for more information.
>>> sdiff: missing operand after `sdiff'
>>> sdiff: Try `sdiff --help' for more information.
>>> /bin/sh: built_programs^Jdiff^Jdiff3^Jsdiff: is not an identifier
>
> I suspect that you are using an old version of GNU make.
> What if you use Solaris' make?
> This patch should work around it, but then again, this patch should
> not be required, and I doubt I will have to resort to that approach.
I forgot to attach that patch, but I didn't like it anyhow.
Here's one that I might actually apply.
Please let me know if it helps.
It is a bit contorted in attempt to keep the final "sed" invocation
portable by operating only on data with at least one trailing newline.
diff --git a/tests/Makefile.am b/tests/Makefile.am
index e8a36ac..a4da757 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -79,7 +79,8 @@ LOG_COMPILER= $(SHELL)
built_programs = \
echo 'spy:;@echo $$(PROGRAMS)' \
- | (cd ../src && MAKEFLAGS= $(MAKE) -s -f Makefile -f - spy)
\
- | tr ' ' '\n' | sed '/^$$/d; s,$(EXEEXT)$$,,' | sort -u
+ | { (cd ../src && MAKEFLAGS= $(MAKE) -s -f Makefile -f - spy) \
+ | tr ' ' '\n' | sed '/^$$/d; s,$(EXEEXT)$$,,' | sort -u \
+ | tr '\n' ' '; echo; } | sed 's/ $$//'
VERBOSE = yes
