Re: [PATCH 7/8] kbuild: move W=... stuff to Kbuild.arch
On Wed, Aug 15, 2012 at 12:41:23PM +0300, Artem Bityutskiy wrote: > On Wed, 2012-06-06 at 17:35 +0200, Sam Ravnborg wrote: > > On Wed, Jun 06, 2012 at 01:18:47PM +0300, Artem Bityutskiy wrote: > > > On Sat, 2012-05-05 at 10:18 +0200, Sam Ravnborg wrote: > > > > Prevent that we eveluate cc-option multiple times for the same > > > > option by moving the definitions to Kbuild.arch. > > > > The file is included once only, thus gcc is not invoked once per > > > > directory. > > > > > > > > Another side-effect of this patch is that KCFLAGS are appended last > > > > to the list of options. This allows us to better control the options. > > > > Artem Bityutskiy noticed this. > > > > > > > > Signed-off-by: Sam Ravnborg > > > > Cc: Artem Bityutskiy > > > > > > Hi, > > > > > > what happened to this patch? I was fixing the real issue I am > > > encountering and I thought it'd be taken instead of my original patch. > > We decided to move this to next merge release because is was not added > > to kbuild thus not enough exposure in -next. > > > > I am planning to resend the serie at around -rc2 time. > > Hi Sam, what happened to this patch-set? At least KCFLAGS patches I am > waiting for are still not merged. Vacation and then I have not yet gotten back to them. Will do soon - thanks for the reminder! Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 7/8] kbuild: move W=... stuff to Kbuild.arch
On Wed, 2012-06-06 at 17:35 +0200, Sam Ravnborg wrote: > On Wed, Jun 06, 2012 at 01:18:47PM +0300, Artem Bityutskiy wrote: > > On Sat, 2012-05-05 at 10:18 +0200, Sam Ravnborg wrote: > > > Prevent that we eveluate cc-option multiple times for the same > > > option by moving the definitions to Kbuild.arch. > > > The file is included once only, thus gcc is not invoked once per > > > directory. > > > > > > Another side-effect of this patch is that KCFLAGS are appended last > > > to the list of options. This allows us to better control the options. > > > Artem Bityutskiy noticed this. > > > > > > Signed-off-by: Sam Ravnborg > > > Cc: Artem Bityutskiy > > > > Hi, > > > > what happened to this patch? I was fixing the real issue I am > > encountering and I thought it'd be taken instead of my original patch. > We decided to move this to next merge release because is was not added > to kbuild thus not enough exposure in -next. > > I am planning to resend the serie at around -rc2 time. Hi Sam, what happened to this patch-set? At least KCFLAGS patches I am waiting for are still not merged. -- Best Regards, Artem Bityutskiy signature.asc Description: This is a digitally signed message part
Re: [PATCH 7/8] kbuild: move W=... stuff to Kbuild.arch
On Wed, 2012-06-06 at 17:35 +0200, Sam Ravnborg wrote: On Wed, Jun 06, 2012 at 01:18:47PM +0300, Artem Bityutskiy wrote: On Sat, 2012-05-05 at 10:18 +0200, Sam Ravnborg wrote: Prevent that we eveluate cc-option multiple times for the same option by moving the definitions to Kbuild.arch. The file is included once only, thus gcc is not invoked once per directory. Another side-effect of this patch is that KCFLAGS are appended last to the list of options. This allows us to better control the options. Artem Bityutskiy dedeki...@gmail.com noticed this. Signed-off-by: Sam Ravnborg s...@ravnborg.org Cc: Artem Bityutskiy dedeki...@gmail.com Hi, what happened to this patch? I was fixing the real issue I am encountering and I thought it'd be taken instead of my original patch. We decided to move this to next merge release because is was not added to kbuild thus not enough exposure in -next. I am planning to resend the serie at around -rc2 time. Hi Sam, what happened to this patch-set? At least KCFLAGS patches I am waiting for are still not merged. -- Best Regards, Artem Bityutskiy signature.asc Description: This is a digitally signed message part
Re: [PATCH 7/8] kbuild: move W=... stuff to Kbuild.arch
On Wed, Aug 15, 2012 at 12:41:23PM +0300, Artem Bityutskiy wrote: On Wed, 2012-06-06 at 17:35 +0200, Sam Ravnborg wrote: On Wed, Jun 06, 2012 at 01:18:47PM +0300, Artem Bityutskiy wrote: On Sat, 2012-05-05 at 10:18 +0200, Sam Ravnborg wrote: Prevent that we eveluate cc-option multiple times for the same option by moving the definitions to Kbuild.arch. The file is included once only, thus gcc is not invoked once per directory. Another side-effect of this patch is that KCFLAGS are appended last to the list of options. This allows us to better control the options. Artem Bityutskiy dedeki...@gmail.com noticed this. Signed-off-by: Sam Ravnborg s...@ravnborg.org Cc: Artem Bityutskiy dedeki...@gmail.com Hi, what happened to this patch? I was fixing the real issue I am encountering and I thought it'd be taken instead of my original patch. We decided to move this to next merge release because is was not added to kbuild thus not enough exposure in -next. I am planning to resend the serie at around -rc2 time. Hi Sam, what happened to this patch-set? At least KCFLAGS patches I am waiting for are still not merged. Vacation and then I have not yet gotten back to them. Will do soon - thanks for the reminder! Sam -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/