Re: [PATCH] Add extra 'info os' information types for Linux (trunk and 7.4)

Mark Kettenis
2011-12-28T20:40:06Z

> Date: Wed, 28 Dec 2011 00:02:57 +0200
> From: Eli Zaretskii <eliz@gnu.org>
>
> > Date: Tue, 27 Dec 2011 22:24:39 +0100 (CET)
> > From: Mark Kettenis <mark.kettenis@xs4all.nl>
> > CC: stanshebs@earthlink.net, gdb-patches@sourceware.org
> >
> > Eli, I don't think your objection makes a lot of sense.
>
> That kind of argument is not an efficient way of making me change my
> mind.

Apologies. That came out a bit stronger than intended. I'm not even
sure I'm trying to change your mind here.

> > The "info os"
> > is a generic command for displaying "osdata" that's made available by
> > the backend. It is for the backend to decide what information is made
> > available. This diff just adds a bit more "osdata" to the Linux
> > native backend and the Linux gdbserver remote backend.
>
> Then let's lump there also the DOS- and Windows-specific "info"
> commands for a good measure.
>
> IOW, either "info os" is a hodgepodge of every OS-specific information
> we provide about the process being debugged, or we have OS-specific
> "info FOO" ("info dos", "info w32", "info linux", etc.) commands.
> Having some of this and some of that is just inconsistent.

I'm not disagreeing here. It's just that the mechanism behind "info
os" is totally different so unification here is not trivial.