All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper
@ 2014-08-07 10:52 Roger Meier
  2014-08-07 12:40 ` Masahiro Yamada
  0 siblings, 1 reply; 14+ messages in thread
From: Roger Meier @ 2014-08-07 10:52 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
CC: Wolfgang Denk <wd@denx.de>
---
 Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 666d291..831e091 100644
--- a/Makefile
+++ b/Makefile
@@ -1221,7 +1221,7 @@ CLOBBER_FILES += u-boot* MLO* SPL System.map
 # Directories & files removed with 'make mrproper'
 MRPROPER_DIRS  += include/config include/generated spl tpl \
 		  .tmp_objdiff
-MRPROPER_FILES += .config .config.old include/autoconf.mk* include/config.h \
+MRPROPER_FILES += .config .config.old boards.cfg include/autoconf.mk* include/config.h \
 		  ctags etags TAGS cscope* GPATH GTAGS GRTAGS GSYMS
 
 # clean - Delete most, but leave enough to build external modules
-- 
1.7.10.4

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

* [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper
  2014-08-07 10:52 [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper Roger Meier
@ 2014-08-07 12:40 ` Masahiro Yamada
  2014-08-07 14:53   ` Roger Meier
  0 siblings, 1 reply; 14+ messages in thread
From: Masahiro Yamada @ 2014-08-07 12:40 UTC (permalink / raw)
  To: u-boot

Hi Roger,



On Thu,  7 Aug 2014 12:52:30 +0200
Roger Meier <roger@bufferoverflow.ch> wrote:

> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
> CC: Wolfgang Denk <wd@denx.de>
> ---
>  Makefile |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Makefile b/Makefile
> index 666d291..831e091 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1221,7 +1221,7 @@ CLOBBER_FILES += u-boot* MLO* SPL System.map
>  # Directories & files removed with 'make mrproper'
>  MRPROPER_DIRS  += include/config include/generated spl tpl \
>  		  .tmp_objdiff
> -MRPROPER_FILES += .config .config.old include/autoconf.mk* include/config.h \
> +MRPROPER_FILES += .config .config.old boards.cfg include/autoconf.mk* include/config.h \
>  		  ctags etags TAGS cscope* GPATH GTAGS GRTAGS GSYMS
>  
>  # clean - Delete most, but leave enough to build external modules
> -- 
> 1.7.10.4



NAK.

MAKEALL needs boards.cfg

Before building each board, MAKEALL runs "make distclean" and boards.cfg gets lost.

With your patch, MAKEALL does not work at all.



This is my log of "./MAKEALL"

Could not find boards.cfg
Generating boards.cfg ...  (jobs: 8)
1177/1177 [=======================================================>]
Building cmi_mpc5xx board...
   text	   data	    bss	    dec	    hex	filename
 106025	  70616	   2532	 179173	  2bbe5	./u-boot
awk: fatal: cannot open file `boards.cfg' for reading (No such file or directory)
Building PATI board...
make[2]: *** [include/autoconf.mk] Error 1
make[2]: *** [include/autoconf.mk.dep] Error 1
make[1]: *** [silentoldconfig] Error 1
size: './u-boot': No such file
/bin/sh: 1: ppc_8xx-gcc: not found
/bin/sh: 1: ppc_8xx-gcc: not found
make[2]: *** [include/autoconf.mk] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [include/autoconf.mk.dep] Error 1
'make -f /home/yamada/workspace/u-boot-org/scripts/Makefile.autoconf obj=include include/autoconf.mk include/autoconf.mk.dep' Failed
make[1]: *** [silentoldconfig] Error 1
make: *** No rule to make target `include/config/auto.conf', needed by `include/config/uboot.release'. 




Best Regards
Masahiro Yamada

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

