[U-Boot] Silence "make clean"

[U-Boot] Silence "make clean"

[Wolfgang Denk]

Dear Marek,

your commit 30ff8918 ("kerneldoc: Implant DocBook from Linux kernel")
causes "make clean" (and all other targets that use this implicitly)
to spit out these messages:

	make[1]: Entering directory `.../doc/DocBook'
	make[1]: Leaving directory `.../doc/DocBook'

We didn't have this before, and we shouldn't have this.

Can you please fix this regression?  Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I have a theory that it's impossible to prove anything, but  I  can't
prove it.

[U-Boot] Silence "make clean"

[Marek Vasut]

Dear Wolfgang Denk,

> Dear Marek,
> 
> your commit 30ff8918 ("kerneldoc: Implant DocBook from Linux kernel")
> causes "make clean" (and all other targets that use this implicitly)
> to spit out these messages:
> 
> 	make[1]: Entering directory `.../doc/DocBook'
> 	make[1]: Leaving directory `.../doc/DocBook'
> 
> We didn't have this before, and we shouldn't have this.
> 
> Can you please fix this regression?  Thanks.

Is this not fixed in latest sources? I believe Tom fixed it.

> Best regards,
> 
> Wolfgang Denk

Best regards,
Marek Vasut

[U-Boot] Silence "make clean"

[Wolfgang Denk]

Dear Marek Vasut,

In message <201211131453.18366.marex@denx.de> you wrote:
> 
> > 	make[1]: Entering directory `.../doc/DocBook'
> > 	make[1]: Leaving directory `.../doc/DocBook'
> > 
> > We didn't have this before, and we shouldn't have this.
> > 
> > Can you please fix this regression?  Thanks.
> 
> Is this not fixed in latest sources? I believe Tom fixed it.

It is not fixed in current top of tree in master, i. e.
v2013.01-rc1-78-g59852d0


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"The good Christian should beware of mathematicians and all those who
make empty prophecies. The danger already exists that  mathematicians
have  made a covenant with the devil to darken the spirit and confine
man in the bonds of Hell."                          - Saint Augustine

[U-Boot] Silence "make clean"

[Andreas Bießmann]

On 13.11.2012 17:21, Wolfgang Denk wrote:
> Dear Marek Vasut,
> 
> In message <201211131453.18366.marex@denx.de> you wrote:
>>
>>> 	make[1]: Entering directory `.../doc/DocBook'
>>> 	make[1]: Leaving directory `.../doc/DocBook'
>>>
>>> We didn't have this before, and we shouldn't have this.
>>>
>>> Can you please fix this regression?  Thanks.
>>
>> Is this not fixed in latest sources? I believe Tom fixed it.
> 
> It is not fixed in current top of tree in master, i. e.
> v2013.01-rc1-78-g59852d0

Tom did only 'fix' a special case for MAKEALL:
ed296d206a1840c0a0b6c1322379c950f9996905

Best regards

Andreas Bie?mann

[U-Boot] Silence "make clean"

[Marek Vasut]

Dear Andreas Bie?mann,

> On 13.11.2012 17:21, Wolfgang Denk wrote:
> > Dear Marek Vasut,
> > 
> > In message <201211131453.18366.marex@denx.de> you wrote:
> >>> 	make[1]: Entering directory `.../doc/DocBook'
> >>> 	make[1]: Leaving directory `.../doc/DocBook'
> >>> 
> >>> We didn't have this before, and we shouldn't have this.
> >>> 
> >>> Can you please fix this regression?  Thanks.
> >> 
> >> Is this not fixed in latest sources? I believe Tom fixed it.
> > 
> > It is not fixed in current top of tree in master, i. e.
> > v2013.01-rc1-78-g59852d0
> 
> Tom did only 'fix' a special case for MAKEALL:
> ed296d206a1840c0a0b6c1322379c950f9996905

How do you even replicate this? I don't see it on my box :-(

Best regards,
Marek Vasut

[U-Boot] [PATCH] Makefile: silence 'make clean'

[Andreas Bießmann]

Signed-off-by: Andreas Bie?mann <andreas.devel@googlemail.com>
Cc: Marek Vasut <marex@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
---
 Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 9dc89f9..5fa2fba 100644
--- a/Makefile
+++ b/Makefile
@@ -811,7 +811,7 @@ clean:
 	@rm -f $(obj)include/generated/asm-offsets.h
 	@rm -f $(obj)$(CPUDIR)/$(SOC)/asm-offsets.s
 	@rm -f $(TIMESTAMP_FILE) $(VERSION_FILE)
