backout patch for bug 378463 due to bustage

Makefile.in

@@ -134,39 +134,16 @@ ifdef MOZ_PROFILE
 	/bin/find $(DIST) -name "*.dbg" -exec mv {} $(DIST)/$(BUILDID) \;
 endif # MOZ_PROFILE
 endif # MOZILLA_OFFICIAL
-endif # WINNT
-
-ifeq ($(OS_ARCH),WINNT)
-SYM_FIND_CMD := /bin/find . -path dist -prune -o -name "*.exe" \
-    -o -name "*.dll" -o -name "*.EXE" | sed "s/\.[^\.]*$$/\.pdb/"
-# we want to copy PDB files on Windows
-MAKE_SYM_STORE_ARGS := -c
-DUMP_SYMS_BIN := $(topsrcdir)/toolkit/airbag/tools/win32/dump_syms.exe
-endif
-ifeq ($(OS_ARCH),Darwin)
-# need to pass arch flags for universal builds
-ifdef UNIVERSAL_BINARY
-MAKE_SYM_STORE_ARGS := -a "ppc i386"
-SYM_DIST := $(DIST)/universal
-else
-MAKE_SYM_STORE_ARGS := -a $(OS_TEST)
-SYM_DIST := $(DIST)
-endif
-# |file| is stupid on universal binaries, it produces one line of output
-# for the file, and one line of output for each architecture contained within.
-SYM_FIND_CMD := find -L $(SYM_DIST) -type f -a -perm -100 -o -name "*.dylib" \
-    | xargs file -L | grep "Mach-O" | grep -v "for architecture" | cut -f1 -d':'
-DUMP_SYMS_BIN := $(DIST)/host/bin/dump_syms
-endif
 
 buildsymbols:
 ifdef MOZ_DEBUG_SYMBOLS
 ifdef MOZ_AIRBAG
 	echo building symbol store
 	mkdir -p $(DIST)/crashreporter-symbols/$(BUILDID)
-	$(SYM_FIND_CMD) | \
-	  xargs $(topsrcdir)/toolkit/airbag/tools/make_symbol_store.pl    \
-	  $(MAKE_SYM_STORE_ARGS) $(DUMP_SYMS_BIN)                         \
+	/bin/find . -path "./dist" -prune -o -name "*.exe"                \
+          -o -name "*.dll" -o -name "*.EXE" | sed "s/\.[^\.]*$$/\.pdb/" | \
+	  xargs $(topsrcdir)/toolkit/airbag/tools/make_symbol_store.pl -c \
+	  $(topsrcdir)/toolkit/airbag/tools/win32/dump_syms.exe           \
 	  $(DIST)/crashreporter-symbols/$(BUILDID) >                      \
 	  $(DIST)/crashreporter-symbols/$(BUILDID)/$(MOZ_APP_NAME)-$(MOZ_APP_VERSION)-$(OS_ARCH)-$(BUILDID)-symbols.txt
 	echo packing symbols
@@ -185,16 +162,15 @@ ifdef MOZ_AIRBAG
 endif
 endif
 
-ifeq ($(OS_ARCH),WINNT)
 signnss:
 ifdef MOZILLA_OFFICIAL
 	echo signing NSS libs
 	cd $(DIST)/bin; ./shlibsign.exe -v -i softokn3.dll
 	cd $(DIST)/bin; ./shlibsign.exe -v -i freebl3.dll
 endif # MOZILLA_OFFICIAL
 
+BUILDID = $(shell cat $(DEPTH)/config/build_number)
 deliver: splitsymbols rebase signnss
 
 endif # WINNT
 
-BUILDID = $(shell cat $(DEPTH)/config/build_number)

build/macosx/universal/flight.mk

@@ -86,9 +86,9 @@ postflight_all:
 # a universal binary too early, before the unified bits have been staged.
 # Set SIGN_NSS= to skip shlibsign.
 	$(MAKE) -C $(OBJDIR_PPC)/$(INSTALLER_DIR) \
