Implement debug symbol extraction for Linux and Android

livecodefraser · livecodefraser · commit b11fd68b10c1 · 2015-04-28T12:14:24.000+01:00
diff --git a/config/android.gypi b/config/android.gypi
@@ -33,6 +33,7 @@
 			'lib_suffix': '',
 			'ext_suffix': '',
 			'exe_suffix': '',
+			'debug_info_suffix': '.dbg',
 		},
 		
 		'cflags':
diff --git a/config/ios.gypi b/config/ios.gypi
@@ -48,6 +48,7 @@
 			'exe_suffix': '',
 			'lib_suffix': '.dylib',
 			'ext_suffix': '.so',
+			'debug_info_suffix': '.dsym',
 		},
 		
 		'defines':
diff --git a/config/linux.gypi b/config/linux.gypi
@@ -13,6 +13,7 @@
 			'lib_suffix': '.so',
 			'ext_suffix': '.so',
 			'exe_suffix': '',
+			'debug_info_suffix': '.dbg',
 			
 			'c++_std': '<!(echo ${CXX_STD:-gnu++03})',
 		},
diff --git a/config/mac.gypi b/config/mac.gypi
@@ -38,6 +38,7 @@
 			'exe_suffix': '',
 			'lib_suffix': '.dylib',
 			'ext_suffix': '.so',
+			'debug_info_suffix': '.dsym',
 		},
 		
 		'target_conditions':
diff --git a/config/win32.gypi b/config/win32.gypi
@@ -20,6 +20,7 @@
 			'lib_suffix': '.dll',
 			'ext_suffix': '.dll',
 			'exe_suffix': '.exe',
+			'debug_info_suffix': '',
 		},
 		
 		# Don't assume a Cygwin environment when invoking actions
diff --git a/livecode.gyp b/livecode.gyp
@@ -90,19 +90,86 @@
 			],
 		},
 		
+		{
+			'target_name': 'debug-symbols',
+			'type': 'none',
+			
+			'dependencies':
+			[
+				'LiveCode-all',
+			],
+			
+			'conditions':
+			[
+				[
+					'OS == "linux" or OS == "android"',
+					{
+						'variables':
+						{
+							'debug_symbol_files':
+							[
+								'>!@(["sh", "-c", "echo $@ | xargs -n1 | sed -e \\\"s/$/>(debug_info_suffix)/g\\\"", "echo", \'>@(dist_files)\'])',
+							],
+						},
+			
+						'actions':
+						[
+							{
+								'action_name': 'extract-debug-symbols',
+								'message': 'Extracting debug symbols',
+					
+								'inputs':
+								[
+									'>@(dist_files)',
+									'./tools/extract-debug-symbols.sh',
+								],
+					
+								'outputs':
+								[
+									'<@(debug_symbol_files)',
+								],
+					
+								'action':
+								[
+									'./tools/extract-debug-symbols.sh',
+									'>(debug_info_suffix)',
+									'<@(_inputs)',
+								],
+							},
+						],
+			
+						'all_dependent_settings':
+						{
+							'variables':
+							{
+								'dist_aux_files': [ '<@(debug_symbol_files)' ],
+							},
+						},
+					}
+				],
+			],
+		},
+		
 		{
 			'target_name': 'binzip-copy',
 			'type': 'none',
 			
+			'variables':
+			{
+				'dist_files': [],
+				'dist_aux_files': [],
+			},
+			
 			'dependencies':
 			[
 				'LiveCode-all',
+				'debug-symbols',
 			],
 			
 			'copies':
 			[{
 				'destination': '<(output_dir)',
-				'files': [ '>@(dist_files)' ],
+				'files': [ '>@(dist_files)', '>@(dist_aux_files)', ],
 			}],
 		},
 	],
diff --git a/tools/extract-debug-symbols.sh b/tools/extract-debug-symbols.sh
@@ -0,0 +1,27 @@
+#!/bin/bash
+
+set -e
+
+suffix="$1"
+shift
+inputs="$@"
+
+for input in $inputs ; do
+	output="${input}${suffix}"	
+
+	# Extract a copy of the debugging information
+	objcopy --only-keep-debug "$input" "$output" 
+	
+	# If this file is a dynamic library, don't do a full strip or we'll
+	# destroy it (no symbols => can't link to it)
+	if [[ "$(objdump -f $input)" =~ "DYNAMIC" ]] ; then
+		strip --preserve-dates --strip-debug "$input"
+	else
+		strip --preserve-dates --strip-debug --strip-unneeded "$input"
+	fi
+	
+	# Add a hint for the debugger so it can find the debug info
+	objcopy --remove-section=.gnu_debuglink "$input"
+	objcopy --preserve-dates --add-gnu-debuglink="$output" "$input"
+done
+
