]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
warn if "source" fails to open the file when from_tty == 0
authorJoel Brobecker <brobecker@gnat.com>
Fri, 11 Oct 2013 08:23:11 +0000 (08:23 +0000)
committerJoel Brobecker <brobecker@gnat.com>
Fri, 11 Oct 2013 08:23:11 +0000 (08:23 +0000)
commit7c647d615590f8d1c4cca70f547534a9792f45c3
treed6f5e31bcced8897c29305006c553eb9af2e0de1
parent0cf4063e29439cdf8693d2357ead6659c6defd74
warn if "source" fails to open the file when from_tty == 0

Consider the following example:

    % gdb -q -batch -ex 'source nonexistant-file'
    [nothing]

One would have at least expected the debugger to warn about
not finding the file, similar to the error shown when using
a more interactive mode. Eg:

    (gdb) source nonexistant-file
    nonexistant-file: No such file or directory.

Not raising an error appears to be intentional, presumably in order
to prevent this situation from stoping the execution of a GDB script.
But the lack of at least a warning makes it harder for a user to
diagnose any issue, if the file was expected to be there and readable.

This patch adds a warning in that case:

    % gdb -q -batch -ex 'source nonexistant-file'
    warning: nonexistant-file: No such file or directory.

gdb/ChangeLog:

        * utils.h (perror_warning_with_name): Add declaration.
        * utils.c (perror_warning_with_name): New function.
        * cli/cli-cmds.c (source_script_with_search): Add call to
        perror_warning_with_name if from_tty is nul.

gdb/testsuite/ChangeLog:

        * gdb.base/source-nofile.gdb: New file.
        * gdb.base/source.exp: Add two tests verifying the behavior when
        the "source" command is given a non-existant filename.
gdb/ChangeLog
gdb/cli/cli-cmds.c
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/source-nofile.gdb [new file with mode: 0644]
gdb/testsuite/gdb.base/source.exp
gdb/utils.c
gdb/utils.h