On Sat, Nov 17, 2007 at 02:39:18PM -0500, Robert P. J. Day wrote:
> On Sat, 17 Nov 2007, Sam Ravnborg wrote:
>
> > On Sat, Nov 17, 2007 at 02:21:10PM -0500, Robert P. J. Day wrote:
> > >
> > > a *very* quick response before i have to dash ...
> > >
> > > On Sat, 17 Nov 2007, Sam Ravnborg wrote:
> > >
> > > > > - Note: as-instr-option uses KBUILD_AFLAGS for $(AS) options
> > > >
> > > > This note is important - do not delete it. And it is correct - no?
> > >
> > > what *is* "as-instr-option"? the reason i removed it was because it
> > > doesn't exist anywhere in the source tree, at least from a quick scan
> > > i did. can you point me at where it's being defined/used? thanks.
> > >
> > Typing error. It should read "as-instr".
> > Thanks for noticing the error.
> >
> > And for as-option it should be documentet that it uses KBUILD_CFLAGS.
> > Can you please add it too.
>
> well, looks like i didn't have to dash after all.
>
> tell you what ... i'll submit a shorter patch fixing just the simple
> typoes, then spend more time on the more involved stuff. might as
> well break it into more bite-sized pieces, how about that?
Break it up only if it helps you.
I would assume that next patch would only have minor issues if any so
I do not see the need for smaller pieces for this patch.
Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
