Package: lua5.1-policy Version: 30 Severity: normal Tags: patch User: ubuntu-de...@lists.ubuntu.com Usertags: origin-ubuntu ubuntu-patch oneiric
https://bugs.launchpad.net/ubuntu/+source/lua-sec/+bug/770817 seems to me to be a lua5.1-policy bug. As http://wiki.debian.org/ToolChain/DSOLinking#Only_link_with_needed_libraries says, a linker that uses --as-needed by default requires object files to appear before the libraries they use on the linker command line. The following patch moves libraries after objects (which is historically the right thing to do with Unix linkers anyway). Could you please have a look and see if this is reasonable to merge? === modified file 'templates/Makefile.Debian.single' --- templates/Makefile.Debian.single 2011-05-20 10:03:36 +0000 +++ templates/Makefile.Debian.single 2011-05-20 10:11:54 +0000 @@ -230,22 +230,24 @@ $(LIBNAME).la $(LUA_MODNAME_PATH).so: $( -rpath $(LUA_LPATH) -o $(LIBNAME).la \ -version-info $(VERSION_INFO) \ -Wl,--no-add-needed \ - $(CLIB_LDFLAGS) $(CLIB_OBJS) + $(CLIB_OBJS) $(CLIB_LDFLAGS) mkdir -p $(dir $(LUA_MODNAME_PATH)) ln -sf $(LUA_MODNAME_DOTS)./.libs/$(REALSO) $(LUA_MODNAME_PATH).so app static-stamp.$(PKG_NAME): $(C_TEST_FILE_STAMP) $(LIBNAME).la pre-static-link-hook rm -f dynamic-stamp.$(PKG_NAME) $(LBTL) --mode=link $(CC) -Wall -O2 $(GCC_FLAGS) -Wl,--no-add-needed \ - $(CLIB_LDFLAGS_STATIC) $(LUA_LBTL_S) -static -o app -I . \ - $(C_TEST_FILE) $(LIBNAME).la + -static -o app -I . \ + $(C_TEST_FILE) $(LIBNAME).la \ + $(CLIB_LDFLAGS_STATIC) $(LUA_LBTL_S) touch static-stamp.$(PKG_NAME) ./.libs/app dynamic-stamp.$(PKG_NAME): $(C_TEST_FILE_STAMP) $(LIBNAME).la pre-dynamic-link-hook rm -f static-stamp.$(PKG_NAME) $(LBTL) --mode=link $(CC) -Wall -O2 $(GCC_FLAGS) -Wl,--no-add-needed \ - $(CLIB_LDFLAGS) $(LUA_LBTL_D) -o app -I . \ - $(C_TEST_FILE) $(LIBNAME).la + -o app -I . \ + $(C_TEST_FILE) $(LIBNAME).la \ + $(CLIB_LDFLAGS) $(LUA_LBTL_D) touch dynamic-stamp.$(PKG_NAME) # }}} Thanks, -- Colin Watson [cjwat...@ubuntu.com] -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
