Re: [PATCH PR gdb/15715] 'set history filename' to by immediately converted to absolute path.

[Muhammad Bilal <mbilal at codesourcery dot com>]

On 07/30/2013 04:00 PM, Pedro Alves wrote:
> On 07/30/2013 11:46 AM, Muhammad Bilal wrote:
>
> > 2012-07-30  Muhammad Bilal  <mbilal@codesorcery.com>
> >
> >           PR gdb/15715
> >           * top.c: Include "filenames.h".
> >           (set_history_filename): New function.
> >           (init_main): Install it as set hook of the "set history filename"
> >           command.
> >
> > /testsuit
> > 2012-07-30  Muhammad Bilal  <mbilal@codesourcery.com>
> >
> >           PR gdb/15715
> >           * gdb.base/setshow.exp: Test that relative paths passed to
> >           'set history filename' are converted to absolute paths
> You've lost the period at end of this sentence.  It was there in
> previous versions.  I'd like to be sure you learn to pay attention to
> these details, hence the multiple iterations, but I'd like to avoid
> having to go through them for all patches.   Please be attentive.  This
> is OK with the missing period added (though as penance, I really should
> have you go through one more round of posting before approving.  ;-) )
committed

http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/top.c.diff?cvsroot=src&r1=1.239&r2=1.240
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/ChangeLog.diff?cvsroot=src&r1=1.15844&r2=1.15845
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/gdb.base/setshow.exp.diff?cvsroot=src&r1=1.33&r2=1.34
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/ChangeLog.diff?cvsroot=src&r1=1.3748&r2=1.3749



Thanks,
-Bilal