189797 2018-09-20 12:31:42 -0700 [WPE] Built RPM of WPE webkit fails to install with "nothing provides libWPEToolingBackends.so()" 2018-09-21 04:42:53 -0700 1 1 1 Unclassified WebKit WPE WebKit WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 bunnyapocalypse aperez annulen aperez bugs-noreply cgarcia commit-queue mcatanzaro zan oldest_to_newest 1461483 0 bunnyapocalypse 2018-09-20 12:31:42 -0700 When trying to install from the RPM of WPE built on fedora, I get this error: Problem: cannot install the best candidate for the job - nothing provides libWPEToolingBackends.so()(64bit) needed by wpewebkit-2.21.92-1.fc30.x86_64 It seems like something that shouldn't be needed as a dependency is being listed as one. Thanks, Chris 1461495 1 aperez 2018-09-20 13:00:14 -0700 (In reply to Chris King from comment #0) > When trying to install from the RPM of WPE built on fedora, I get this error: > > Problem: cannot install the best candidate for the job > - nothing provides libWPEToolingBackends.so()(64bit) needed by > wpewebkit-2.21.92-1.fc30.x86_64 > > It seems like something that shouldn't be needed as a dependency is being > listed as one. I suppose the Fedora packaging is building _and_ installing the WPE MiniBrowser. Currently is the only installable component that uses libWPEToolingBackends.so Either we tell CMake to install the library when the MiniBrowser is also installed, or figure out whether it might be possible to link the library statically in the MiniBrowser. I'll try poke a bit and see what can be done. 1461511 2 aperez 2018-09-20 13:33:02 -0700 My impression is that it should be possible to make WPEToolingBackends a static library. I am going to make a build locally with it turned into into a static library which gets with PIC enabled (because its contents will be linked into shared objects afterwards) and make sure nothing breaks before posting a patch. 1461703 3 mcatanzaro 2018-09-20 20:42:01 -0700 (In reply to Adrian Perez from comment #1) > I suppose the Fedora packaging is building _and_ installing the WPE > MiniBrowser. Currently is the only installable component that uses > libWPEToolingBackends.so Ah good catch, I couldn't figure out where the dependency was coming from. > Either we tell CMake to install the library when the MiniBrowser is > also installed, or figure out whether it might be possible to link > the library statically in the MiniBrowser. I'll try poke a bit and > see what can be done. Yeah, one or the other. Static would be nicer, since it should be pretty small and won't be used anywhere else. 1461745 4 350334 aperez 2018-09-20 23:18:30 -0700 Created attachment 350334 Patch 1461746 5 aperez 2018-09-20 23:20:23 -0700 (In reply to Michael Catanzaro from comment #3) > (In reply to Adrian Perez from comment #1) > > I suppose the Fedora packaging is building _and_ installing the WPE > > MiniBrowser. Currently is the only installable component that uses > > libWPEToolingBackends.so > > Ah good catch, I couldn't figure out where the dependency was coming from. > > > Either we tell CMake to install the library when the MiniBrowser is > > also installed, or figure out whether it might be possible to link > > the library statically in the MiniBrowser. I'll try poke a bit and > > see what can be done. > > Yeah, one or the other. Static would be nicer, since it should be pretty > small and won't be used anywhere else. Just turning the library into a static one worked perfectly, I've just tested a build of MiniBrowser and running tests after building with the change in the attached patch. Let's also try and sneak in this small fix before releasing 2.22.0 :-) 1461747 6 aperez 2018-09-20 23:22:00 -0700 By the way, thanks a lot to Chris for reporting this! — I had been doing my testing with Cog instead of trying to install MiniBrowser and that's why this went unnoticed until now. Ooops O:-) 1461790 7 350334 commit-queue 2018-09-21 04:42:51 -0700 Comment on attachment 350334 Patch Clearing flags on attachment: 350334 Committed r236309: <https://trac.webkit.org/changeset/236309> 1461791 8 commit-queue 2018-09-21 04:42:53 -0700 All reviewed patches have been landed. Closing bug. 350334 2018-09-20 23:18:30 -0700 2018-09-21 04:42:51 -0700 Patch bug-189797-20180921091829.patch text/plain 1753 aperez U3VidmVyc2lvbiBSZXZpc2lvbjogMjM2MzA2CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggNzQ4YzE4ZDNhOWYyNzdlOWNjNzk0MGM4YTExNTViYzJm YTRiYjkyMC4uMmRkOWJkNWY4ODExNDdiOWNkODcwYzZkZTJkMTJjNTA5ZjQ3ZDllOCAxMDA2NDQK LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0 IEBACisyMDE4LTA5LTIwICBBZHJpYW4gUGVyZXogZGUgQ2FzdHJvICA8YXBlcmV6QGlnYWxpYS5j b20+CisKKyAgICAgICAgW1dQRV0gQnVpbHQgUlBNIG9mIFdQRSB3ZWJraXQgZmFpbHMgdG8gaW5z dGFsbCB3aXRoICJub3RoaW5nIHByb3ZpZGVzIGxpYldQRVRvb2xpbmdCYWNrZW5kcy5zbygpIgor ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTg5Nzk3CisK KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiB3cGUvYmFj a2VuZHMvQ01ha2VMaXN0cy50eHQ6IE1ha2UgbGliV1BFVG9vbGluZ0JhY2tlbmRzIGEgc3RhdGlj IGxpYnJhcnkgdG8gYXZvaWQKKyAgICAgICAgdGhlIG5lZWQgZm9yIGluc3RhbGxpbmcgaXQuIFRo aXMgd29ya3MgZmluZSBiZWNhdXNlIHRoZSBsaWJyYXJ5IG9ubHkgY29udGFpbnMgdXRpbGl0eQor ICAgICAgICBjb2RlIGludGVuZGVkIHRvIGJlIHJldXNlZCBmcm9tIG90aGVyIHZhcmlvdXMgY29t cG9uZW50cyAoTWluaUJyb3dzZXIsIFdlYktpdFRlc3RSdW5uZXIpLgorCiAyMDE4LTA5LTIwICBE YW5pZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KIAogICAgICAgICBNb3ZlIElPS2l0U1BJ LmggZnJvbSBUZXN0UnVubmVyU2hhcmVkIHRvIFBBTApkaWZmIC0tZ2l0IGEvVG9vbHMvd3BlL2Jh Y2tlbmRzL0NNYWtlTGlzdHMudHh0IGIvVG9vbHMvd3BlL2JhY2tlbmRzL0NNYWtlTGlzdHMudHh0 CmluZGV4IDQzOWU4ZTNmNDhmNDg5ZGZiMzUxMTZlNmE1YzkwYjY1NGZhNTFiZjkuLmY2YTE3MDg0 NTI1MzkwMzE4ZThhZDViNjNjNzgzMGVhMTg0OWMwZWUgMTAwNjQ0Ci0tLSBhL1Rvb2xzL3dwZS9i YWNrZW5kcy9DTWFrZUxpc3RzLnR4dAorKysgYi9Ub29scy93cGUvYmFja2VuZHMvQ01ha2VMaXN0 cy50eHQKQEAgLTQ5LDcgKzQ5LDcgQEAgYWRkX2N1c3RvbV9jb21tYW5kKAogICAgIENPTU1BTkQg JHtXQVlMQU5EX1NDQU5ORVJ9IGNsaWVudC1oZWFkZXIgJHtXQVlMQU5EX1BST1RPQ09MU19EQVRB RElSfS91bnN0YWJsZS94ZGctc2hlbGwveGRnLXNoZWxsLXVuc3RhYmxlLXY2LnhtbCAke0RFUklW RURfU09VUkNFU19XUEVUT09MSU5HQkFDS0VORFNfRElSfS94ZGctc2hlbGwtdW5zdGFibGUtdjYt Y2xpZW50LXByb3RvY29sLmgKICAgICBWRVJCQVRJTSkKIAotYWRkX2xpYnJhcnkoV1BFVG9vbGlu Z0JhY2tlbmRzICR7V1BFVG9vbGluZ0JhY2tlbmRzX1NPVVJDRVN9KQorYWRkX2xpYnJhcnkoV1BF VG9vbGluZ0JhY2tlbmRzIFNUQVRJQyAke1dQRVRvb2xpbmdCYWNrZW5kc19TT1VSQ0VTfSkKIGlu Y2x1ZGVfZGlyZWN0b3JpZXMoJHtXUEVUb29saW5nQmFja2VuZHNfSU5DTFVERV9ESVJFQ1RPUklF U30pCiBpbmNsdWRlX2RpcmVjdG9yaWVzKFNZU1RFTSAke1dQRVRvb2xpbmdCYWNrZW5kc19TWVNU RU1fSU5DTFVERV9ESVJFQ1RPUklFU30pCiB0YXJnZXRfbGlua19saWJyYXJpZXMoV1BFVG9vbGlu Z0JhY2tlbmRzICR7V1BFVG9vbGluZ0JhY2tlbmRzX0xJQlJBUklFU30pCg==