All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] Makefile: Add CROSS_COMPILE for Makefile
@ 2015-09-29  1:58 Yuanbin Zhou
  2015-09-29 12:15 ` John Kacur
  0 siblings, 1 reply; 3+ messages in thread
From: Yuanbin Zhou @ 2015-09-29  1:58 UTC (permalink / raw
  To: williams, jkacur; +Cc: linux-rt-users

Signed-off-by: Yuanbin Zhou <hduffddybz@gmail.com>
---
 Makefile | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 87946fb..35a7d5a 100644
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,15 @@
 VERSION = 0.94
-CC=$(CROSS_COMPILE)gcc
-AR=$(CROSS_COMPILE)ar
+
+CROSS_COMPILE ?= $(CONFIG_CROSS_COMPILE:"%"=%)
+AS = $(CROSS_COMPILE)as
+LD = $(CROSS_COMPILE)ld
+CC = $(CROSS_COMPILE)gcc
+CPP = $(CC) -E
+AR = $(CROSS_COMPILE)ar
+NM = $(CROSS_COMPILE)nm
+STRIP = $(CROSS_COMPILE)strip
+OBJCOPY = $(CROSS_COMPILE)objcopy
+OBJDUMP = $(CROSS_COMPILE)objdump
 
 OBJDIR = bld
 
-- 
2.1.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] Makefile: Add CROSS_COMPILE for Makefile
  2015-09-29  1:58 [PATCH v2] Makefile: Add CROSS_COMPILE for Makefile Yuanbin Zhou
@ 2015-09-29 12:15 ` John Kacur
  2015-10-05 18:25   ` John Kacur
  0 siblings, 1 reply; 3+ messages in thread
From: John Kacur @ 2015-09-29 12:15 UTC (permalink / raw
  To: Yuanbin Zhou; +Cc: williams, linux-rt-users



On Mon, 28 Sep 2015, Yuanbin Zhou wrote:

> Signed-off-by: Yuanbin Zhou <hduffddybz@gmail.com>
> ---
>  Makefile | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 87946fb..35a7d5a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1,6 +1,15 @@
>  VERSION = 0.94
> -CC=$(CROSS_COMPILE)gcc
> -AR=$(CROSS_COMPILE)ar
> +
> +CROSS_COMPILE ?= $(CONFIG_CROSS_COMPILE:"%"=%)
> +AS = $(CROSS_COMPILE)as
> +LD = $(CROSS_COMPILE)ld
> +CC = $(CROSS_COMPILE)gcc
> +CPP = $(CC) -E
> +AR = $(CROSS_COMPILE)ar
> +NM = $(CROSS_COMPILE)nm
> +STRIP = $(CROSS_COMPILE)strip
> +OBJCOPY = $(CROSS_COMPILE)objcopy
> +OBJDUMP = $(CROSS_COMPILE)objdump
>  
>  OBJDIR = bld
>  
> -- 
> 2.1.4
> 
> 
> --

Signed-off-by: John Kacur <jkacur@redhat.com>

Thanks!

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] Makefile: Add CROSS_COMPILE for Makefile
  2015-09-29 12:15 ` John Kacur
@ 2015-10-05 18:25   ` John Kacur
  0 siblings, 0 replies; 3+ messages in thread
From: John Kacur @ 2015-10-05 18:25 UTC (permalink / raw
  To: Yuanbin Zhou; +Cc: williams, linux-rt-users

I'm really sorry to do this, after signing-off on it, but I have taken a 
closer look at this patch, and I have to reverse it.

You've added a lot of cross compiling options for things we don't use like 
AS. CONFIG_CROSS_COMPILE looks like something out of the kernel.

Thanks for your efforts anyway, but in the future if you want to send me 
patches you are going to have to include examples, of what is not working 
for you (with output), and then output demonstrating how your patch 
corrects the problem. Thanks

On Tue, 29 Sep 2015, John Kacur wrote:

> 
> 
> On Mon, 28 Sep 2015, Yuanbin Zhou wrote:
> 
> > Signed-off-by: Yuanbin Zhou <hduffddybz@gmail.com>
> > ---
> >  Makefile | 13 +++++++++++--
> >  1 file changed, 11 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Makefile b/Makefile
> > index 87946fb..35a7d5a 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -1,6 +1,15 @@
> >  VERSION = 0.94
> > -CC=$(CROSS_COMPILE)gcc
> > -AR=$(CROSS_COMPILE)ar
> > +
> > +CROSS_COMPILE ?= $(CONFIG_CROSS_COMPILE:"%"=%)
> > +AS = $(CROSS_COMPILE)as
> > +LD = $(CROSS_COMPILE)ld
> > +CC = $(CROSS_COMPILE)gcc
> > +CPP = $(CC) -E
> > +AR = $(CROSS_COMPILE)ar
> > +NM = $(CROSS_COMPILE)nm
> > +STRIP = $(CROSS_COMPILE)strip
> > +OBJCOPY = $(CROSS_COMPILE)objcopy
> > +OBJDUMP = $(CROSS_COMPILE)objdump
> >  
> >  OBJDIR = bld
> >  
> > -- 
> > 2.1.4
> > 
> > 
> > --
> 
> Signed-off-by: John Kacur <jkacur@redhat.com>
> 
> Thanks!
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-10-05 18:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-29  1:58 [PATCH v2] Makefile: Add CROSS_COMPILE for Makefile Yuanbin Zhou
2015-09-29 12:15 ` John Kacur
2015-10-05 18:25   ` John Kacur

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.