This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [python] [patch] PR python/13345


>>>>> "Meador" == Meador Inge <meadori@codesourcery.com> writes:

Meador> On 10/31/2011 10:51 AM, Phil Muldoon wrote:
>> This patch fixes a case where the tilde (~) command was being passed to
>> Python via the "source" command.  Python does not understand what to do
>> with a tilde, so we have to expand it first.

Meador> While I can't give an OK, I did review this and was able to reproduce
Meador> the stated problem and the patch fixes it.  So, LGTM.  What about
Meador> a test case, though?  You could construct a relative path to a
Meador> test directory from '~/'.

I think in this case it is ok not to have a test.

But if you want to write one it would be fine by me.
It would have to be xfail'd if not under $HOME.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]