Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] basename: a faulty warning 'extra operand --test-name' in tests causes test-driver to fail
From: Jeff Squyres (jsquyres) (jsquyres_at_[hidden])
Date: 2013-07-12 08:26:05


I'm sorry, I'm still unclear what you're trying to tell us. :-(

1. What version of Open MPI are you testing? If you're testing Open MPI 1.6.x with very new Automake, I'm not surprised that there's some failures. We usually pick the newest GNU Autotools when we begin a release series, and then stick with those tool versions for the life of that series. We do not attempt to forward-port to newer Autotools on that series, meaning that sometimes newer versions of the Autotools will break the builds of that series. That's ok.

2. Assumedly, you're seeing this failure when you run "make check". Is that correct? What test, exactly, is failing? It's very difficult to grok what you're reporting when you only include the last few lines of output, which exclude the majority of the context that we need to know what you're talking about.

Your bug reports have been *extremely* helpful in cleaning out some old kruft from our tree, but could you include more context in the future? E.g., include all the "compile problems" items from here:

    http://www.open-mpi.org/community/help/

3. We don't have a test named "basename" or "test-driver"; basename is usually an OS utility, and test-driver is part of the new Automake testing framework. If there's a mistake in how these are being invoked, it's coming from Automake, and you should report the bug to them.

...unless we're doing something wrong in our Makefile.am's in how we list the tests to be run. Is that what you're saying?

On Jul 12, 2013, at 3:59 AM, Vasiliy <testtest_2005_at_[hidden]> wrote:

