[Buildroot] [RFC] package/gdb: simplify the disable doc hook

[Buildroot] [RFC] package/gdb: simplify the disable doc hook

[Romain Naour]

Depending on the gdb version we need to remove the documentation
build from different place in the gdb source tree.

From: gdb/Makefile.in
3aecfbd03dd2a8d49c97f1e68df42b077dbbd396

From: bfd/doc/Makefile.in and gprof/Makefile.in
1180bafd8e1de2b12db3c7c235b128c9e48d4abb

From: etc/Makefile.in
as proposed by [1]

The issue come from the commit e30465112ed4c6320dd19107302057a5f7712cf2
in the gdb tree that update the "missing" script which now return the
error code 127 when a tool is missing.

The same kind of hook will be needed when the binutils package will be
updated to 2.26 release.

Fixes:
http://autobuild.buildroot.net/results/55d/55d95d7decc8ff8c451601a38ce2c2818fe23bcc

[1] http://patchwork.ozlabs.org/patch/572150/

Signed-off-by: Romain Naour <romain.naour@gmail.com>
Cc: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/gdb/gdb.mk | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
index 3b0f501..65b2e67 100644
--- a/package/gdb/gdb.mk
+++ b/package/gdb/gdb.mk
@@ -53,13 +53,8 @@ endif
 
 # Prevent gdb to build the documentation
 define GDB_DISABLE_DOC
-	$(SED) '/^SUBDIRS =/ s/doc//' $(@D)/gdb/Makefile.in
-	if test -e $(@D)/bfd/doc/Makefile.in ; then \
-		$(SED) 's/^INFO_DEPS =.*$$/INFO_DEPS =/' $(@D)/bfd/doc/Makefile.in ; \
-	fi
-	if test -e $(@D)/gprof/Makefile.in ; then \
-		$(SED) 's/^INFO_DEPS =.*$$/INFO_DEPS =/' $(@D)/gprof/Makefile.in ; \
-	fi
+	echo "#!/bin/sh" > $(@D)/missing
+	echo "exit 0" >> $(@D)/missing
 endef
 GDB_PRE_CONFIGURE_HOOKS += GDB_DISABLE_DOC
 HOST_GDB_PRE_CONFIGURE_HOOKS += GDB_DISABLE_DOC
-- 
2.4.3

[Buildroot] [RFC] package/gdb: simplify the disable doc hook

[Thomas Petazzoni]

Romain,

On Mon,  1 Feb 2016 15:22:27 +0100, Romain Naour wrote:

> diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
> index 3b0f501..65b2e67 100644
> --- a/package/gdb/gdb.mk
> +++ b/package/gdb/gdb.mk
> @@ -53,13 +53,8 @@ endif
>  
>  # Prevent gdb to build the documentation
>  define GDB_DISABLE_DOC
> -	$(SED) '/^SUBDIRS =/ s/doc//' $(@D)/gdb/Makefile.in
> -	if test -e $(@D)/bfd/doc/Makefile.in ; then \
> -		$(SED) 's/^INFO_DEPS =.*$$/INFO_DEPS =/' $(@D)/bfd/doc/Makefile.in ; \
> -	fi
> -	if test -e $(@D)/gprof/Makefile.in ; then \
> -		$(SED) 's/^INFO_DEPS =.*$$/INFO_DEPS =/' $(@D)/gprof/Makefile.in ; \
> -	fi
> +	echo "#!/bin/sh" > $(@D)/missing
> +	echo "exit 0" >> $(@D)/missing

Is "missing" only used to check for makeinfo, or also for other things ?

We also have makeinfo related hacks in the gcc and binutils packages,
which basically consists in pointing them to "missing" as being the
makeinfo program.

It is really annoying that we have to do those hacks :-/

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [RFC] package/gdb: simplify the disable doc hook

[Arnout Vandecappelle]

On 01-02-16 16:01, Thomas Petazzoni wrote:
> Romain,
> 
> On Mon,  1 Feb 2016 15:22:27 +0100, Romain Naour wrote:
> 
>> diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
>> index 3b0f501..65b2e67 100644
>> --- a/package/gdb/gdb.mk
>> +++ b/package/gdb/gdb.mk
>> @@ -53,13 +53,8 @@ endif
>>  
>>  # Prevent gdb to build the documentation
>>  define GDB_DISABLE_DOC
>> -	$(SED) '/^SUBDIRS =/ s/doc//' $(@D)/gdb/Makefile.in
>> -	if test -e $(@D)/bfd/doc/Makefile.in ; then \
>> -		$(SED) 's/^INFO_DEPS =.*$$/INFO_DEPS =/' $(@D)/bfd/doc/Makefile.in ; \
>> -	fi
>> -	if test -e $(@D)/gprof/Makefile.in ; then \
>> -		$(SED) 's/^INFO_DEPS =.*$$/INFO_DEPS =/' $(@D)/gprof/Makefile.in ; \
>> -	fi
>> +	echo "#!/bin/sh" > $(@D)/missing
>> +	echo "exit 0" >> $(@D)/missing
> 
> Is "missing" only used to check for makeinfo, or also for other things ?
> 
> We also have makeinfo related hacks in the gcc and binutils packages,
> which basically consists in pointing them to "missing" as being the
> makeinfo program.

 IUC, the 'missing' program will only be used if there is no makeinfo installed
on the system. So if your build host happens to have makeinfo, the documentation
will be built after all.

 I don't think that is a good thing, so ac_cv_prog_MAKEINFO=missing should
probably be added as well...

> 
> It is really annoying that we have to do those hacks :-/

 Maybe an easier workaround is to add a directory support/wrappers in the PATH
that has empty definitions or other wrappers for programs that we want to work
around.

 Regards,
 Arnout

> 
> Thomas
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF