WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
22779
Unit test for the WebKit/Gtk+ API should also check GTK version
https://bugs.webkit.org/show_bug.cgi?id=22779
Summary
Unit test for the WebKit/Gtk+ API should also check GTK version
Arthur Webkid
Reported
2008-12-10 06:40:49 PST
trunk/WebKit/gtk/tests/main.c It should check for GTK version >= 2.14 as gtk_test_init is not avaiable for older version of GTK. Currently, it only checks for GLIB>=2.16 The current result is I can't build Webkit after
r39082
with GLIB 2.16 and GTK 2.12. It fails with the following messages: ./doltlibtool --tag=CC --mode=link gcc -O2 -o Programs/UnitTests WebKit/gtk/tests/Programs_UnitTests-main.o libwebkit-1.0.la -lgtk-x11-2.0 -lgdk-x11-2.0 -latk-1.0 -lgdk_pixbuf-2.0 -lm -lpangocairo-1.0 -lpango-1.0 -lcairo -lgobject-2.0 -lgmodule-2.0 -ldl -lglib-2.0 gcc -O2 -o Programs/.libs/UnitTests WebKit/gtk/tests/Programs_UnitTests-main.o ./.libs/libwebkit-1.0.so /usr/lib/libgtk-x11-2.0.so /usr/lib/libgdk-x11-2.0.so /usr/lib/libatk-1.0.so /usr/lib/libgdk_pixbuf-2.0.so -lm /usr/lib/libpangocairo-1.0.so /usr/lib/libpango-1.0.so /usr/lib/libcairo.so /usr/lib/libgobject-2.0.so /usr/lib/libgmodule-2.0.so -ldl /usr/lib/libglib-2.0.so WebKit/gtk/tests/Programs_UnitTests-main.o: In function `main': main.c:(.text+0x37): undefined reference to `gtk_test_init' collect2: ld returned 1 exit status make[1]: *** [Programs/UnitTests] Error 1 make[1]: Leaving directory `/home/arthur/src/WebKit-
r39090
' make: *** [all] Error 2 Reference:
http://trac.webkit.org/changeset/39082
Attachments
patch
(1.26 KB, patch)
2008-12-11 02:28 PST
,
Adam Bergkvist
zecke
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Adam Bergkvist
Comment 1
2008-12-11 02:28:11 PST
Created
attachment 25946
[details]
patch Here's a quick bug fix since people are having problems building.
Holger Freyther
Comment 2
2008-12-13 09:47:32 PST
Comment on
attachment 25946
[details]
patch Yes (and I will remove the extra parentheses).
Holger Freyther
Comment 3
2008-12-13 10:39:41 PST
Landed in
r39275
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug