Patch for PCORE on linuxppc_2_4_devel

[Tom Rini]

On Mon, Apr 07, 2003 at 11:22:26AM -0700, Mark A. Greer wrote:

> Tom Rini wrote:
>
> >On Fri, Apr 04, 2003 at 06:12:17PM -0700, Mark A. Greer wrote:
> >
> >>This patch does the following:
> >>a) Modifies the CONFIG_MORE_COMPILE_OPTIONS setup at the bottom of
> >>arch/ppc/config.in.  The way that it was didn't work well and seemed
> >>difficult to read.  I believe that the new way is "more correct" and
> >>easier to understand
> >
> >Didn't work well when?  Then intent is that if you don't select this
> >option, you still get -g added when CONFIG_KGDB=y so that you can
> >effectivly use gdb.
>
> Try playing with it a little...

I did.  It works like it should.
No (XMON || KGDB || BDI) -> nothing
XMON || BDI && !MORE -> nothing
KGDB && !MORE -> '-g' && MORE being set.

I even gave xconfig a whirl with both MORE being both set and unset at
the same time in the .config.  It won't let you say no, which is again,
the intent.

> I understand the intent, it just didn't work well.

When doesn't it work well?

> The option is still
> there with a reasonable default.  If they want it, they can use it; if
> they don't, they don't.  If they don't know that they probably want -g
> if they have KGDB, they'll find out soon enough.  Simple.

That's just it.  You never don't want some form of debugging symbols in
your image if you use KGDB.  This is why the i386 kgdb docucmentation
says that -g will be on if KGDB is enabled.

--
Tom Rini
http://gate.crashing.org/~trini/

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/