90593 2012-07-05 01:54:04 -0700 [EFL][GTK] jhbuild : Disable pixman demos build depending on GTK+ 2012-07-06 09:01:54 -0700 1 1 1 Unclassified WebKit WebKit EFL 528+ (Nightly build) Other Linux RESOLVED FIXED P2 Normal --- 0 dw.im webkit-unassigned cdumez donggwan.kim d-r gyuyoung.kim gyuyoung.kim lucas.de.marchi pnormand webkit.review.bot oldest_to_newest 663397 0 dw.im 2012-07-05 01:54:04 -0700 When I tried to build efl port, I got pixman build error when the jhbuild is running. - undefined reference to `g_resources_lookup_data'. - some more undefined references. If we give glib build dependency to pixman, it can be solved. 663399 1 150894 dw.im 2012-07-05 01:55:41 -0700 Created attachment 150894 Patch 663450 2 gyuyoung.kim 2012-07-05 03:20:24 -0700 To add glib dependency is not difficult problem. I wonder why there was this problem before. I think intel guys need take a look this. CC'ing dominik and christophe. 663451 3 gyuyoung.kim 2012-07-05 03:21:42 -0700 (In reply to comment #2) > To add glib dependency is not difficult problem. I wonder why there was this problem before. I think intel guys need take a look this. > > CC'ing dominik and christophe. I wonder why there was this problem before. => I wonder why there was no this problem before. 663475 4 150920 dw.im 2012-07-05 04:09:05 -0700 Created attachment 150920 Patch I got same error when I try to build gtk port. - undefined reference to `g_resources_lookup_data'. - undefined reference to `g_resources_open_stream' - undefined reference to `g_resources_get_info' So, I've fixed gtk port, as well. 663477 5 dw.im 2012-07-05 04:10:30 -0700 *** Bug 90599 has been marked as a duplicate of this bug. *** 663575 6 pnormand 2012-07-05 08:29:59 -0700 That's odd because pixman does not have any explicit dependency to glib, AFAIK. Can you paste the link error please? 663828 7 dw.im 2012-07-05 18:54:29 -0700 (In reply to comment #6) > That's odd because pixman does not have any explicit dependency to glib, AFAIK. Can you paste the link error please? I've pasted those at the previous comments ;) I paste those here again. They are link error during building pixman. - undefined reference to `g_resources_lookup_data'. - undefined reference to `g_resources_open_stream' - undefined reference to `g_resources_get_info' 663836 8 pnormand 2012-07-05 19:20:12 -0700 I wasn't clear, sorry. Including the command line please. 663838 9 dw.im 2012-07-05 19:27:01 -0700 (In reply to comment #8) > I wasn't clear, sorry. Including the command line please. Actually, I executed this command below. Tools/Script/build-webkit --efl (--gtk) pixman is the first library which jhbuild try to build, and it failed. 663841 10 dw.im 2012-07-05 19:28:53 -0700 (In reply to comment #6) > That's odd because pixman does not have any explicit dependency to glib, AFAIK. Can you paste the link error please? As I know, pixman has libc dependency, that's why this problem is solved when I gave the "glib" dependency to the pixman. Do we need to add "glibc" itself in the jhbuild? Or, just give the "glib" dependency to the pixman..? 663845 11 pnormand 2012-07-05 19:43:28 -0700 (In reply to comment #10) > (In reply to comment #6) > > That's odd because pixman does not have any explicit dependency to glib, AFAIK. Can you paste the link error please? > > As I know, pixman has libc dependency, > that's why this problem is solved when I gave the "glib" dependency to the pixman. > > Do we need to add "glibc" itself in the jhbuild? > Or, just give the "glib" dependency to the pixman..? Dude. glibc != glib.... As I asked in comment 9, can you provide the command line that fails? Which means, to be clear, the ld command that triggers this link error. 663889 12 dw.im 2012-07-05 22:19:29 -0700 (In reply to comment #11) > > Dude. glibc != glib.... > As I asked in comment 9, can you provide the command line that fails? Which means, to be clear, the ld command that triggers this link error. Ye... I know those are different.. ;) Below is the error log. When trying to link clip-test, link error is occurred. make[2]: Entering directory `./WebKitBuild/Dependencies/Source/pixman-0.24.0/demos' CC clip-test.o CC gtk-utils.o CCLD clip-test /usr/lib/gcc/i686-linux-gnu/4.6/../../../i386-linux-gnu/libgdk_pixbuf-2.0.so: undefined reference to `g_resources_lookup_data' /usr/lib/gcc/i686-linux-gnu/4.6/../../../i386-linux-gnu/libgdk_pixbuf-2.0.so: undefined reference to `g_resources_open_stream' /usr/lib/gcc/i686-linux-gnu/4.6/../../../i386-linux-gnu/libgdk_pixbuf-2.0.so: undefined reference to `g_resources_get_info' These are the libraries which the CCLD command links. clip_test_LDADD = $(LDADD) LDADD = $(top_builddir)/pixman/libpixman-1.la -lm $(GTK_LIBS) $(PNG_LIBS) GTK_LIBS are below. GTK_LIBS = -L/usr/local/lib -lgtk-x11-2.0 -lgdk-x11-2.0 -latk-1.0 -lgio-2.0 -lpangoft2-1.0 -lpangocairo-1.0 -lgdk_pixbuf-2.0 -lcairo -lpango-1.0 -lfreetype -lfontconfig -lgobject-2.0 -lglib-2.0 -lpixman-1 That's why we need to build glib library before we build the pixman. 663917 13 dw.im 2012-07-06 00:19:44 -0700 (In reply to comment #12) > /usr/lib/gcc/i686-linux-gnu/4.6/../../../i386-linux-gnu/libgdk_pixbuf-2.0.so: undefined reference to `g_resources_lookup_data' > /usr/lib/gcc/i686-linux-gnu/4.6/../../../i386-linux-gnu/libgdk_pixbuf-2.0.so: undefined reference to `g_resources_open_stream' > /usr/lib/gcc/i686-linux-gnu/4.6/../../../i386-linux-gnu/libgdk_pixbuf-2.0.so: undefined reference to `g_resources_get_info' > As we can see from this build log, pixman needs gdk_pixbuf library to build, for some reason. To build pixman using the jhbuild on any system, I think we need to build gdk_pixbuf before building pixman. we can simply give 'gdk_pixbuf' dependency to 'pixman', or build glib for the first time. (maybe, for gtk port, after 'make' will be fine.) 664081 14 pnormand 2012-07-06 06:44:29 -0700 Ah so GTK+ is needed for the demos, but it can be disabled with --enable-gtk=no at configure time. I would prefer this approach! 664107 15 150920 pnormand 2012-07-06 07:40:47 -0700 Comment on attachment 150920 Patch Please use --enable-gtk=no at configure time instead. 664122 16 151086 dw.im 2012-07-06 08:21:57 -0700 Created attachment 151086 Patch 664125 17 dw.im 2012-07-06 08:32:06 -0700 (In reply to comment #15) > (From update of attachment 150920 [details]) > Please use --enable-gtk=no at configure time instead. With this option, pixman is built successfully on my system. 664126 18 dw.im 2012-07-06 08:42:04 -0700 wow! thanks for the kind help and review~! 664131 19 151086 webkit.review.bot 2012-07-06 09:01:48 -0700 Comment on attachment 151086 Patch Clearing flags on attachment: 151086 Committed r121975: <http://trac.webkit.org/changeset/121975> 664132 20 webkit.review.bot 2012-07-06 09:01:54 -0700 All reviewed patches have been landed. Closing bug. 150894 2012-07-05 01:55:41 -0700 2012-07-05 04:09:05 -0700 Patch patch_90593_0705.patch text/plain 1039 dw.im ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhYjZk NmQwLi5jNzU5ZDg4IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh bmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTItMDctMDUgIERvbmd3b28gSW0gIDxkdy5pbUBz YW1zdW5nLmNvbT4KKworICAgICAgICBbRUZMXSBqaGJ1aWxkIDogcGl4bWFuIHNob3VsZCBoYXZl IGdsaWIgYnVpbGQgZGVwZW5kZW5jeS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTkwNTkzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuCisKKyAgICAgICAgKiBlZmwvamhidWlsZC5tb2R1bGVzOiBHaXZlIGdsaWIgYnVpbGQgZGVw ZW5kZW5jeSB0byBwaXhtYW4uCisKIDIwMTItMDctMDQgIFlvc2hpZnVtaSBJbm91ZSAgPHlvc2lu QGNocm9taXVtLm9yZz4KIAogICAgICAgICBbVG9vbHNdIHdlYmtpdF91bml0dGVzdC5weSBnb3Qg YXNzZXJ0aW9uCmRpZmYgLS1naXQgYS9Ub29scy9lZmwvamhidWlsZC5tb2R1bGVzIGIvVG9vbHMv ZWZsL2poYnVpbGQubW9kdWxlcwppbmRleCAxYTUzZDQxLi5jMDRkZjU3IDEwMDY0NAotLS0gYS9U b29scy9lZmwvamhidWlsZC5tb2R1bGVzCisrKyBiL1Rvb2xzL2VmbC9qaGJ1aWxkLm1vZHVsZXMK QEAgLTUyLDYgKzUyLDkgQEAKICAgPC9hdXRvdG9vbHM+CiAKICAgPGF1dG90b29scyBpZD0icGl4 bWFuIiBhdXRvZ2VuLXNoPSJjb25maWd1cmUiPgorICAgIDxkZXBlbmRlbmNpZXM+CisgICAgICA8 ZGVwIHBhY2thZ2U9ImdsaWIiLz4KKyAgICA8L2RlcGVuZGVuY2llcz4KICAgICA8YnJhbmNoIG1v ZHVsZT0icmVsZWFzZXMvcGl4bWFuLTAuMjQuMC50YXIuZ3oiIHZlcnNpb249IjAuMjQuMCIKICAg ICAgICAgICAgIHJlcG89ImNhaXJvZ3JhcGhpY3Mub3JnIgogICAgICAgICAgICAgaGFzaD0ic2hh MjU2OmE1NjQ3YzcxNThmMTAzZWVkZmY1ZmJhNzk5MDE4ZjQxNjlmNmIyNmI1NzNhYjc2ODU4MTJl YmM5YTFjNWQyZTQiCg== 150920 2012-07-05 04:09:05 -0700 2012-07-06 08:21:57 -0700 Patch patch_90593_0705_1.patch text/plain 1628 dw.im ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA4Y2U2 MmRmLi42Mzk5NzgxIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh bmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTItMDctMDUgIERvbmd3b28gSW0gIDxkdy5pbUBz YW1zdW5nLmNvbT4KKworICAgICAgICBbRUZMXVtHVEtdIGpoYnVpbGQgOiBwaXhtYW4gc2hvdWxk IGhhdmUgZ2xpYiBidWlsZCBkZXBlbmRlbmN5LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTA1OTMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKworICAgICAgICAqIGVmbC9qaGJ1aWxkLm1vZHVsZXM6IEdpdmUgZ2xpYiBidWls ZCBkZXBlbmRlbmN5IHRvIHBpeG1hbi4KKyAgICAgICAgKiBndGsvamhidWlsZC5tb2R1bGVzOiBH aXZlIGdsaWIgYnVpbGQgZGVwZW5kZW5jeSB0byBwaXhtYW4uCisKIDIwMTItMDctMDUgIFJ5dWFu IENob2kgIDxyeXVhbi5jaG9pQHNhbXN1bmcuY29tPgogCiAgICAgICAgIFtXazJdW0VGTF0gRUZM IG5lZWRzIGEgV2ViS2l0VGVzdFJ1bm5lcgpkaWZmIC0tZ2l0IGEvVG9vbHMvZWZsL2poYnVpbGQu bW9kdWxlcyBiL1Rvb2xzL2VmbC9qaGJ1aWxkLm1vZHVsZXMKaW5kZXggMWE1M2Q0MS4uYzA0ZGY1 NyAxMDA2NDQKLS0tIGEvVG9vbHMvZWZsL2poYnVpbGQubW9kdWxlcworKysgYi9Ub29scy9lZmwv amhidWlsZC5tb2R1bGVzCkBAIC01Miw2ICs1Miw5IEBACiAgIDwvYXV0b3Rvb2xzPgogCiAgIDxh dXRvdG9vbHMgaWQ9InBpeG1hbiIgYXV0b2dlbi1zaD0iY29uZmlndXJlIj4KKyAgICA8ZGVwZW5k ZW5jaWVzPgorICAgICAgPGRlcCBwYWNrYWdlPSJnbGliIi8+CisgICAgPC9kZXBlbmRlbmNpZXM+ CiAgICAgPGJyYW5jaCBtb2R1bGU9InJlbGVhc2VzL3BpeG1hbi0wLjI0LjAudGFyLmd6IiB2ZXJz aW9uPSIwLjI0LjAiCiAgICAgICAgICAgICByZXBvPSJjYWlyb2dyYXBoaWNzLm9yZyIKICAgICAg ICAgICAgIGhhc2g9InNoYTI1NjphNTY0N2M3MTU4ZjEwM2VlZGZmNWZiYTc5OTAxOGY0MTY5ZjZi MjZiNTczYWI3Njg1ODEyZWJjOWExYzVkMmU0IgpkaWZmIC0tZ2l0IGEvVG9vbHMvZ3RrL2poYnVp bGQubW9kdWxlcyBiL1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMKaW5kZXggMzMwNmQ3NC4uYTE1 NzkyYyAxMDA2NDQKLS0tIGEvVG9vbHMvZ3RrL2poYnVpbGQubW9kdWxlcworKysgYi9Ub29scy9n dGsvamhidWlsZC5tb2R1bGVzCkBAIC01OSw2ICs1OSw5IEBACiAgIDwvYXV0b3Rvb2xzPgogCiAg IDxhdXRvdG9vbHMgaWQ9InBpeG1hbiIgYXV0b2dlbi1zaD0iY29uZmlndXJlIj4KKyAgICA8ZGVw ZW5kZW5jaWVzPgorICAgICAgPGRlcCBwYWNrYWdlPSJnbGliIi8+CisgICAgPC9kZXBlbmRlbmNp ZXM+CiAgICAgPGJyYW5jaCBtb2R1bGU9InJlbGVhc2VzL3BpeG1hbi0wLjI0LjAudGFyLmd6IiB2 ZXJzaW9uPSIwLjI0LjAiCiAgICAgICAgICAgICByZXBvPSJjYWlyb2dyYXBoaWNzLm9yZyIKICAg ICAgICAgICAgIGhhc2g9InNoYTI1NjphNTY0N2M3MTU4ZjEwM2VlZGZmNWZiYTc5OTAxOGY0MTY5 ZjZiMjZiNTczYWI3Njg1ODEyZWJjOWExYzVkMmU0Igo= 151086 2012-07-06 08:21:57 -0700 2012-07-06 09:01:48 -0700 Patch patch_90593_0706.patch text/plain 1713 dw.im ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA4MGUy NzhiLi5mNWYwMjJjIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh bmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTItMDctMDYgIERvbmd3b28gSW0gIDxkdy5pbUBz YW1zdW5nLmNvbT4KKworICAgICAgICBbRUZMXVtHVEtdIGpoYnVpbGQgOiBEaXNhYmxlIHBpeG1h biBkZW1vcyBidWlsZCBkZXBlbmRpbmcgb24gR1RLKworICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTA1OTMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICAqIGVmbC9qaGJ1aWxkLm1vZHVsZXM6IEdpdmUgJy0tZW5h YmxlLWd0az1ubycgb3B0aW9uIHRvIHBpeG1hbi4KKyAgICAgICAgKiBndGsvamhidWlsZC5tb2R1 bGVzOiBHaXZlICctLWVuYWJsZS1ndGs9bm8nIG9wdGlvbiB0byBwaXhtYW4uCisKIDIwMTItMDct MDUgIEphc29uIExpdSAgPGphc29uLmxpdUB0b3JjaG1vYmlsZS5jb20uY24+CiAKICAgICAgICAg QWRkIEphc29uIExpdSB0byBjb21taXR0ZXJzLnB5LgpkaWZmIC0tZ2l0IGEvVG9vbHMvZWZsL2po YnVpbGQubW9kdWxlcyBiL1Rvb2xzL2VmbC9qaGJ1aWxkLm1vZHVsZXMKaW5kZXggMWE1M2Q0MS4u OTA4NGQ3OSAxMDA2NDQKLS0tIGEvVG9vbHMvZWZsL2poYnVpbGQubW9kdWxlcworKysgYi9Ub29s cy9lZmwvamhidWlsZC5tb2R1bGVzCkBAIC01MSw3ICs1MSw4IEBACiAgICAgPC9icmFuY2g+CiAg IDwvYXV0b3Rvb2xzPgogCi0gIDxhdXRvdG9vbHMgaWQ9InBpeG1hbiIgYXV0b2dlbi1zaD0iY29u ZmlndXJlIj4KKyAgPGF1dG90b29scyBpZD0icGl4bWFuIiBhdXRvZ2VuLXNoPSJjb25maWd1cmUi CisgICAgICAgICAgICAgYXV0b2dlbmFyZ3M9Ii0tZW5hYmxlLWd0az1ubyI+CiAgICAgPGJyYW5j aCBtb2R1bGU9InJlbGVhc2VzL3BpeG1hbi0wLjI0LjAudGFyLmd6IiB2ZXJzaW9uPSIwLjI0LjAi CiAgICAgICAgICAgICByZXBvPSJjYWlyb2dyYXBoaWNzLm9yZyIKICAgICAgICAgICAgIGhhc2g9 InNoYTI1NjphNTY0N2M3MTU4ZjEwM2VlZGZmNWZiYTc5OTAxOGY0MTY5ZjZiMjZiNTczYWI3Njg1 ODEyZWJjOWExYzVkMmU0IgpkaWZmIC0tZ2l0IGEvVG9vbHMvZ3RrL2poYnVpbGQubW9kdWxlcyBi L1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMKaW5kZXggMzMwNmQ3NC4uODEzMjJmMjUgMTAwNjQ0 Ci0tLSBhL1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMKKysrIGIvVG9vbHMvZ3RrL2poYnVpbGQu bW9kdWxlcwpAQCAtNTgsNyArNTgsOCBAQAogICAgIDwvYnJhbmNoPgogICA8L2F1dG90b29scz4K IAotICA8YXV0b3Rvb2xzIGlkPSJwaXhtYW4iIGF1dG9nZW4tc2g9ImNvbmZpZ3VyZSI+CisgIDxh dXRvdG9vbHMgaWQ9InBpeG1hbiIgYXV0b2dlbi1zaD0iY29uZmlndXJlIgorICAgICAgICAgICAg IGF1dG9nZW5hcmdzPSItLWVuYWJsZS1ndGs9bm8iPgogICAgIDxicmFuY2ggbW9kdWxlPSJyZWxl YXNlcy9waXhtYW4tMC4yNC4wLnRhci5neiIgdmVyc2lvbj0iMC4yNC4wIgogICAgICAgICAgICAg cmVwbz0iY2Fpcm9ncmFwaGljcy5vcmciCiAgICAgICAgICAgICBoYXNoPSJzaGEyNTY6YTU2NDdj NzE1OGYxMDNlZWRmZjVmYmE3OTkwMThmNDE2OWY2YjI2YjU3M2FiNzY4NTgxMmViYzlhMWM1ZDJl NCIK