Re: [PATCH] Support gzip compressed exec and core files in gdb

[Michael Eager <eager at eagerm dot com>]

On 03/11/15 03:07, Gary Benson wrote:
> Michael Eager wrote:
> > Add support to automatically unzip compressed executable and core
> > files.  Files will be uncompressed into temporary directory (/tmp or
> > $TMPDIR) and are deleted when GDB exits.  This should be transparent
> > to users, except for disk space requirements.  The name of the
> > uncompressed file is mentioned, but all references to the file in
> > GDB messages is to the file which the user specified.
> ...
> > diff --git a/gdb/common/filestuff.c b/gdb/common/filestuff.c
> > index 14d6324..b2c31fd 100644
> > --- a/gdb/common/filestuff.c
> > +++ b/gdb/common/filestuff.c
> ...
> > +#ifndef GDBSERVER
>
> Please do not add GDBSERVER conditionals to gdb/common, I spent half
> a year removing them all.
>
> It looks like this code is only used by GDB, not gdbserver, so the
> fix is simple, just put the code somewhere GDB-specific.

Sure.  Any suggestions?  gdb_bfd.c seems like the wrong place.


--
Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077