22811 2008-12-11 14:27:07 -0800 Underlinking in Programs_UnitTests (GTK+ build) 2009-02-19 13:43:18 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) PC Linux RESOLVED FIXED P2 Normal --- 0 adamw webkit-unassigned zecke oldest_to_newest 102279 0 adamw 2008-12-11 14:27:07 -0800 Since the unit tests were added to the webkit GTK+ build by zecke (in 39082), there's a linking problem. If you build with -Wl,--as-needed -Wl,--no-undefined (which strictly exposes underlinking issues), the build fails with the following: gcc -fno-strict-aliasing -O2 -Wall -W -Wcast-align -Wchar-subscripts -Wreturn-type -Wformat -Wformat-security -Wno-format-y2k -Wundef -Wmissing-format-attribute -Wpointer-arith -Wwrite-strings -Wno-unused-parameter -Wno-parentheses -fno-exceptions -fvisibility=hidden -I/usr/include/gtk-2.0 -I/usr/lib64/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng12 -O2 -g -pipe -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -O2 -Wl,--as-needed -Wl,--no-undefined -o Programs/.libs/UnitTests WebKit/gtk/tests/Programs_UnitTests-main.o ./.libs/libwebkit-1.0.so /usr/lib64/libgtk-x11-2.0.so /usr/lib64/libgdk-x11-2.0.so /usr/lib64/libatk-1.0.so /usr/lib64/libgio-2.0.so /usr/lib64/libpangoft2-1.0.so /usr/lib64/libgdk_pixbuf-2.0.so /usr/lib64/libpangocairo-1.0.so /usr/lib64/libcairo.so /usr/lib64/libpango-1.0.so /usr/lib64/libfreetype.so -lz /usr/lib64/libfontconfig.so /usr/lib64/libgobject-2.0.so /usr/lib64/libgmodule-2.0.so /usr/lib64/libglib-2.0.so WebKit/gtk/tests/Programs_UnitTests-main.o: In function `main': /home/mandrake/rpm/BUILD/WebKit-r39090/WebKit/gtk/tests/main.c:27: undefined reference to `g_thread_init' The attached patch fixes this - I'm not sure if it's a correct fix, it may be overbroad, but I couldn't find a LIBS variable which seemed more correct, so I just added GLOBALDEPS_LIBS. 102280 1 25964 adamw 2008-12-11 14:29:13 -0800 Created attachment 25964 underlink fix 110274 2 25964 jmalonzo 2009-02-19 12:37:41 -0800 Comment on attachment 25964 underlink fix > --- WebKit-r39090/GNUmakefile.am 2008-12-07 19:56:15.000000000 -0800 > +++ WebKit-r39090/GNUmakefile.am.new 2008-12-11 13:09:30.000000000 -0800 > @@ -481,7 +481,9 @@ > > Programs_UnitTests_LDADD = \ > libwebkit-1.0.la \ > - $(GTK_LIBS) > + $(GTK_LIBS) \ > + $(GLOBALDEPS_LIBS) > + Hi! GLOBALDEPS_LIBS is too much. Please link against GLIB_LIBS instead. Thanks 110276 3 adamw 2009-02-19 12:47:52 -0800 Ah, thanks. I figured there was a less huge variable I could use, but couldn't figure out what it was :) 110294 4 christian 2009-02-19 13:43:18 -0800 2009-02-19 Christian Dywan <christian@twotoasts.de> Rubber-stamped by Holger Freyther. http://bugs.webkit.org/show_bug.cgi?id=22811 Underlinking in Programs_UnitTests (GTK+ build) * GNUmakefile.am: Add GLIB_LIBS to unit test library flags. 25964 2008-12-11 14:29:13 -0800 2008-12-11 14:29:13 -0800 underlink fix webkit-39090-underlink.patch text/plain 316 adamw LS0tIFdlYktpdC1yMzkwOTAvR05VbWFrZWZpbGUuYW0JMjAwOC0xMi0wNyAxOTo1NjoxNS4wMDAw MDAwMDAgLTA4MDAKKysrIFdlYktpdC1yMzkwOTAvR05VbWFrZWZpbGUuYW0ubmV3CTIwMDgtMTIt MTEgMTM6MDk6MzAuMDAwMDAwMDAwIC0wODAwCkBAIC00ODEsNyArNDgxLDkgQEAKIAogUHJvZ3Jh bXNfVW5pdFRlc3RzX0xEQUREID0gXAogCWxpYndlYmtpdC0xLjAubGEgXAotCSQoR1RLX0xJQlMp CisJJChHVEtfTElCUykJXAorCSQoR0xPQkFMREVQU19MSUJTKQorCQogCiAjIEF1dG9nZW5lcmF0 ZWQgc291cmNlcwogQlVJTFRfU09VUkNFUyA6PSBcCg==