[Buildroot] [PATCH V7 1/1] google-breakpad: Integration into Makefile and Config.in
Yann E. MORIN
yann.morin.1998 at free.fr
Tue Jul 29 23:03:35 UTC 2014
Pascal, All,
On 2014-07-23 16:09 +0200, Pascal Huerst spake thusly:
> Signed-off-by: Pascal Huerst <pascal.huerst at gmail.com>
I'll be shepherding this patch, since it still needs a bit of work.
For example, there is no host package for google breakpad, so it does
not work for me as I do not have googlebreakpad install on my host.
Also, I now know why you need this eval in the script. I do not like it,
so I'll try to come up with an alternative solution. Still, I have one
question: do you expect that one would enter shell wildcards in the list
of files to "breakpadize"?
It's a bit late here, now, so I'll continue tomorrow evening (GMT+2).
Regards,
Yann E. MORIN.
> ---
> Config.in | 28 ++++++++++++++++++++++++++++
> package/google-breakpad/gen-syms.sh | 25 +++++++++++++++++++++++++
> package/google-breakpad/google-breakpad.mk | 8 ++++++++
> 3 files changed, 61 insertions(+)
> create mode 100755 package/google-breakpad/gen-syms.sh
>
> diff --git a/Config.in b/Config.in
> index b169678..432ec28 100644
> --- a/Config.in
> +++ b/Config.in
> @@ -484,6 +484,34 @@ config BR2_OPTIMIZE_S
>
> endchoice
>
> +config BR2_GOOGLE_BREAKPAD_ENABLE
> + bool "Enable google-breakpad support"
> + select BR2_PACKAGE_GOOGLE_BREAKPAD
> + help
> + This option will enable the use of google breakpad, a library and tool
> + suite that allows you to distribute an application to users with
> + compiler-provided debugging information removed, record crashes in
> + compact "minidump" files, send them back to your server and produce
> + C and C++ stack traces from these minidumps. Breakpad can also write
> + minidumps on request for programs that have not crashed.
> +
> +if BR2_GOOGLE_BREAKPAD_ENABLE
> +
> +config BR2_GOOGLE_BREAKPAD_INCLUDE_FILES
> + string "List of executables and libraries to extract symbols from"
> + default ""
> + help
> + You may specify a space-separated list of binaries and libraries
> + with full paths relative to $(TARGET_DIR) of which debug symbols
> + will be dumped for further use with google breakpad.
> +
> + A directory structure that can be used by minidump-stackwalk will
> + be created at:
> +
> + $(STAGING_DIR)/usr/share/google-breakpad-symbols
> +
> +endif
> +
> config BR2_ENABLE_SSP
> bool "build code with Stack Smashing Protection"
> depends on BR2_TOOLCHAIN_HAS_SSP
> diff --git a/package/google-breakpad/gen-syms.sh b/package/google-breakpad/gen-syms.sh
> new file mode 100755
> index 0000000..f29c8fe
> --- /dev/null
> +++ b/package/google-breakpad/gen-syms.sh
> @@ -0,0 +1,25 @@
> +#!/bin/sh
> +STAGING_DIR="${1}"
> +TARGET_DIR="${2}"
> +shift 2
> +
> +SYMBOLS_DIR="${STAGING_DIR}/usr/share/google-breakpad-symbols"
> +rm -rf "${SYMBOLS_DIR}"
> +mkdir -p "${SYMBOLS_DIR}"
> +
> +for FILE in $(eval ls "${TARGET_DIR}/${@}"); do
> + if [ -d "${FILE}" ]; then
> + printf "Error: '%s' is a directory\n" "${FILE}" >&2
> + exit 1
> + fi
> + if dump_syms "${FILE}" > "${SYMBOLS_DIR}/tmp.sym" 2>/dev/null; then
> + HASH=$(head -n1 "${SYMBOLS_DIR}/tmp.sym" | cut -d ' ' -f 4);
> + FILENAME=$(basename "$FILE");
> + mkdir -p "${SYMBOLS_DIR}/${FILENAME}/${HASH}"
> + mv "${SYMBOLS_DIR}/tmp.sym" "${SYMBOLS_DIR}/${FILENAME}/${HASH}/${FILENAME}.sym";
> + else
> + printf "Error dumping symbols for: '%s'\n" "${FILE}" >&2
> + exit 1
> + fi
> +done
> +rm -rf "${SYMBOLS_DIR}/tmp"
> diff --git a/package/google-breakpad/google-breakpad.mk b/package/google-breakpad/google-breakpad.mk
> index bf857ba..8dea916 100644
> --- a/package/google-breakpad/google-breakpad.mk
> +++ b/package/google-breakpad/google-breakpad.mk
> @@ -14,5 +14,13 @@ GOOGLE_BREAKPAD_INSTALL_STAGING = YES
> GOOGLE_BREAKPAD_LICENSE = BSD-3c
> GOOGLE_BREAKPAD_LICENSE_FILES = LICENSE
>
> +ifeq ($(BR2_PACKAGE_GOOGLE_BREAKPAD),y)
> +define GOOGLE_BREAKPAD_EXTRACT_SYMBOLS
> + $(EXTRA_ENV) package/google-breakpad/gen-syms.sh $(STAGING_DIR) \
> + $(TARGET_DIR) $(call qstrip,$(BR2_GOOGLE_BREAKPAD_INCLUDE_FILES))
> +endef
> +TARGET_FINALIZE_HOOKS += GOOGLE_BREAKPAD_EXTRACT_SYMBOLS
> +endif
> +
> $(eval $(autotools-package))
> $(eval $(host-autotools-package))
> --
> 1.9.3
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list