> Oh, sorry. It is an Automake bug in terms of reacting to the
> --log-file option, but 'basename' tells also it does not understand /
> do not pass --test-name to 'test-driver', which, in turn, triggers the
> above failure for yet another reason. So, it is combined.
>
> On Thu, Jul 11, 2013 at 11:18 PM, Jeff Squyres (jsquyres)
> <jsquyres_at_[hidden]> wrote:
>> I'm not sure what you're saying -- isn't this an Automake bug?
>>
>> Or are you saying that we're doing something wrong in OMPI's Makefile.am's?
>>
>>
>>
>> On Jul 11, 2013, at 7:47 AM, Vasiliy <testtest_2005_at_[hidden]> wrote:
>>
>>> I've also tracked down that problem with 'test-driver'. Look at that:
>>>
>>> $ gdb --args /usr/bin/sh /usr/share/automake-1.14/test-driver
>>> GNU gdb (GDB) 7.6.50.20130320-cvs
>>> Copyright (C) 2013 Free Software Foundation, Inc.
>>> License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
>>> This is free software: you are free to change and redistribute it.
>>> There is NO WARRANTY, to the extent permitted by law. Type "show copying"
>>> and "show warranty" for details.
>>> This GDB was configured as "x86_64-unknown-cygwin".
>>> For bug reporting instructions, please see:
>>> <http://www.gnu.org/software/gdb/bugs/>...
>>> Reading symbols from /usr/bin/sh...Reading symbols from
>>> /usr/lib/debug/usr/bin/sh.exe.dbg...done.
>>> done.
>>> (gdb) run
>>> Starting program: /usr/bin/sh /usr/share/automake-1.14/test-driver
>>> [New Thread 9900.0xc10]
>>> [New Thread 9900.0x1bec]
>>> [New Thread 9900.0xe38]
>>> /usr/share/automake-1.14/test-driver: line 95: $log_file: ambiguous redirect
>>> FAIL:
>>> /usr/share/automake-1.14/test-driver: line 114: $trs_file: ambiguous redirect
>>> /usr/share/automake-1.14/test-driver: line 115: $trs_file: ambiguous redirect
>>> /usr/share/automake-1.14/test-driver: line 116: $trs_file: ambiguous redirect
>>> /usr/share/automake-1.14/test-driver: line 117: $trs_file: ambiguous redirect
>>> [Inferior 1 (process 9900) exited with code 01]
>>> (gdb) quit
>>>
>>> $ gdb --args /usr/bin/sh /usr/share/automake-1.14/test-driver --log-file=/tmp
>>> GNU gdb (GDB) 7.6.50.20130320-cvs
>>> Copyright (C) 2013 Free Software Foundation, Inc.
>>> License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
>>> This is free software: you are free to change and redistribute it.
>>> There is NO WARRANTY, to the extent permitted by law. Type "show copying"
>>> and "show warranty" for details.
>>> This GDB was configured as "x86_64-unknown-cygwin".
>>> For bug reporting instructions, please see:
>>> <http://www.gnu.org/software/gdb/bugs/>...
>>> Reading symbols from /usr/bin/sh...Reading symbols from
>>> /usr/lib/debug/usr/bin/sh.exe.dbg...done.
>>> done.
>>> (gdb) run
>>> Starting program: /usr/bin/sh /usr/share/automake-1.14/test-driver
>>> --log-file=/tmp
>>> [New Thread 2164.0x164c]
>>> [New Thread 2164.0x24a4]
>>> [New Thread 2164.0x2550]
>>> /usr/share/automake-1.14/test-driver: invalid option: '--log-file=/tmp'
>>> [New Thread 2164.0x19d4]
>>> Usage:
>>> test-driver --test-name=NAME --log-file=PATH --trs-file=PATH
>>> [--expect-failure={yes|no}] [--color-tests={yes|no}]
>>> [--enable-hard-errors={yes|no}] [--] TEST-SCRIPT
>>> The '--test-name', '--log-file' and '--trs-file' options are mandatory.
>>>
>>> So, there is a problem with 'test-driver' either because a testsuite
>>> does not provide --test-name=NAME or because --log-file=/tmp or
>>> --log-file=/tmp/delme is wrongly considered an invalid option. It
>>> applies to automake 1.13 as well.
>>>
>>> Could an Open MPI Team suggest if we could change that behavior, or,
>>> at least, make omitting --test-name not so critical?
>>>
>>>
>>> ---------- Forwarded message ----------
>>> From: Vasiliy
>>> Date: Thu, Jul 11, 2013 at 1:31 PM
>>> Subject: basename: a faulty warning 'extra operand --test-name' in
>>> tests causes test-driver to fail
>>> To: Open MPI Developers
>>>
>>>
>>> upon inspecting:
>>> $ /usr/share/automake-1.14/test-driver --help
>>> Usage:
>>> test-driver --test-name=NAME --log-file=PATH --trs-file=PATH
>>> [--expect-failure={yes|no}] [--color-tests={yes|no}]
>>> [--enable-hard-errors={yes|no}] [--] TEST-SCRIPT
>>> The '--test-name', '--log-file' and '--trs-file' options are mandatory.
>>> <code>
>>> make check-TESTS
>>> make[1]: Entering directory
>>> '/usr/src/64bit/release/openmpi/openmpi-1.9.0-a1/build/test/asm'
>>> make[2]: Entering directory
>>> '/usr/src/64bit/release/openmpi/openmpi-1.9.0-a1/build/test/asm'
>>> basename: extra operand `--test-name'
>>> Try `basename --help' for more information.
>>> --> Testing
>>> basename: extra operand `--test-name'
>>> Try `basename --help' for more information.
>>> --> Testing
>>> basename: extra operand `--test-name'
>>> Try `basename --help' for more information.
>>> --> Testing
>>> basename: extra operand `--test-name'
>>> Try `basename --help' for more information.
>>> --> Testing
>>> ...
>>>
>>> /usr/src/64bit/release/openmpi/openmpi-1.9.0-a1/src/openmpi-1.9.0/config/test-driver:
>>> line 95: <PID> Segmentation fault (core dumped) "$@" > $log_file
>>> 2>&1
>>> </code>
>>> _______________________________________________
>>> devel mailing list
>>> devel_at_[hidden]
>>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>
>>
>> --
>> Jeff Squyres
>> jsquyres_at_[hidden]
>> For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/
>>
>>
>> _______________________________________________
>> devel mailing list
>> devel_at_[hidden]
>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
> _______________________________________________
> devel mailing list
> devel_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/devel

-- 
Jeff Squyres
jsquyres_at_[hidden]
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/