Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Nathaniel Flath <flat0103@gmail.com>
To: Jan Kratochvil <jan.kratochvil@redhat.com>
Cc: Eli Zaretskii <eliz@gnu.org>,
	tromey@redhat.com, gdb-patches@sourceware.org
Subject: Re: [commit] [PATCH] cd command defaults to ~
Date: Tue, 07 Aug 2012 07:31:00 -0000	[thread overview]
Message-ID: <CAPrg3HD+R24FeeF4uqshCO73prZe2AuuKghPwFOJ6o+gttZQhg@mail.gmail.com> (raw)
In-Reply-To: <20120806171437.GA10148@host2.jankratochvil.net>

Sorry - I saw your later email about the regression - do I need to fix
this commit or did you fix/apply it?

Thanks,
Nathaniel Flath

On Mon, Aug 6, 2012 at 10:14 AM, Jan Kratochvil
<jan.kratochvil@redhat.com> wrote:
> On Mon, 06 Aug 2012 11:14:57 +0200, Nathaniel Flath wrote:
>> gdb/
>> 2012-07-21  Nathaniel Flath  <flat0103@gmail.com>
>>
>>         * NEWS: New entry for 'cd' default parameters.
>>         * cli/cli-cmds.c (cd_command): Replace error_no_arg by DIR assignment.
>>
>> doc/
> gdb/doc/
>> 2012-07-21  Nathaniel Flath  <flat0103@gmail.com>
>>
>>         * gdb.texinfo (Working Directory): Added information about new
>>           default argument for 'cd' command.
>           ^^
> Here the indentation should be two spaces left, ignoring '* ' above.
>
> The NEWS entry was against too old tree, adjusted its position (to be post-7.5).
>
>
> Thanks,
> Jan
>
>
> http://sourceware.org/ml/gdb-cvs/2012-08/msg00048.html
>
> --- src/gdb/ChangeLog   2012/08/03 20:52:46     1.14559
> +++ src/gdb/ChangeLog   2012/08/06 17:13:23     1.14560
> @@ -1,3 +1,8 @@
> +2012-08-06  Nathaniel Flath  <flat0103@gmail.com>
> +
> +       * NEWS: New entry for 'cd' default parameters.
> +       * cli/cli-cmds.c (cd_command): Replace error_no_arg by DIR assignment.
> +
>  2012-08-03  Tom Tromey  <tromey@redhat.com>
>
>         * dwarf2-frame.c (dwarf2_frame_cache): Call do_cleanups before
> --- src/gdb/NEWS        2012/07/26 15:28:21     1.537
> +++ src/gdb/NEWS        2012/08/06 17:13:26     1.538
> @@ -3,6 +3,9 @@
>
>  *** Changes since GDB 7.5
>
> +* The 'cd' command now defaults to using '~' (the home directory) if not
> +  given an argument.
> +
>  * New configure options
>
>  --enable-libmcheck/--disable-libmcheck
> --- src/gdb/cli/cli-cmds.c      2012/07/26 16:57:22     1.132
> +++ src/gdb/cli/cli-cmds.c      2012/08/06 17:13:26     1.133
> @@ -365,7 +365,7 @@
>    dont_repeat ();
>
>    if (dir == 0)
> -    error_no_arg (_("new working directory"));
> +    dir = "~";
>
>    dir = tilde_expand (dir);
>    make_cleanup (xfree, dir);
> --- src/gdb/doc/ChangeLog       2012/08/06 14:28:45     1.1352
> +++ src/gdb/doc/ChangeLog       2012/08/06 17:13:27     1.1353
> @@ -1,3 +1,8 @@
> +2012-08-06  Nathaniel Flath  <flat0103@gmail.com>
> +
> +       * gdb.texinfo (Working Directory): Added information about new
> +       default argument for 'cd' command.
> +
>  2012-08-06  Yao Qi  <yao@codesourcery.com>
>
>         * gdb.texinfo (Remote Configuration): Add kindex for 'set
> --- src/gdb/doc/gdb.texinfo     2012/08/06 14:28:45     1.996
> +++ src/gdb/doc/gdb.texinfo     2012/08/06 17:13:27     1.997
> @@ -2267,8 +2267,9 @@
>  @table @code
>  @kindex cd
>  @cindex change working directory
> -@item cd @var{directory}
> -Set the @value{GDBN} working directory to @var{directory}.
> +@item cd @r{[}@var{directory}@r{]}
> +Set the @value{GDBN} working directory to @var{directory}.  If not
> +given, @var{directory} uses @file{'~'}.
>
>  @kindex pwd
>  @item pwd


  parent reply	other threads:[~2012-08-07  7:31 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-05  2:31 Nathaniel Flath
2012-07-05 14:00 ` Jan Kratochvil
2012-07-05 17:34   ` Nathaniel Flath
2012-07-05 17:50     ` Eli Zaretskii
2012-07-05 18:09     ` Tom Tromey
2012-07-07  4:29       ` Nathaniel Flath
2012-07-07  6:17         ` Jan Kratochvil
     [not found]           ` <CAPrg3HBCBs5tmuM9qH=Q=_PgPVb8Rb-=fbPX_ZSy1dEWgKcY9g@mail.gmail.com>
     [not found]             ` <20120722163408.GA20790@host2.jankratochvil.net>
2012-07-22 16:37               ` Nathaniel Flath
2012-07-22 16:50                 ` Jan Kratochvil
2012-07-23  7:42                   ` Sergio Durigan Junior
2012-07-23 14:14                   ` Tom Tromey
2012-08-04  5:55                     ` Nathaniel Flath
2012-08-04  6:05                       ` Sergio Durigan Junior
2012-08-04  7:11                       ` Eli Zaretskii
2012-08-04 17:10                         ` Nathaniel Flath
2012-08-04 17:16                           ` Eli Zaretskii
2012-08-04 18:16                             ` Nathaniel Flath
2012-08-04 18:28                               ` Jan Kratochvil
     [not found]                                 ` <CAPrg3HDpFytQmCHe2GSzW=_ckAcNXN5UPJ5qPWDPHgwzY8RWzw@mail.gmail.com>
2012-08-06  9:15                                   ` Nathaniel Flath
2012-08-06 18:43                                     ` [commit] " Jan Kratochvil
2012-08-07  5:52                                       ` [obv] testsuite: default.exp: Fix cd regression [Re: [commit] [PATCH] cd command defaults to ~] Jan Kratochvil
2012-08-07  7:31                                       ` Nathaniel Flath [this message]
2012-08-07  7:53                                         ` [commit] [PATCH] cd command defaults to ~ Jan Kratochvil
2012-08-07 16:02                                           ` Nathaniel Flath
2012-07-22 17:13                 ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAPrg3HD+R24FeeF4uqshCO73prZe2AuuKghPwFOJ6o+gttZQhg@mail.gmail.com \
    --to=flat0103@gmail.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=tromey@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox