Re: [PATCH] perf: Let O= makes handle relative paths
Em Thu, Aug 16, 2012 at 12:07:54PM -0400, Steven Rostedt escreveu: > Acked-by: Steven Rostedt > > Arnaldo, can you pick this patch? Keep both the Signed-off-bys that > Boris placed. ack, done, - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Wed, 2012-08-15 at 18:39 +0200, Borislav Petkov wrote: > diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include > index bde8521d56bb..96ce80a3743b 100644 > --- a/tools/scripts/Makefile.include > +++ b/tools/scripts/Makefile.include > @@ -1,6 +1,8 @@ > ifeq ("$(origin O)", "command line") > - OUTPUT := $(O)/ > - COMMAND_O := O=$(O) > + dummy := $(if $(shell test -d $(O) || echo $(O)),$(error O=$(O) does > not exist),) > + ABSOLUTE_O := $(shell cd $(O) ; pwd) > + OUTPUT := $(ABSOLUTE_O)/ > + COMMAND_O := O=$(ABSOLUTE_O) > endif > Acked-by: Steven Rostedt Arnaldo, can you pick this patch? Keep both the Signed-off-bys that Boris placed. Thanks! -- Steve -- 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] perf: Let O= makes handle relative paths
On Wed, 2012-08-15 at 18:39 +0200, Borislav Petkov wrote: diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521d56bb..96ce80a3743b 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,8 @@ ifeq ($(origin O), command line) - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + dummy := $(if $(shell test -d $(O) || echo $(O)),$(error O=$(O) does not exist),) + ABSOLUTE_O := $(shell cd $(O) ; pwd) + OUTPUT := $(ABSOLUTE_O)/ + COMMAND_O := O=$(ABSOLUTE_O) endif Acked-by: Steven Rostedt rost...@goodmis.org Arnaldo, can you pick this patch? Keep both the Signed-off-bys that Boris placed. Thanks! -- Steve -- 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] perf: Let O= makes handle relative paths
Em Thu, Aug 16, 2012 at 12:07:54PM -0400, Steven Rostedt escreveu: Acked-by: Steven Rostedt rost...@goodmis.org Arnaldo, can you pick this patch? Keep both the Signed-off-bys that Boris placed. ack, done, - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Wed, Aug 15, 2012 at 11:59:35AM -0300, Arnaldo Carvalho de Melo wrote: > Of course! Those are the ultimate kernel killers! Haha, this is what I'm answering next time I'm asked about what I do for a living :-). How about this? -- From: Steven Rostedt Date: Mon, 13 Aug 2012 10:23:02 -0400 Subject: [PATCH] perf: Let O= makes handle relative paths When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory "../../../nobackup/perf/" does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Boris: Teach Makefile to check whether the supplied O= directory exists and bail out if not. Reportedly, kernel dudes are idiots and need to be guarded so as not to shoot themselves in the foot when playing in the sandbox. Signed-off-by: Steven Rostedt Signed-off-by: Borislav Petkov --- tools/scripts/Makefile.include | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521d56bb..96ce80a3743b 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,8 @@ ifeq ("$(origin O)", "command line") - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + dummy := $(if $(shell test -d $(O) || echo $(O)),$(error O=$(O) does not exist),) + ABSOLUTE_O := $(shell cd $(O) ; pwd) + OUTPUT := $(ABSOLUTE_O)/ + COMMAND_O := O=$(ABSOLUTE_O) endif ifneq ($(OUTPUT),) -- 1.7.11.rc1 -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
Em Wed, Aug 15, 2012 at 04:42:55PM +0200, Borislav Petkov escreveu: > On Wed, Aug 15, 2012 at 04:38:05PM +0200, Peter Zijlstra wrote: > > Because you're an idiot and made a typo ;-) Happens to me all the time. > > I'm not an idiot - I'm perfection at its purest! :-) > > > The regular kernel build doesn't create non-existent O= targets either > > iirc. > > Ok, I see. The kernel needs to protect itself against the people who > wrote it :-). Of course! Those are the ultimate kernel killers! - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Wed, Aug 15, 2012 at 04:38:05PM +0200, Peter Zijlstra wrote: > Because you're an idiot and made a typo ;-) Happens to me all the time. I'm not an idiot - I'm perfection at its purest! :-) > The regular kernel build doesn't create non-existent O= targets either > iirc. Ok, I see. The kernel needs to protect itself against the people who wrote it :-). -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
On Wed, 2012-08-15 at 15:18 +0200, Borislav Petkov wrote: > On Wed, Aug 15, 2012 at 10:06:34AM -0300, Arnaldo Carvalho de Melo wrote: > > That was the case in the past, but IIRC PeterZ advocated not to and I > > agreed. > > Maybe you guys need to explain yourselves :) I mean, the dir is not > present so we're not overwriting anything. And since we say "O=..." on > the command line, it is actually expected that we really mean it... why > type it, otherwise? Because you're an idiot and made a typo ;-) Happens to me all the time. The regular kernel build doesn't create non-existent O= targets either iirc. -- 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] perf: Let O= makes handle relative paths
On Wed, Aug 15, 2012 at 10:06:34AM -0300, Arnaldo Carvalho de Melo wrote: > That was the case in the past, but IIRC PeterZ advocated not to and I > agreed. Maybe you guys need to explain yourselves :) I mean, the dir is not present so we're not overwriting anything. And since we say "O=..." on the command line, it is actually expected that we really mean it... why type it, otherwise? Hmmm. -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
Em Wed, Aug 15, 2012 at 12:27:22PM +0200, Borislav Petkov escreveu: > On Mon, Aug 13, 2012 at 03:02:49PM -0300, Arnaldo Carvalho de Melo wrote: > > [acme@sandy linux]$ rm -rf ../build/perf > > [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind > > O=/home/acme/git/build/perf install > > /bin/sh: line 0: cd: /home/acme/git/build/perf: No such file or directory > > make: Entering directory `/home/git/linux/tools/perf' > > GEN perf-archive > > GEN /home/git/linux/tools/perf/python/perf.so > > make[1]: Entering directory `/home/git/linux/tools/lib/traceevent' > > * new build flags or cross compiler > > CC /home/git/linux/tools/perf/perf.o > > I.e. it should stop if the O= provided directory is not there. > Why stop? Don't we want to make the directory instead and continue > building in there? That was the case in the past, but IIRC PeterZ advocated not to and I agreed. - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Mon, Aug 13, 2012 at 03:02:49PM -0300, Arnaldo Carvalho de Melo wrote: > [acme@sandy linux]$ rm -rf ../build/perf > [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind > O=/home/acme/git/build/perf install > /bin/sh: line 0: cd: /home/acme/git/build/perf: No such file or directory > make: Entering directory `/home/git/linux/tools/perf' > GEN perf-archive > GEN /home/git/linux/tools/perf/python/perf.so > make[1]: Entering directory `/home/git/linux/tools/lib/traceevent' > make[2]: warning: jobserver unavailable: using -j1. Add `+' to parent make > rule. > * new build flags or cross compiler > CC /home/git/linux/tools/perf/perf.o > CC /home/git/linux/tools/perf/builtin-annotate.o > CC /home/git/linux/tools/perf/builtin-bench.o > CC /home/git/linux/tools/perf/bench/sched-messaging.o > CC /home/git/linux/tools/perf/bench/sched-pipe.o > > I.e. it should stop if the O= provided directory is not there. Why stop? Don't we want to make the directory instead and continue building in there? -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
On Mon, Aug 13, 2012 at 03:02:49PM -0300, Arnaldo Carvalho de Melo wrote: [acme@sandy linux]$ rm -rf ../build/perf [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind O=/home/acme/git/build/perf install /bin/sh: line 0: cd: /home/acme/git/build/perf: No such file or directory make: Entering directory `/home/git/linux/tools/perf' GEN perf-archive GEN /home/git/linux/tools/perf/python/perf.so make[1]: Entering directory `/home/git/linux/tools/lib/traceevent' make[2]: warning: jobserver unavailable: using -j1. Add `+' to parent make rule. * new build flags or cross compiler CC /home/git/linux/tools/perf/perf.o CC /home/git/linux/tools/perf/builtin-annotate.o CC /home/git/linux/tools/perf/builtin-bench.o CC /home/git/linux/tools/perf/bench/sched-messaging.o CC /home/git/linux/tools/perf/bench/sched-pipe.o I.e. it should stop if the O= provided directory is not there. Why stop? Don't we want to make the directory instead and continue building in there? -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
Em Wed, Aug 15, 2012 at 12:27:22PM +0200, Borislav Petkov escreveu: On Mon, Aug 13, 2012 at 03:02:49PM -0300, Arnaldo Carvalho de Melo wrote: [acme@sandy linux]$ rm -rf ../build/perf [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind O=/home/acme/git/build/perf install /bin/sh: line 0: cd: /home/acme/git/build/perf: No such file or directory make: Entering directory `/home/git/linux/tools/perf' GEN perf-archive GEN /home/git/linux/tools/perf/python/perf.so make[1]: Entering directory `/home/git/linux/tools/lib/traceevent' * new build flags or cross compiler CC /home/git/linux/tools/perf/perf.o I.e. it should stop if the O= provided directory is not there. Why stop? Don't we want to make the directory instead and continue building in there? That was the case in the past, but IIRC PeterZ advocated not to and I agreed. - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Wed, Aug 15, 2012 at 10:06:34AM -0300, Arnaldo Carvalho de Melo wrote: That was the case in the past, but IIRC PeterZ advocated not to and I agreed. Maybe you guys need to explain yourselves :) I mean, the dir is not present so we're not overwriting anything. And since we say O=... on the command line, it is actually expected that we really mean it... why type it, otherwise? Hmmm. -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
On Wed, 2012-08-15 at 15:18 +0200, Borislav Petkov wrote: On Wed, Aug 15, 2012 at 10:06:34AM -0300, Arnaldo Carvalho de Melo wrote: That was the case in the past, but IIRC PeterZ advocated not to and I agreed. Maybe you guys need to explain yourselves :) I mean, the dir is not present so we're not overwriting anything. And since we say O=... on the command line, it is actually expected that we really mean it... why type it, otherwise? Because you're an idiot and made a typo ;-) Happens to me all the time. The regular kernel build doesn't create non-existent O= targets either iirc. -- 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] perf: Let O= makes handle relative paths
On Wed, Aug 15, 2012 at 04:38:05PM +0200, Peter Zijlstra wrote: Because you're an idiot and made a typo ;-) Happens to me all the time. I'm not an idiot - I'm perfection at its purest! :-) The regular kernel build doesn't create non-existent O= targets either iirc. Ok, I see. The kernel needs to protect itself against the people who wrote it :-). -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
Em Wed, Aug 15, 2012 at 04:42:55PM +0200, Borislav Petkov escreveu: On Wed, Aug 15, 2012 at 04:38:05PM +0200, Peter Zijlstra wrote: Because you're an idiot and made a typo ;-) Happens to me all the time. I'm not an idiot - I'm perfection at its purest! :-) The regular kernel build doesn't create non-existent O= targets either iirc. Ok, I see. The kernel needs to protect itself against the people who wrote it :-). Of course! Those are the ultimate kernel killers! - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Wed, Aug 15, 2012 at 11:59:35AM -0300, Arnaldo Carvalho de Melo wrote: Of course! Those are the ultimate kernel killers! Haha, this is what I'm answering next time I'm asked about what I do for a living :-). How about this? -- From: Steven Rostedt rost...@goodmis.org Date: Mon, 13 Aug 2012 10:23:02 -0400 Subject: [PATCH] perf: Let O= makes handle relative paths When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory ../../../nobackup/perf/ does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Boris: Teach Makefile to check whether the supplied O= directory exists and bail out if not. Reportedly, kernel dudes are idiots and need to be guarded so as not to shoot themselves in the foot when playing in the sandbox. Signed-off-by: Steven Rostedt rost...@goodmis.org Signed-off-by: Borislav Petkov borislav.pet...@amd.com --- tools/scripts/Makefile.include | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521d56bb..96ce80a3743b 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,8 @@ ifeq ($(origin O), command line) - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + dummy := $(if $(shell test -d $(O) || echo $(O)),$(error O=$(O) does not exist),) + ABSOLUTE_O := $(shell cd $(O) ; pwd) + OUTPUT := $(ABSOLUTE_O)/ + COMMAND_O := O=$(ABSOLUTE_O) endif ifneq ($(OUTPUT),) -- 1.7.11.rc1 -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
Em Mon, Aug 13, 2012 at 04:29:09PM +0200, Borislav Petkov escreveu: > On Mon, Aug 13, 2012 at 10:23:02AM -0400, Steven Rostedt wrote: > > On Mon, 2012-08-13 at 15:24 +0200, Borislav Petkov wrote: > > > > > So wouldn't it make sense to call this ABSOLUTE_O or so then? > > > > That's just because you're russian, and are promoting Vodka! ;-) Before: [acme@sandy linux]$ rm -rf ../build/perf [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind O=/home/acme/git/build/perf install /bin/sh: line 0: cd: /home/acme/git/build/perf/: No such file or directory make: Entering directory `/home/git/linux/tools/perf' ../scripts/Makefile.include:9: *** output directory "/home/acme/git/build/perf/" does not exist. Stop. make: Leaving directory `/home/git/linux/tools/perf' [acme@sandy linux]$ Now: [acme@sandy linux]$ rm -rf ../build/perf [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind O=/home/acme/git/build/perf install /bin/sh: line 0: cd: /home/acme/git/build/perf: No such file or directory make: Entering directory `/home/git/linux/tools/perf' GEN perf-archive GEN /home/git/linux/tools/perf/python/perf.so make[1]: Entering directory `/home/git/linux/tools/lib/traceevent' make[2]: warning: jobserver unavailable: using -j1. Add `+' to parent make rule. * new build flags or cross compiler CC /home/git/linux/tools/perf/perf.o CC /home/git/linux/tools/perf/builtin-annotate.o CC /home/git/linux/tools/perf/builtin-bench.o CC /home/git/linux/tools/perf/bench/sched-messaging.o CC /home/git/linux/tools/perf/bench/sched-pipe.o I.e. it should stop if the O= provided directory is not there. Can you fix this? Avoid vodka while doing it :-P - Arnaldo -- 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] perf: Let O= makes handle relative paths
On Mon, Aug 13, 2012 at 10:23:02AM -0400, Steven Rostedt wrote: > On Mon, 2012-08-13 at 15:24 +0200, Borislav Petkov wrote: > > > So wouldn't it make sense to call this ABSOLUTE_O or so then? > > That's just because you're russian, and are promoting Vodka! ;-) > > > Done... > > -- > > When I did a compile of perf using a relative path for the output > directory, the build failed when it tried to compile libtraceevent. This > is because it continues to use the same relative path when the new > working directory is in a different path. > > SUBDIR ../lib/traceevent/ > /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory > Makefile:74: *** output directory "../../../nobackup/perf/" does not exist. > Stop. > make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 > > > Make the path used an absolute path when building perf with O=. > > Signed-off-by: Steven Rostedt Acked-by: The Russian Vodka Promoter. 8-). Thanks and Na Zdorovia! -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
On Mon, 2012-08-13 at 15:24 +0200, Borislav Petkov wrote: > So wouldn't it make sense to call this ABSOLUTE_O or so then? That's just because you're russian, and are promoting Vodka! ;-) Done... -- When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory "../../../nobackup/perf/" does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Signed-off-by: Steven Rostedt diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521..e90e1b4 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,7 @@ ifeq ("$(origin O)", "command line") - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + ABSOLUTE_O := $(shell cd $(O) ; pwd) + OUTPUT := $(ABSOLUTE_O)/ + COMMAND_O := O=$(ABSOLUTE_O) endif ifneq ($(OUTPUT),) -- 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] perf: Let O= makes handle relative paths
On Mon, Aug 13, 2012 at 09:20:55AM -0400, Steven Rostedt wrote: > When I did a compile of perf using a relative path for the output > directory, the build failed when it tried to compile libtraceevent. This > is because it continues to use the same relative path when the new > working directory is in a different path. > > SUBDIR ../lib/traceevent/ > /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory > Makefile:74: *** output directory "../../../nobackup/perf/" does not exist. > Stop. > make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 > > > Make the path used an absolute path when building perf with O=. > > Signed-off-by: Steven Rostedt > > diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include > index bde8521..9e3e31e 100644 > --- a/tools/scripts/Makefile.include > +++ b/tools/scripts/Makefile.include > @@ -1,6 +1,7 @@ > ifeq ("$(origin O)", "command line") > - OUTPUT := $(O)/ > - COMMAND_O := O=$(O) > + RAW_O := $(shell cd $(O) ; pwd) > + OUTPUT := $(RAW_O)/ > + COMMAND_O := O=$(RAW_O) So wouldn't it make sense to call this ABSOLUTE_O or so then? Thanks. -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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/
[PATCH] perf: Let O= makes handle relative paths
When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory "../../../nobackup/perf/" does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Signed-off-by: Steven Rostedt diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521..9e3e31e 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,7 @@ ifeq ("$(origin O)", "command line") - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + RAW_O := $(shell cd $(O) ; pwd) + OUTPUT := $(RAW_O)/ + COMMAND_O := O=$(RAW_O) endif ifneq ($(OUTPUT),) -- 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/
[PATCH] perf: Let O= makes handle relative paths
When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory ../../../nobackup/perf/ does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Signed-off-by: Steven Rostedt rost...@goodmis.org diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521..9e3e31e 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,7 @@ ifeq ($(origin O), command line) - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + RAW_O := $(shell cd $(O) ; pwd) + OUTPUT := $(RAW_O)/ + COMMAND_O := O=$(RAW_O) endif ifneq ($(OUTPUT),) -- 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] perf: Let O= makes handle relative paths
On Mon, Aug 13, 2012 at 09:20:55AM -0400, Steven Rostedt wrote: When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory ../../../nobackup/perf/ does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Signed-off-by: Steven Rostedt rost...@goodmis.org diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521..9e3e31e 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,7 @@ ifeq ($(origin O), command line) - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + RAW_O := $(shell cd $(O) ; pwd) + OUTPUT := $(RAW_O)/ + COMMAND_O := O=$(RAW_O) So wouldn't it make sense to call this ABSOLUTE_O or so then? Thanks. -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
On Mon, 2012-08-13 at 15:24 +0200, Borislav Petkov wrote: So wouldn't it make sense to call this ABSOLUTE_O or so then? That's just because you're russian, and are promoting Vodka! ;-) Done... -- When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory ../../../nobackup/perf/ does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Signed-off-by: Steven Rostedt rost...@goodmis.org diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index bde8521..e90e1b4 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -1,6 +1,7 @@ ifeq ($(origin O), command line) - OUTPUT := $(O)/ - COMMAND_O := O=$(O) + ABSOLUTE_O := $(shell cd $(O) ; pwd) + OUTPUT := $(ABSOLUTE_O)/ + COMMAND_O := O=$(ABSOLUTE_O) endif ifneq ($(OUTPUT),) -- 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] perf: Let O= makes handle relative paths
On Mon, Aug 13, 2012 at 10:23:02AM -0400, Steven Rostedt wrote: On Mon, 2012-08-13 at 15:24 +0200, Borislav Petkov wrote: So wouldn't it make sense to call this ABSOLUTE_O or so then? That's just because you're russian, and are promoting Vodka! ;-) Done... -- When I did a compile of perf using a relative path for the output directory, the build failed when it tried to compile libtraceevent. This is because it continues to use the same relative path when the new working directory is in a different path. SUBDIR ../lib/traceevent/ /bin/sh: line 0: cd: ../../../nobackup/perf/: No such file or directory Makefile:74: *** output directory ../../../nobackup/perf/ does not exist. Stop. make: *** [../../../nobackup/perf///libtraceevent.a] Error 2 Make the path used an absolute path when building perf with O=. Signed-off-by: Steven Rostedt rost...@goodmis.org Acked-by: The Russian Vodka Promoter. 8-). Thanks and Na Zdorovia! -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- 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] perf: Let O= makes handle relative paths
Em Mon, Aug 13, 2012 at 04:29:09PM +0200, Borislav Petkov escreveu: On Mon, Aug 13, 2012 at 10:23:02AM -0400, Steven Rostedt wrote: On Mon, 2012-08-13 at 15:24 +0200, Borislav Petkov wrote: So wouldn't it make sense to call this ABSOLUTE_O or so then? That's just because you're russian, and are promoting Vodka! ;-) Before: [acme@sandy linux]$ rm -rf ../build/perf [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind O=/home/acme/git/build/perf install /bin/sh: line 0: cd: /home/acme/git/build/perf/: No such file or directory make: Entering directory `/home/git/linux/tools/perf' ../scripts/Makefile.include:9: *** output directory /home/acme/git/build/perf/ does not exist. Stop. make: Leaving directory `/home/git/linux/tools/perf' [acme@sandy linux]$ Now: [acme@sandy linux]$ rm -rf ../build/perf [acme@sandy linux]$ make -j8 -C tools/perf/ LIBUNWIND_DIR=/opt/libunwind O=/home/acme/git/build/perf install /bin/sh: line 0: cd: /home/acme/git/build/perf: No such file or directory make: Entering directory `/home/git/linux/tools/perf' GEN perf-archive GEN /home/git/linux/tools/perf/python/perf.so make[1]: Entering directory `/home/git/linux/tools/lib/traceevent' make[2]: warning: jobserver unavailable: using -j1. Add `+' to parent make rule. * new build flags or cross compiler CC /home/git/linux/tools/perf/perf.o CC /home/git/linux/tools/perf/builtin-annotate.o CC /home/git/linux/tools/perf/builtin-bench.o CC /home/git/linux/tools/perf/bench/sched-messaging.o CC /home/git/linux/tools/perf/bench/sched-pipe.o I.e. it should stop if the O= provided directory is not there. Can you fix this? Avoid vodka while doing it :-P - Arnaldo -- 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/