[patch] libunwind: Use .so with its major version number

[patch] libunwind: Use .so with its major version number

[Jan Kratochvil]

[-- Attachment #1: Type: text/plain, Size: 456 bytes --]

Hi,

GDB currently dlopen()s `libunwind-ia64.so':
	#define LIBUNWIND_SO "libunwind-" STRINGIFY(UNW_TARGET) ".so"

and uses its symbol in a defined ABI way afterwards.
This makes an incompatible ABI change impossible.

`libunwind-ia64.so' should be used only for the build time while runtime should
use `libunwind-ia64.so.7' in this case.

libunwind author's approval:
	http://lists.gnu.org/archive/html/libunwind-devel/2007-04/msg00006.html


Regards,
Jan

[-- Attachment #2: gdb-6.6-libunwind-major-version.patch --]
[-- Type: text/plain, Size: 784 bytes --]

2007-04-11  Jan Kratochvil  <jan.kratochvil@redhat.com>

	* libunwind-frame.c (LIBUNWIND_SO): Use major version number for `.so'.


libunwind author's approval:
	http://lists.gnu.org/archive/html/libunwind-devel/2007-04/msg00006.html


--- gdb-6.6-orig/gdb/libunwind-frame.c	2007-04-08 20:49:09.000000000 +0200
+++ gdb-6.6/gdb/libunwind-frame.c	2007-04-10 23:45:22.000000000 +0200
@@ -74,7 +74,9 @@
 #define STRINGIFY(name)		STRINGIFY2(name)
 
 #ifndef LIBUNWIND_SO
-#define LIBUNWIND_SO "libunwind-" STRINGIFY(UNW_TARGET) ".so"
+/* Use the stable ABI major version number.  `libunwind-ia64.so' is a link time
+   only library, not a runtime one.  */
+#define LIBUNWIND_SO "libunwind-" STRINGIFY(UNW_TARGET) ".so.7"
 #endif
 
 static char *get_reg_name = STRINGIFY(UNW_OBJ(get_reg));

Re: [patch] libunwind: Use .so with its major version number

[Daniel Jacobowitz]

On Wed, Apr 11, 2007 at 02:32:09PM +0200, Jan Kratochvil wrote:
> Hi,
> 
> GDB currently dlopen()s `libunwind-ia64.so':
> 	#define LIBUNWIND_SO "libunwind-" STRINGIFY(UNW_TARGET) ".so"
> 
> and uses its symbol in a defined ABI way afterwards.
> This makes an incompatible ABI change impossible.
> 
> `libunwind-ia64.so' should be used only for the build time while runtime should
> use `libunwind-ia64.so.7' in this case.
> 
> libunwind author's approval:
> 	http://lists.gnu.org/archive/html/libunwind-devel/2007-04/msg00006.html

Yes, this is OK.

-- 
Daniel Jacobowitz
CodeSourcery