Re: Python pretty-printing [1/6]

Tom Tromey
2009-04-07T18:43:11Z

>>>>> "Joel" == Joel Brobecker <brobecker@adacore.com> writes:

>> +#ifdef PYTHONDIR
>> + PyModule_AddStringConstant (gdb_module, "pythondir", PYTHONDIR);
>> +#else
>> + if (gdb_datadir)
>> + PyModule_AddStringConstant (gdb_module, "datadir", gdb_datadir);
>> +#endif

Joel> Can we change that, for instance by assuming that we always have
Joel> PYTHONDIR, and that by default, configure will set to the datadir?

This code is gone from the latest revision of the patch.
We don't actually need the pythondir stuff until we have a library of
Python code installed with gdb, and this patch series doesn't add that.

But, yes, I think we can clean this up when the time comes.

Joel> I was going to comment on the ..._RELOCATABLE macro, but I see that
Joel> we're already using that idiom for at least the system gdbinit file.

Yeah. I cleaned all this up a bit in the latest patch as well. That
is, I unified almost all of the cut-and-pasted code.

Tom