Re: [RFA] Add -s option to source command.

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Tue, Apr 6, 2010 at 2:57 PM, Doug Evans <dje@google.com> wrote:
> Hi.
>
> In a big source tree with multiple disparate components it's useful to be
> able to load scripts without having to type full path names.
>
> This patch adds a -s option to the source command to make it search
> for the script in the source search path even if the script name specifies
> a directory.
> For example, adding the top level source directory to the source search path
> lets one load scripts from any component by only typing the path from
> the top directory.
>
> I thought of simply extending the source command to do the search anyway
> but the current code explicitly doesn't do that to make "./foo" only mean
> "foo" in the current directory. ?Granted, the comment (see openp) is
> referring to executables, but it's not clear to me it shouldn't also
> apply to scripts. ?Plus, "source /absolute/path" should probably not
> scan the search path by default.
> So I went with adding an option.
>
> This patch also does a minor reorg of find_and_open_script so that
> it's useful in a subsequent patch I'll be submitting.
> I think the reorg is useful in itself, e.g. removing the from_tty
> arg to find_and_open_script, so I've including it here.
>
> Included are doc and testcase additions.
>
> Ok to check in?
>
> 2010-04-06 ?Doug Evans ?<dje@google.com>
>
> ? ? ? ?Add -s option to source command.
> ? ? ? ?* cli/cli-cmds.c (find_and_open_script): Add function comment.
> ? ? ? ?Delete from_tty and cleanupp args. ?Split filep arg into file and
> ? ? ? ?full_pathp. ?New arg search_path.
> ? ? ? ?(source_script_from_stream): New function.
> ? ? ? ?(source_script_with_search): New function.
> ? ? ? ?(source_script): Rewrite.
> ? ? ? ?(source_command): Parse "-s" option.
> ? ? ? ?(init_cli_cmds): Add "-s" docs to source command help.
> ? ? ? ?* python/python.c (source_python_script): Make file arg a const char *.
> ? ? ? ?Don't call fclose, leave for caller.
> ? ? ? ?* python/python.h (source_python_script): Update.
>
> ? ? ? ?testsuite/
> ? ? ? ?* gdb.base/source-test.gdb: New file.
> ? ? ? ?* gdb.base/source.exp: Add tests for "source -v" and "source -s".
>
> ? ? ? ?doc/
> ? ? ? ?* gdb.texinfo (Command Files): Add docs for new "source -s" option.
>

This caused:

help source^M
Read commands from a file named FILE.^M
^M
Usage: source [-s] [-v] FILE^M
-s: search for the script in the source search path,^M
    even if FILE contains directories.^M
-v: each command in FILE is echoed as it is executed.^M
^M
Note that the file ".gdbinit" is read automatically in this way^M
when GDB is started.^M
(gdb) FAIL: gdb.base/help.exp: help source



-- 
H.J.