137656 2014-10-13 05:26:17 -0700 [GTK][CMake] Non-ninja build is broken since r174422 2014-10-13 23:58:31 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED Gtk, Regression P2 Normal --- 1 cgarcia webkit-unassigned gustavo pnormand svillar oldest_to_newest 1041269 0 cgarcia 2014-10-13 05:26:17 -0700 For some reason CMake doesn't know how to generate DerivedSources/WebInspectorUI/UserInterface/Protocol/InspectorJSBackendCommands.js. The rule is there and it works when using ninja. It even fails if JavaScriptCore is built before and the file is already generated. make[2]: *** No rule to make target 'DerivedSources/WebInspectorUI/UserInterface/Protocol/InspectorJSBackendCommands.js', needed by 'DerivedSources/webkit2gtk/InspectorGResourceBundle.xml'. Alto. [ 1%] [ 1%] CMakeFiles/Makefile2:986: recipe for target 'Source/WebKit2/CMakeFiles/WebKit2-forwarding-headers.dir/all' failed make[1]: *** [Source/WebKit2/CMakeFiles/WebKit2-forwarding-headers.dir/all] Error 2 1041270 1 239723 cgarcia 2014-10-13 05:29:28 -0700 Created attachment 239723 Patch Making the forwarding headers generation rule not depend on generated sources seems to fix the problem 1041273 2 239723 svillar 2014-10-13 06:13:55 -0700 Comment on attachment 239723 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239723&action=review Do not have enough knowledge to review this... > Source/WebKit2/ChangeLog:3 > + [GTK][CMake] Non-nonja build is broken since r174422 Nit: ninja 1041275 3 cgarcia 2014-10-13 06:16:14 -0700 (In reply to comment #2) > (From update of attachment 239723 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=239723&action=review > > Do not have enough knowledge to review this... > > > Source/WebKit2/ChangeLog:3 > > + [GTK][CMake] Non-nonja build is broken since r174422 > > Nit: ninja Oops 1041486 4 cgarcia 2014-10-13 23:58:31 -0700 Committed r174671: <http://trac.webkit.org/changeset/174671> 239723 2014-10-13 05:29:28 -0700 2014-10-13 07:25:56 -0700 Patch wk-forwarding-headers-generated-sources.diff text/plain 2853 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NNYWtlTGlzdHMudHh0IGIvU291cmNlL1dlYktp dDIvQ01ha2VMaXN0cy50eHQKaW5kZXggNGE0NjU0NS4uYTc2MDg0MiAxMDA2NDQKLS0tIGEvU291 cmNlL1dlYktpdDIvQ01ha2VMaXN0cy50eHQKKysrIGIvU291cmNlL1dlYktpdDIvQ01ha2VMaXN0 cy50eHQKQEAgLTY4NSwxMSArNjg1LDExIEBAIGVuZG1hY3JvKCkKIAogV0VCS0lUX0lOQ0xVREVf Q09ORklHX0ZJTEVTX0lGX0VYSVNUUygpCiAKLUdFTkVSQVRFX1dFQktJVDJfTUVTU0FHRV9TT1VS Q0VTKFdlYktpdDJfU09VUkNFUyAiJHtXZWJLaXQyX01FU1NBR0VTX0lOX0ZJTEVTfSIpCitHRU5F UkFURV9XRUJLSVQyX01FU1NBR0VfU09VUkNFUyhXZWJLaXQyX0RFUklWRURfU09VUkNFUyAiJHtX ZWJLaXQyX01FU1NBR0VTX0lOX0ZJTEVTfSIpCiAKIGFkZF9kZWZpbml0aW9ucygtREJVSUxESU5H X1dFQktJVCkKIGluY2x1ZGVfZGlyZWN0b3JpZXMoJHtXZWJLaXQyX0lOQ0xVREVfRElSRUNUT1JJ RVN9KQotYWRkX2xpYnJhcnkoV2ViS2l0MiAke1dlYktpdDJfTElCUkFSWV9UWVBFfSAke1dlYktp dDJfU09VUkNFU30pCithZGRfbGlicmFyeShXZWJLaXQyICR7V2ViS2l0Ml9MSUJSQVJZX1RZUEV9 ICR7V2ViS2l0Ml9TT1VSQ0VTfSAke1dlYktpdDJfREVSSVZFRF9TT1VSQ0VTfSkKIAogYWRkX2Rl cGVuZGVuY2llcyhXZWJLaXQyIFdlYkNvcmUpCiBhZGRfZGVwZW5kZW5jaWVzKFdlYktpdDIgJHtX RUJLSVQyX0VYVFJBX0RFUEVOREVOQ0lFU30pCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYmFiZjI4MS4uYWYxZDg5 MGEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAxNC0xMC0xMyAgQ2FybG9zIEdhcmNp YSBDYW1wb3MgIDxjZ2FyY2lhQGlnYWxpYS5jb20+CisKKyAgICAgICAgW0dUS11bQ01ha2VdIE5v bi1ub25qYSBidWlsZCBpcyBicm9rZW4gc2luY2UgcjE3NDQyMgorICAgICAgICBodHRwczovL2J1 Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTM3NjU2CisKKyAgICAgICAgUmV2aWV3ZWQg YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTWFrZSBXZWJLaXQyLWZvcndhcmRpbmctaGVh ZGVycyBkZXBlbmQgb25seSBvbiBzdGF0aWMsCisgICAgICAgIG5vbi1nZW5lcmF0ZWQgc291cmNl cy4KKworICAgICAgICAqIENNYWtlTGlzdHMudHh0OgorICAgICAgICAqIFBsYXRmb3JtR1RLLmNt YWtlOgorCiAyMDE0LTEwLTEyICBTZXJnaW8gVmlsbGFyIFNlbmluICA8c3ZpbGxhckBpZ2FsaWEu Y29tPgogCiAgICAgICAgIFtTT1VQXSBbV0syXSBXZWJQcm9jZXNzIGFuZCBOZXR3b3JrUHJvY2Vz cyBpbml0aWFsaXphdGlvbiBjbGVhcnMgY2FjaGUgY29udGVudHMKZGlmZiAtLWdpdCBhL1NvdXJj ZS9XZWJLaXQyL1BsYXRmb3JtR1RLLmNtYWtlIGIvU291cmNlL1dlYktpdDIvUGxhdGZvcm1HVEsu Y21ha2UKaW5kZXggZmZkNjIzZC4uMzlhZmM3YyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIv UGxhdGZvcm1HVEsuY21ha2UKKysrIGIvU291cmNlL1dlYktpdDIvUGxhdGZvcm1HVEsuY21ha2UK QEAgLTE1LDEzICsxNSw2IEBAIGFkZF9kZWZpbml0aW9ucygtRExJQkRJUj0iJHtMSUJfSU5TVEFM TF9ESVJ9IikKIHNldChXZWJLaXQyX1VTRV9QUkVGSVhfSEVBREVSIE9OKQogCiBsaXN0KEFQUEVO RCBXZWJLaXQyX1NPVVJDRVMKLSAgICAke0RFUklWRURfU09VUkNFU19XRUJLSVQyR1RLX0RJUn0v SW5zcGVjdG9yR1Jlc291cmNlQnVuZGxlLmMKLSAgICAke0RFUklWRURfU09VUkNFU19XRUJLSVQy R1RLX0RJUn0vV2ViS2l0Mkluc3BlY3RvckdSZXNvdXJjZUJ1bmRsZS5jCi0gICAgJHtERVJJVkVE X1NPVVJDRVNfV0VCS0lUMkdUS19ESVJ9L1dlYktpdDJSZXNvdXJjZXNHUmVzb3VyY2VCdW5kbGUu YwotCi0gICAgJHtERVJJVkVEX1NPVVJDRVNfV0VCS0lUMkdUS19BUElfRElSfS9XZWJLaXRFbnVt VHlwZXMuY3BwCi0gICAgJHtERVJJVkVEX1NPVVJDRVNfV0VCS0lUMkdUS19BUElfRElSfS9XZWJL aXRNYXJzaGFsLmNwcAotCiAgICAgTmV0d29ya1Byb2Nlc3MvZ3RrL05ldHdvcmtQcm9jZXNzTWFp bkd0ay5jcHAKIAogICAgIE5ldHdvcmtQcm9jZXNzL3NvdXAvTmV0d29ya1Byb2Nlc3NTb3VwLmNw cApAQCAtMzE1LDYgKzMwOCwxNSBAQCBsaXN0KEFQUEVORCBXZWJLaXQyX1NPVVJDRVMKICAgICBX ZWJQcm9jZXNzL3NvdXAvV2ViUHJvY2Vzc1NvdXAuY3BwCiApCiAKK2xpc3QoQVBQRU5EIFdlYktp dDJfREVSSVZFRF9TT1VSQ0VTCisgICAgJHtERVJJVkVEX1NPVVJDRVNfV0VCS0lUMkdUS19ESVJ9 L0luc3BlY3RvckdSZXNvdXJjZUJ1bmRsZS5jCisgICAgJHtERVJJVkVEX1NPVVJDRVNfV0VCS0lU MkdUS19ESVJ9L1dlYktpdDJJbnNwZWN0b3JHUmVzb3VyY2VCdW5kbGUuYworICAgICR7REVSSVZF RF9TT1VSQ0VTX1dFQktJVDJHVEtfRElSfS9XZWJLaXQyUmVzb3VyY2VzR1Jlc291cmNlQnVuZGxl LmMKKworICAgICR7REVSSVZFRF9TT1VSQ0VTX1dFQktJVDJHVEtfQVBJX0RJUn0vV2ViS2l0RW51 bVR5cGVzLmNwcAorICAgICR7REVSSVZFRF9TT1VSQ0VTX1dFQktJVDJHVEtfQVBJX0RJUn0vV2Vi S2l0TWFyc2hhbC5jcHAKKykKKwogc2V0KFdlYktpdDJHVEtfSU5TVEFMTEVEX0hFQURFUlMKICAg ICAke0RFUklWRURfU09VUkNFU19XRUJLSVQyR1RLX0FQSV9ESVJ9L1dlYktpdEVudW1UeXBlcy5o CiAgICAgJHtERVJJVkVEX1NPVVJDRVNfV0VCS0lUMkdUS19BUElfRElSfS9XZWJLaXRWZXJzaW9u LmgK