-	@$(MAKE) -C doc/DocBook/ cleandocs
+	@$(MAKE) -s -C doc/DocBook/ cleandocs
 	@find $(OBJTREE) -type f \
 		\( -name 'core' -o -name '*.bak' -o -name '*~' -o -name '*.su' \
 		-o -name '*.o'	-o -name '*.a' -o -name '*.exe'	\) -print \
-- 
1.7.10.4

[U-Boot] Silence "make clean"

[Andreas Bießmann]

Dear Marek Vasut,

On 13.11.2012 17:30, Marek Vasut wrote:
> Dear Andreas Bie?mann,
> 
>> On 13.11.2012 17:21, Wolfgang Denk wrote:
>>> Dear Marek Vasut,
>>>
>>> In message <201211131453.18366.marex@denx.de> you wrote:
>>>>> 	make[1]: Entering directory `.../doc/DocBook'
>>>>> 	make[1]: Leaving directory `.../doc/DocBook'
>>>>>
>>>>> We didn't have this before, and we shouldn't have this.
>>>>>
>>>>> Can you please fix this regression?  Thanks.
>>>>
>>>> Is this not fixed in latest sources? I believe Tom fixed it.
>>>
>>> It is not fixed in current top of tree in master, i. e.
>>> v2013.01-rc1-78-g59852d0
>>
>> Tom did only 'fix' a special case for MAKEALL:
>> ed296d206a1840c0a0b6c1322379c950f9996905
> 
> How do you even replicate this? 

just call 'make clean' in your u-boot source ;)

Best regards

Andreas Bie?mann

[U-Boot] Silence "make clean"

[Marek Vasut]

Dear Andreas Bie?mann,

> Dear Marek Vasut,
> 
> On 13.11.2012 17:30, Marek Vasut wrote:
> > Dear Andreas Bie?mann,
> > 
> >> On 13.11.2012 17:21, Wolfgang Denk wrote:
> >>> Dear Marek Vasut,
> >>> 
> >>> In message <201211131453.18366.marex@denx.de> you wrote:
> >>>>> 	make[1]: Entering directory `.../doc/DocBook'
> >>>>> 	make[1]: Leaving directory `.../doc/DocBook'
> >>>>> 
> >>>>> We didn't have this before, and we shouldn't have this.
> >>>>> 
> >>>>> Can you please fix this regression?  Thanks.
> >>>> 
> >>>> Is this not fixed in latest sources? I believe Tom fixed it.
> >>> 
> >>> It is not fixed in current top of tree in master, i. e.
> >>> v2013.01-rc1-78-g59852d0
> >> 
> >> Tom did only 'fix' a special case for MAKEALL:
> >> ed296d206a1840c0a0b6c1322379c950f9996905
> > 
> > How do you even replicate this?
> 
> just call 'make clean' in your u-boot source ;)

Let me just crawl under my rock now ... I'll pop up again once I'm over my 
shame.

... sum ... sum ... sum ... plop ...

Best regards,
Marek Vasut

[U-Boot] [PATCH] Makefile: silence 'make clean'

[Anatolij Gustschin]

Hi,

On Tue, 13 Nov 2012 17:31:39 +0100
Andreas Bie?mann <andreas.devel@googlemail.com> wrote:

> Signed-off-by: Andreas Bie?mann <andreas.devel@googlemail.com>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Wolfgang Denk <wd@denx.de>
> ---
>  Makefile |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Appied to staging/agust at denx.de, thanks!

Anatolij

[U-Boot] [PATCH] Makefile: silence 'make clean'

[Dirk Behme]

On 13.11.2012 17:31, Andreas Bie?mann wrote:
> Signed-off-by: Andreas Bie?mann <andreas.devel@googlemail.com>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Wolfgang Denk <wd@denx.de>

Tested-by: Dirk Behme <dirk.behme@de.bosch.com>

Thanks

Dirk

> ---
>  Makefile |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Makefile b/Makefile
> index 9dc89f9..5fa2fba 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -811,7 +811,7 @@ clean:
>  	@rm -f $(obj)include/generated/asm-offsets.h
>  	@rm -f $(obj)$(CPUDIR)/$(SOC)/asm-offsets.s
>  	@rm -f $(TIMESTAMP_FILE) $(VERSION_FILE)
> -	@$(MAKE) -C doc/DocBook/ cleandocs
> +	@$(MAKE) -s -C doc/DocBook/ cleandocs
>  	@find $(OBJTREE) -type f \
>  		\( -name 'core' -o -name '*.bak' -o -name '*~' -o -name '*.su' \
>  		-o -name '*.o'	-o -name '*.a' -o -name '*.exe'	\) -print \