From 08df4201a7f9ae6154eed47de8d9e3323e1af2e7 Mon Sep 17 00:00:00 2001 From: Vincent Palatin Date: Wed, 18 Jan 2012 21:43:02 -0800 Subject: [PATCH] Fix amd64 platform detection for minidump symbols generation Not all 64-bit platforms have names starting with amd64-, so we should use portageq to get the board architecture name. BUG=chromium-os:25228 TEST=./cros_generate_breakpad_symbols --board=x86-alex, amd64-corei7, link Change-Id: I83769575dbd19112b929724995d0c97ed4df2b02 Reviewed-on: https://gerrit.chromium.org/gerrit/14444 Commit-Ready: Vincent Palatin Tested-by: Vincent Palatin Reviewed-by: David James --- cros_generate_breakpad_symbols | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cros_generate_breakpad_symbols b/cros_generate_breakpad_symbols index 67fe516afc..ee3b323b57 100755 --- a/cros_generate_breakpad_symbols +++ b/cros_generate_breakpad_symbols @@ -173,8 +173,8 @@ function main() { [ -n "$FLAGS_board" ] || die "--board is required." - case "$FLAGS_board" in - amd64-*) + case "$(portageq-${FLAGS_board} envvar ARCH)" in + amd64) echo "Detected amd64 board.." DUMP_SYMS="/build/${FLAGS_board}/usr/bin/dump_syms" ;;