-          UNIVERSAL_BINARY= SIGN_NSS= PKG_SKIP_STRIP=1 stage-package
+          UNIVERSAL_BINARY= SIGN_NSS= stage-package
 	$(MAKE) -C $(OBJDIR_X86)/$(INSTALLER_DIR) \
-          UNIVERSAL_BINARY= SIGN_NSS= PKG_SKIP_STRIP=1 stage-package
+          UNIVERSAL_BINARY= SIGN_NSS= stage-package
 # Remove .chk files that may have been copied from the NSS build.  These will
 # cause unify to warn or fail if present.  New .chk files that are
 # appropriate for the merged libraries will be generated when the universal

toolkit/airbag/Makefile.in

@@ -61,12 +61,9 @@ ifeq  ($(OS_ARCH),WINNT)
 endif
 
 ifeq ($(OS_ARCH),Darwin)
-DIRS += \
-  airbag/src/common/ \
-  airbag/src/common/mac \
-  airbag/src/client/mac/handler \
-  airbag/src/tools/mac/dump_syms \
-  $(NULL)
+	DIRS += airbag/src/common/mac \
+		airbag/src/client/mac/handler \
+		$(NULL)
 endif
 
 DIRS += client

toolkit/airbag/airbag/src/common/mac/Makefile.in

@@ -43,29 +43,24 @@ include $(DEPTH)/config/autoconf.mk
 
 MODULE		= breakpad_mac_common
 LIBRARY_NAME	= breakpad_mac_common_s
-HOST_LIBRARY_NAME = host_breakpad_mac_common_s
 
 LOCAL_INCLUDES 	= -I$(srcdir)/../..
 
-# This is a little weird, but we're building a host and a target lib here.
-# The host lib is used for dump_syms, and the target lib for the
-# crash reporter client.  Therefore, we don't need all the srcs in both.
-CPPSRCS	= \
-  file_id.cc \
-  macho_id.cc \
-  macho_walker.cc \
-  string_utilities.cc \
-  $(NULL)
+CPPSRCS		= \
+		file_id.cc \
+		macho_id.cc \
+		macho_walker.cc \
+		string_utilities.cc \
+		../string_conversion.cc \
+		$(NULL)
 
-CMSRCS = \
-  HTTPMultipartUpload.m \
-  $(NULL)
+CMSRCS		= \
+		HTTPMultipartUpload.m \
+		$(NULL)
 
-HOST_CPPSRCS = $(CPPSRCS)
+CSRCS           = \
+		../convert_UTF.c
 
-HOST_CMMSRCS = \
-  dump_syms.mm \
-  $(NULL)
 
 # need static lib
 FORCE_STATIC_LIB = 1

toolkit/airbag/tools/make_symbol_store.pl

@@ -35,49 +35,41 @@
 #
 # ***** END LICENSE BLOCK *****
 #
-# Usage: make_symbol_store.pl <params> <dump_syms path> <symbol store path>
-#                             <debug info files>
-#   Runs dump_syms on each debug info file specified on the command line,
+# Usage: make_symbol_store.pl <params> <dump_syms path> <symbol store path> <pdb files>
+#   Runs dump_syms on each pdb file specified on the command line,
 #   then places the resulting symbol file in the proper directory
-#   structure in the symbol store path.  Accepts multiple files
+#   structure in the symbol store path.  Accepts multiple pdb files
 #   on the command line, so can be called as part of a pipe using
-#   find <dir> | xargs make_symbol_store.pl <dump_syms> <storepath>
+#   find <dir> -name "*.pdb" | xargs make_symbol_store.pl <dump_syms> <storepath>
 #   Parameters accepted:
-#     -c           : Copy debug info files to the same directory structure
-#                    as sym files
-#     -a "<archs>" : Run dump_syms -a <arch> for each space separated
-#                    cpu architecture in <archs> (only on OS X)
+#     -c : Copy pdb files to the same directory structure as sym files
 
 use FileHandle;
 use File::Path;
 use File::Copy;
 use File::Basename;
 