* [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper
  2014-08-07 12:40 ` Masahiro Yamada
@ 2014-08-07 14:53   ` Roger Meier
  2014-08-08 12:48     ` Tom Rini
  2014-08-09  0:19     ` [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper Masahiro YAMADA
  0 siblings, 2 replies; 14+ messages in thread
From: Roger Meier @ 2014-08-07 14:53 UTC (permalink / raw)
  To: u-boot

Hi Masahiro
Quoting Masahiro Yamada <yamada.m@jp.panasonic.com>:

> Hi Roger,
>
>
>
> On Thu,  7 Aug 2014 12:52:30 +0200
> Roger Meier <roger@bufferoverflow.ch> wrote:
>
>> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
>> CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> CC: Wolfgang Denk <wd@denx.de>
>> ---
>>  Makefile |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Makefile b/Makefile
>> index 666d291..831e091 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -1221,7 +1221,7 @@ CLOBBER_FILES += u-boot* MLO* SPL System.map
>>  # Directories & files removed with 'make mrproper'
>>  MRPROPER_DIRS  += include/config include/generated spl tpl \
>>  		  .tmp_objdiff
>> -MRPROPER_FILES += .config .config.old include/autoconf.mk*  
>> include/config.h \
>> +MRPROPER_FILES += .config .config.old boards.cfg  
>> include/autoconf.mk* include/config.h \
>>  		  ctags etags TAGS cscope* GPATH GTAGS GRTAGS GSYMS
>>
>>  # clean - Delete most, but leave enough to build external modules
>> --
>> 1.7.10.4
>
>
>
> NAK.
>
> MAKEALL needs boards.cfg
>
> Before building each board, MAKEALL runs "make distclean" and  
> boards.cfg gets lost.
oops, did not recognize that. Thanks!

>
> With your patch, MAKEALL does not work at all.
How should we ensure that boards.cfg is up-to-date?
What about removing boards.cfg within "make distclean" and use
"make mrproper" within MAKEALL?

best!
-roger

>
>
>
> This is my log of "./MAKEALL"
>
> Could not find boards.cfg
> Generating boards.cfg ...  (jobs: 8)
> 1177/1177 [=======================================================>]
> Building cmi_mpc5xx board...
>    text	   data	    bss	    dec	    hex	filename
>  106025	  70616	   2532	 179173	  2bbe5	./u-boot
> awk: fatal: cannot open file `boards.cfg' for reading (No such file  
> or directory)
> Building PATI board...
> make[2]: *** [include/autoconf.mk] Error 1
> make[2]: *** [include/autoconf.mk.dep] Error 1
> make[1]: *** [silentoldconfig] Error 1
> size: './u-boot': No such file
> /bin/sh: 1: ppc_8xx-gcc: not found
> /bin/sh: 1: ppc_8xx-gcc: not found
> make[2]: *** [include/autoconf.mk] Error 1
> make[2]: *** Waiting for unfinished jobs....
> make[2]: *** [include/autoconf.mk.dep] Error 1
> 'make -f /home/yamada/workspace/u-boot-org/scripts/Makefile.autoconf  
> obj=include include/autoconf.mk include/autoconf.mk.dep' Failed
> make[1]: *** [silentoldconfig] Error 1
> make: *** No rule to make target `include/config/auto.conf', needed  
> by `include/config/uboot.release'.
>
>
>
>
> Best Regards
> Masahiro Yamada

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

* [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper
  2014-08-07 14:53   ` Roger Meier
@ 2014-08-08 12:48     ` Tom Rini
  2014-08-10 12:37       ` [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean Roger Meier
  2014-08-09  0:19     ` [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper Masahiro YAMADA
  1 sibling, 1 reply; 14+ messages in thread
From: Tom Rini @ 2014-08-08 12:48 UTC (permalink / raw)
  To: u-boot

On Thu, Aug 07, 2014 at 02:53:47PM +0000, Roger Meier wrote:
> Hi Masahiro
> Quoting Masahiro Yamada <yamada.m@jp.panasonic.com>:
> 
> >Hi Roger,
> >
> >
> >
> >On Thu,  7 Aug 2014 12:52:30 +0200
> >Roger Meier <roger@bufferoverflow.ch> wrote:
> >
> >>Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> >>CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
> >>CC: Wolfgang Denk <wd@denx.de>
> >>---
> >> Makefile |    2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >>diff --git a/Makefile b/Makefile
> >>index 666d291..831e091 100644
> >>--- a/Makefile
> >>+++ b/Makefile
> >>@@ -1221,7 +1221,7 @@ CLOBBER_FILES += u-boot* MLO* SPL System.map
> >> # Directories & files removed with 'make mrproper'
> >> MRPROPER_DIRS  += include/config include/generated spl tpl \
> >> 		  .tmp_objdiff
> >>-MRPROPER_FILES += .config .config.old include/autoconf.mk*
> >>include/config.h \
> >>+MRPROPER_FILES += .config .config.old boards.cfg
> >>include/autoconf.mk* include/config.h \
> >> 		  ctags etags TAGS cscope* GPATH GTAGS GRTAGS GSYMS
> >>
> >> # clean - Delete most, but leave enough to build external modules
> >>--
> >>1.7.10.4
> >
> >
> >
> >NAK.
> >
> >MAKEALL needs boards.cfg
> >
> >Before building each board, MAKEALL runs "make distclean" and
> >boards.cfg gets lost.
> oops, did not recognize that. Thanks!
> 
> >
> >With your patch, MAKEALL does not work at all.
> How should we ensure that boards.cfg is up-to-date?
> What about removing boards.cfg within "make distclean" and use
> "make mrproper" within MAKEALL?

Sounds good to me.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140808/6f276e3f/attachment.pgp>

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

* [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper
  2014-08-07 14:53   ` Roger Meier
  2014-08-08 12:48     ` Tom Rini
@ 2014-08-09  0:19     ` Masahiro YAMADA
  1 sibling, 0 replies; 14+ messages in thread
From: Masahiro YAMADA @ 2014-08-09  0:19 UTC (permalink / raw)
  To: u-boot

Hi Roger,

> How should we ensure that boards.cfg is up-to-date?
> What about removing boards.cfg within "make distclean" and use
> "make mrproper" within MAKEALL?

Sounds good to me, too.
Thanks!


Best Regards
Masahiro Yamada

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

* [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean
  2014-08-08 12:48     ` Tom Rini
@ 2014-08-10 12:37       ` Roger Meier
  2014-08-11 22:52         ` Masahiro YAMADA
                           ` (2 more replies)
  0 siblings, 3 replies; 14+ messages in thread
From: Roger Meier @ 2014-08-10 12:37 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
CC: Tom Rini <trini@ti.com>
CC: Wolfgang Denk <wd@denx.de>

---
Changes for v2:
   - remove boards.cfg within make distclean instead of mrproper
   - use mrproper instead of distclean within MAKEALL
---
 MAKEALL  |    2 +-
 Makefile |    3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/MAKEALL b/MAKEALL
index 929fe88..392ea8d 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -658,7 +658,7 @@ build_target() {
 		MAKE="${MAKE} O=${output_dir}"
 	fi
 
-	${MAKE} distclean >/dev/null
+	${MAKE} mrproper >/dev/null
 
 	echo "Building ${target} board..."
 	${MAKE} -s ${target}_defconfig >/dev/null
diff --git a/Makefile b/Makefile
index 666d291..067a301 100644
--- a/Makefile
+++ b/Makefile
@@ -1285,7 +1285,8 @@ distclean: mrproper
 		-o -name '*.bak' -o -name '#*#' -o -name '.*.orig' \
 		-o -name '.*.rej' -o -name '*%' -o -name 'core' \
 		-o -name '*.pyc' \) \
-		-type f -print | xargs rm -f
+		-type f -print | xargs rm -f \
+	rm -f boards.cfg
 
 backup:
 	F=`basename $(srctree)` ; cd .. ; \
-- 
1.7.10.4

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

* [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean
  2014-08-10 12:37       ` [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean Roger Meier
@ 2014-08-11 22:52         ` Masahiro YAMADA
  2014-08-12 14:37           ` Simon Glass
  2014-08-18  4:23         ` Masahiro Yamada
  2014-08-19 23:13         ` [U-Boot] [PATCH v3] " Roger Meier
  2 siblings, 1 reply; 14+ messages in thread
From: Masahiro YAMADA @ 2014-08-11 22:52 UTC (permalink / raw)
  To: u-boot

Hi Roger,

(ccing Simon)


2014-08-10 21:37 GMT+09:00 Roger Meier <roger@bufferoverflow.ch>:
> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
> CC: Tom Rini <trini@ti.com>
> CC: Wolfgang Denk <wd@denx.de>
>
> ---
> Changes for v2:
>    - remove boards.cfg within make distclean instead of mrproper
>    - use mrproper instead of distclean within MAKEALL
> ---
>  MAKEALL  |    2 +-
>  Makefile |    3 ++-
>  2 files changed, 3 insertions(+), 2 deletions(-)


I think it is OK as for MAKEALL.
But I am not sure it is safe for buildman, which also uses boards.cfg.


I just did grep under tools/buildman directory and
found some lines referring 'distclean'.

./builder.py:179:                        distclean - can be called to
clean source
./builder.py:275:                    result = self.Make(commit, brd,
'distclean', work_dir,
./builder.py:276:                            'distclean', *args, env=env)
./builder.py:700:            stage: Stage that we are at (distclean,
config, build)


Unfortunately, I am not an expert of buildman.
I suspect this patch has an impact on it, but I am not sure.
I think Simon can help you.



-- 
Best Regards
Masahiro Yamada

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

* [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean
  2014-08-11 22:52         ` Masahiro YAMADA
@ 2014-08-12 14:37           ` Simon Glass
  0 siblings, 0 replies; 14+ messages in thread
From: Simon Glass @ 2014-08-12 14:37 UTC (permalink / raw)
  To: u-boot

Hi,

On 11 August 2014 16:52, Masahiro YAMADA <yamada.m@jp.panasonic.com> wrote:
> Hi Roger,
>
> (ccing Simon)
>
>
> 2014-08-10 21:37 GMT+09:00 Roger Meier <roger@bufferoverflow.ch>:
>> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
>> CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
>> CC: Tom Rini <trini@ti.com>
>> CC: Wolfgang Denk <wd@denx.de>
>>
>> ---
>> Changes for v2:
>>    - remove boards.cfg within make distclean instead of mrproper
>>    - use mrproper instead of distclean within MAKEALL
>> ---
>>  MAKEALL  |    2 +-
>>  Makefile |    3 ++-
>>  2 files changed, 3 insertions(+), 2 deletions(-)
>
>
> I think it is OK as for MAKEALL.
> But I am not sure it is safe for buildman, which also uses boards.cfg.
>
>
> I just did grep under tools/buildman directory and
> found some lines referring 'distclean'.
>
> ./builder.py:179:                        distclean - can be called to
> clean source
> ./builder.py:275:                    result = self.Make(commit, brd,
> 'distclean', work_dir,
> ./builder.py:276:                            'distclean', *args, env=env)
> ./builder.py:700:            stage: Stage that we are at (distclean,
> config, build)
>
>
> Unfortunately, I am not an expert of buildman.
> I suspect this patch has an impact on it, but I am not sure.
> I think Simon can help you.
>

Yes I think buildman should be updated to change all 'disclean' to 'mrproper'.

Regards,
Simon

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

* [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean
  2014-08-10 12:37       ` [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean Roger Meier
  2014-08-11 22:52         ` Masahiro YAMADA
@ 2014-08-18  4:23         ` Masahiro Yamada
  2014-08-19 23:13         ` [U-Boot] [PATCH v3] " Roger Meier
  2 siblings, 0 replies; 14+ messages in thread
From: Masahiro Yamada @ 2014-08-18  4:23 UTC (permalink / raw)
  To: u-boot

Hi Roger,


On Sun, 10 Aug 2014 14:37:40 +0200
Roger Meier <roger@bufferoverflow.ch> wrote:

> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
> CC: Tom Rini <trini@ti.com>
> CC: Wolfgang Denk <wd@denx.de>
> 
> ---
> Changes for v2:
>    - remove boards.cfg within make distclean instead of mrproper
>    - use mrproper instead of distclean within MAKEALL
> ---
>  MAKEALL  |    2 +-
>  Makefile |    3 ++-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/MAKEALL b/MAKEALL
> index 929fe88..392ea8d 100755
> --- a/MAKEALL
> +++ b/MAKEALL
> @@ -658,7 +658,7 @@ build_target() {
>  		MAKE="${MAKE} O=${output_dir}"
>  	fi
>  
> -	${MAKE} distclean >/dev/null
> +	${MAKE} mrproper >/dev/null
>  
>  	echo "Building ${target} board..."
>  	${MAKE} -s ${target}_defconfig >/dev/null
> diff --git a/Makefile b/Makefile
> index 666d291..067a301 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1285,7 +1285,8 @@ distclean: mrproper
>  		-o -name '*.bak' -o -name '#*#' -o -name '.*.orig' \
>  		-o -name '.*.rej' -o -name '*%' -o -name 'core' \
>  		-o -name '*.pyc' \) \
> -		-type f -print | xargs rm -f
> +		-type f -print | xargs rm -f \
> +	rm -f boards.cfg
>  
>  backup:
>  	F=`basename $(srctree)` ; cd .. ; \


The backslash after "xargs rm -f" is unnecessary, I think.


Should be like follows?


		-type f -print | xargs rm -f
	@rm -f boards.cfg





Best Regards
Masahiro Yamada

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

* [U-Boot] [PATCH v3] Makefile: remove generated boards.cfg within make distclean
  2014-08-10 12:37       ` [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean Roger Meier
  2014-08-11 22:52         ` Masahiro YAMADA
  2014-08-18  4:23         ` Masahiro Yamada
@ 2014-08-19 23:13         ` Roger Meier
  2014-08-20  1:06           ` Masahiro Yamada
  2014-08-20 20:10           ` [U-Boot] [PATCH v4] " Roger Meier
  2 siblings, 2 replies; 14+ messages in thread
From: Roger Meier @ 2014-08-19 23:13 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
CC: Tom Rini <trini@ti.com>
CC: Wolfgang Denk <wd@denx.de>

---
Changes for v2:
   - remove boards.cfg within make distclean instead of mrproper
   - use mrproper instead of distclean within MAKEALL
Changes for v3:
   - use @rm instead of rm as suggested by Masahiro Yamada
---
 MAKEALL  |    2 +-
 Makefile |    3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/MAKEALL b/MAKEALL
index 929fe88..392ea8d 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -658,7 +658,7 @@ build_target() {
 		MAKE="${MAKE} O=${output_dir}"
 	fi
 
-	${MAKE} distclean >/dev/null
+	${MAKE} mrproper >/dev/null
 
 	echo "Building ${target} board..."
 	${MAKE} -s ${target}_defconfig >/dev/null
diff --git a/Makefile b/Makefile
index 666d291..6679408 100644
--- a/Makefile
+++ b/Makefile
@@ -1285,7 +1285,8 @@ distclean: mrproper
 		-o -name '*.bak' -o -name '#*#' -o -name '.*.orig' \
 		-o -name '.*.rej' -o -name '*%' -o -name 'core' \
 		-o -name '*.pyc' \) \
-		-type f -print | xargs rm -f
+		-type f -print | xargs rm -f \
+	@rm -f boards.cfg
 
 backup:
 	F=`basename $(srctree)` ; cd .. ; \
-- 
1.7.10.4

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

* [U-Boot] [PATCH v3] Makefile: remove generated boards.cfg within make distclean
  2014-08-19 23:13         ` [U-Boot] [PATCH v3] " Roger Meier
@ 2014-08-20  1:06           ` Masahiro Yamada
  2014-08-20 20:10           ` [U-Boot] [PATCH v4] " Roger Meier
  1 sibling, 0 replies; 14+ messages in thread
From: Masahiro Yamada @ 2014-08-20  1:06 UTC (permalink / raw)
  To: u-boot

Hi Roger,


On Wed, 20 Aug 2014 01:13:50 +0200
Roger Meier <roger@bufferoverflow.ch> wrote:

> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> CC: Masahiro Yamada <yamada.m@jp.panasonic.com>
> CC: Tom Rini <trini@ti.com>
> CC: Wolfgang Denk <wd@denx.de>


Most people use 'Cc:' instead of 'CC:'.


$ git log v2013.01..v2014.07 | grep 'Cc:' | wc
   2007    8047   87385
$ git log v2013.01..v2014.07 | grep 'CC:' | wc
    147     598    6443




> diff --git a/Makefile b/Makefile
> index 666d291..6679408 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1285,7 +1285,8 @@ distclean: mrproper
>  		-o -name '*.bak' -o -name '#*#' -o -name '.*.orig' \
>  		-o -name '.*.rej' -o -name '*%' -o -name 'core' \
>  		-o -name '*.pyc' \) \
> -		-type f -print | xargs rm -f
> +		-type f -print | xargs rm -f \
> +	@rm -f boards.cfg
>  
>  backup:
>  	F=`basename $(srctree)` ; cd .. ; \


No.  You missed one of my comments against v2.

Please remove the backslash after "xargs rm -f".


Here, you are doing

@find ... | xargs rm -f @rm -f boards.cfg


that is, you are trying to delete
'@rm' as well as 'boards.cfg' if they exist.


Besides, you also need to fix buildman
as Simon suggested.


Please also cc Simon when you post the next version.




Best Regards
Masahiro Yamada

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

* [U-Boot] [PATCH v4] Makefile: remove generated boards.cfg within make distclean
  2014-08-19 23:13         ` [U-Boot] [PATCH v3] " Roger Meier
  2014-08-20  1:06           ` Masahiro Yamada
@ 2014-08-20 20:10           ` Roger Meier
  2014-08-22  6:30             ` Masahiro Yamada
  2014-08-23 12:43             ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 2 replies; 14+ messages in thread
From: Roger Meier @ 2014-08-20 20:10 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
Cc: Masahiro Yamada <yamada.m@jp.panasonic.com>
Cc: Tom Rini <trini@ti.com>
Cc: Wolfgang Denk <wd@denx.de>
Cc: Simon Glass <sjg@chromium.org>

---
Changes for v2:
   - remove boards.cfg within make distclean instead of mrproper
   - use mrproper instead of distclean within MAKEALL
Changes for v3:
   - use @rm instead of rm as suggested by Masahiro Yamada
Changes for v4:
   - remove the backslash after "xargs rm -f"
   - use mrproper instead of distclean within buildman
---
 MAKEALL                         |    2 +-
 Makefile                        |    1 +
 tools/buildman/builder.py       |    2 +-
 tools/buildman/builderthread.py |    6 +++---
 4 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/MAKEALL b/MAKEALL
index 929fe88..392ea8d 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -658,7 +658,7 @@ build_target() {
 		MAKE="${MAKE} O=${output_dir}"
 	fi
 
-	${MAKE} distclean >/dev/null
+	${MAKE} mrproper >/dev/null
 
 	echo "Building ${target} board..."
 	${MAKE} -s ${target}_defconfig >/dev/null
diff --git a/Makefile b/Makefile
index b5d5e01..c3d7afd 100644
--- a/Makefile
+++ b/Makefile
@@ -1287,6 +1287,7 @@ distclean: mrproper
 		-o -name '.*.rej' -o -name '*%' -o -name 'core' \
 		-o -name '*.pyc' \) \
 		-type f -print | xargs rm -f
+	@rm -f boards.cfg
 
 backup:
 	F=`basename $(srctree)` ; cd .. ; \
diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index d5b8454..a555bd8 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -308,7 +308,7 @@ class Builder:
         Args:
             commit: Commit object that is being built
             brd: Board object that is being built
-            stage: Stage that we are at (distclean, config, build)
+            stage: Stage that we are at (mrproper, config, build)
             cwd: Directory where make should be run
             args: Arguments to pass to make
             kwargs: Arguments to pass to command.RunPipe()
diff --git a/tools/buildman/builderthread.py b/tools/buildman/builderthread.py
index 32297e2..8214662 100644
--- a/tools/buildman/builderthread.py
+++ b/tools/buildman/builderthread.py
@@ -91,7 +91,7 @@ class BuilderThread(threading.Thread):
             commit: Commit object that is being built
             brd: Board object that is being built
             stage: Stage of the build. Valid stages are:
-                        distclean - can be called to clean source
+                        mrproper - can be called to clean source
                         config - called to configure for a board
                         build - the main make invocation - it does the build
             args: A list of arguments to pass to 'make'
@@ -200,8 +200,8 @@ class BuilderThread(threading.Thread):
 
                 # If we need to reconfigure, do that now
                 if do_config:
-                    result = self.Make(commit, brd, 'distclean', cwd,
-                            'distclean', *args, env=env)
+                    result = self.Make(commit, brd, 'mrproper', cwd,
+                            'mrproper', *args, env=env)
                     result = self.Make(commit, brd, 'config', cwd,
                             *(args + config_args), env=env)
                     config_out = result.combined
-- 
1.7.10.4

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

* [U-Boot] [PATCH v4] Makefile: remove generated boards.cfg within make distclean
  2014-08-20 20:10           ` [U-Boot] [PATCH v4] " Roger Meier
@ 2014-08-22  6:30             ` Masahiro Yamada
  2014-08-23 12:43             ` [U-Boot] [U-Boot, " Tom Rini
  1 sibling, 0 replies; 14+ messages in thread
From: Masahiro Yamada @ 2014-08-22  6:30 UTC (permalink / raw)
  To: u-boot


On Wed, 20 Aug 2014 22:10:29 +0200
Roger Meier <roger@bufferoverflow.ch> wrote:

> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> Cc: Masahiro Yamada <yamada.m@jp.panasonic.com>
> Cc: Tom Rini <trini@ti.com>
> Cc: Wolfgang Denk <wd@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> 


Reviewed-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Tested-by: Masahiro Yamada <yamada.m@jp.panasonic.com>

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

* [U-Boot] [U-Boot, v4] Makefile: remove generated boards.cfg within make distclean
  2014-08-20 20:10           ` [U-Boot] [PATCH v4] " Roger Meier
  2014-08-22  6:30             ` Masahiro Yamada
@ 2014-08-23 12:43             ` Tom Rini
  1 sibling, 0 replies; 14+ messages in thread
From: Tom Rini @ 2014-08-23 12:43 UTC (permalink / raw)
  To: u-boot

On Wed, Aug 20, 2014 at 10:10:29PM +0200, Roger Meier wrote:

> Signed-off-by: Roger Meier <roger@bufferoverflow.ch>
> Cc: Masahiro Yamada <yamada.m@jp.panasonic.com>
> Cc: Tom Rini <trini@ti.com>
> Cc: Wolfgang Denk <wd@denx.de>
> Cc: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140823/cadca205/attachment.pgp>

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

end of thread, other threads:[~2014-08-23 12:43 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-07 10:52 [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper Roger Meier
2014-08-07 12:40 ` Masahiro Yamada
2014-08-07 14:53   ` Roger Meier
2014-08-08 12:48     ` Tom Rini
2014-08-10 12:37       ` [U-Boot] [PATCH v2] Makefile: remove generated boards.cfg within make distclean Roger Meier
2014-08-11 22:52         ` Masahiro YAMADA
2014-08-12 14:37           ` Simon Glass
2014-08-18  4:23         ` Masahiro Yamada
2014-08-19 23:13         ` [U-Boot] [PATCH v3] " Roger Meier
2014-08-20  1:06           ` Masahiro Yamada
2014-08-20 20:10           ` [U-Boot] [PATCH v4] " Roger Meier
2014-08-22  6:30             ` Masahiro Yamada
2014-08-23 12:43             ` [U-Boot] [U-Boot, " Tom Rini
2014-08-09  0:19     ` [U-Boot] [PATCH] Makefile: remove generated boards.cfg within make mrproper Masahiro YAMADA

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.