diff mbox series

[2/2] package/gdb: also disable gprof

Message ID 20200928201128.78005-2-thomas.petazzoni@bootlin.com
State Accepted
Headers show
Series [1/2] package/gdb: fix build of gdbserver-only on the ARC gdb version | expand

Commit Message

Thomas Petazzoni Sept. 28, 2020, 8:11 p.m. UTC
When gdb is built from sources fetched from Git, it contains both the
gdb and the binutils code base. In order to really build only gdb, we
disable a number of binutils components in the
GDB_DISABLE_BINUTILS_CONF_OPTS variable: --disable-binutils,
--disable-ld, --disable-gas, etc. However, gprof was still being
built, so disable it as well.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
 package/gdb/gdb.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Yann E. MORIN Sept. 30, 2020, 8:52 p.m. UTC | #1
Thomas, All,

On 2020-09-28 22:11 +0200, Thomas Petazzoni spake thusly:
> When gdb is built from sources fetched from Git, it contains both the
> gdb and the binutils code base. In order to really build only gdb, we
> disable a number of binutils components in the
> GDB_DISABLE_BINUTILS_CONF_OPTS variable: --disable-binutils,
> --disable-ld, --disable-gas, etc. However, gprof was still being
> built, so disable it as well.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  package/gdb/gdb.mk | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
> index 2e5588bfbc..f31b168bf1 100644
> --- a/package/gdb/gdb.mk
> +++ b/package/gdb/gdb.mk
> @@ -81,7 +81,8 @@ GDB_DISABLE_BINUTILS_CONF_OPTS = \
>  	--disable-binutils \
>  	--disable-install-libbfd \
>  	--disable-ld \
> -	--disable-gas
> +	--disable-gas \
> +	--disable-gprof
>  
>  GDB_CONF_ENV = \
>  	ac_cv_type_uintptr_t=yes \
> -- 
> 2.26.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Peter Korsgaard Oct. 2, 2020, 8:33 p.m. UTC | #2
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

 > When gdb is built from sources fetched from Git, it contains both the
 > gdb and the binutils code base. In order to really build only gdb, we
 > disable a number of binutils components in the
 > GDB_DISABLE_BINUTILS_CONF_OPTS variable: --disable-binutils,
 > --disable-ld, --disable-gas, etc. However, gprof was still being
 > built, so disable it as well.

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Committed to 2020.02.x, 2020.05.x and 2020.08.x, thanks.
diff mbox series

Patch

diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
index 2e5588bfbc..f31b168bf1 100644
--- a/package/gdb/gdb.mk
+++ b/package/gdb/gdb.mk
@@ -81,7 +81,8 @@  GDB_DISABLE_BINUTILS_CONF_OPTS = \
 	--disable-binutils \
 	--disable-install-libbfd \
 	--disable-ld \
-	--disable-gas
+	--disable-gas \
+	--disable-gprof
 
 GDB_CONF_ENV = \
 	ac_cv_type_uintptr_t=yes \