-print "Usage: make_symbol_store.pl <params>" .
-  "<dump_syms path> <storepath> <debug info files>\n"
-  and exit if scalar @ARGV < 3;
+print "Usage: make_symbol_store.pl <dump_syms path> <storepath> <pdb files>>\n" and exit if scalar @ARGV < 3;
 
 # Given a symbol file generated by dump_syms,
 # and a directory to store the resulting symbol path,
 # move the symbol file into the directory structure
-# expected by the breakpad processor.  For details, see:
-# http://google-breakpad.googlecode.com/svn/trunk/src/processor/simple_symbol_supplier.h
+# expected by the airbag processor:
+# file.pdb/GUID+age/file.sym
 sub rename_symbol_file
 {
     my ($symbol_file, $dest_path) = @_;
     my $fh = FileHandle->new($symbol_file, "r");
     return "" unless $fh;
 
     my $line = <$fh>;
-    return "" unless $line;
     $line =~ s/\s*$//;
     $fh->close();
     return "" unless $line =~ m/^MODULE/;
 
-    # the first line of a sym file looks like:
-    # MODULE os cpu identifier debug_file
-    my ($guid,$dbgfile) = (split(/ +/, $line))[3..4];
-    my $newpath = $dest_path . "/" . $dbgfile . "/" . $guid;
+    my ($guid,$age,$dbgfile) = (split(/ +/, $line))[1..3];
+    $guid =~ s/\-//g;
+    my $newpath = $dest_path . "/" . $dbgfile . "/" . uc($guid . $age);
     eval { mkpath($newpath) };
     return "" if $@;
 
@@ -93,36 +85,24 @@ sub rename_symbol_file
     return "";
 }
 
-my $copy_dbg = 0;
-my @archs = ('');
-while (@ARGV && $ARGV[0] =~ m/^-/) {
-    my $arg = shift;
-    if ($arg eq '-c') {
-        $copy_dbg = 1;
-    }
-    elsif ($arg eq '-a') {
-        @archs = (split(/\s+/, shift));
-    }
+my $copy_pdbs = 0;
+if($ARGV[0] eq '-c') {
+    $copy_pdbs = 1;
+    shift;
 }
-
 my $dump_syms = shift;
 my $symbol_path = shift;
-foreach my $dbgfile (@ARGV) {
-    next unless -f $dbgfile;
-    # get filename without path or .pdb extension, if it exists
-    my ($sf) = fileparse($dbgfile, ".pdb");
+foreach my $pdbfile (@ARGV) {
+    next unless -f $pdbfile;
+    my ($sf) = fileparse($pdbfile, ".pdb");
     my $symfile = $symbol_path . "/" . $sf . ".sym";
-    foreach my $arch (@archs) {
-      my $a = '';
-      $a = "-a $arch" if $arch ne '';
-      system("${dump_syms} ${a} ${dbgfile} > ${symfile}");
-      my $newpath = rename_symbol_file $symfile, $symbol_path;
-      if ($copy_dbg && $newpath ne "") {
+    system("$dump_syms $pdbfile > $symfile");
+    my $newpath = rename_symbol_file $symfile, $symbol_path;
+    if($copy_pdbs && $newpath ne "") {
         my $out = $newpath;
         $out =~ s/^$symbol_path//;
         $out =~ s|^/||;
         print "$out/${sf}.pdb\n";
-        copy($dbgfile, $newpath);
-      }
+        copy($pdbfile, $newpath);
     }
 }

toolkit/xre/Makefile.in

@@ -200,7 +200,6 @@ endif
 ifeq ($(OS_ARCH),Darwin)
 SHARED_LIBRARY_LIBS += \
 	$(DEPTH)/toolkit/airbag/airbag/src/client/mac/handler/$(LIB_PREFIX)exception_handler_s.$(LIB_SUFFIX) \
-	$(DEPTH)/toolkit/airbag/airbag/src/common/$(LIB_PREFIX)breakpad_common_s.$(LIB_SUFFIX) \
 	$(DEPTH)/toolkit/airbag/airbag/src/common/mac/$(LIB_PREFIX)breakpad_mac_common_s.$(LIB_SUFFIX)
 endif
 endif