121405 2013-09-15 15:29:30 -0700 [GTK] Cannot find WebInspectorUI/UserInterface/Main.html referenced in GResourceBundle.xml 2013-09-18 07:54:58 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 calvaris gustavo cgarcia commit-queue gustavo zan oldest_to_newest 929512 0 calvaris 2013-09-15 15:29:30 -0700 When building WebKit from upstream in GNOME Jhbuild with moduleset 3.10 I am getting this problem ./DerivedSources/WebInspectorUI/GResourceBundle.xml: Error on line 5 char 1: Failed to locate 'Source/WebInspectorUI/UserInterface/Main.html' in any source directory. 929514 1 calvaris 2013-09-15 15:31:27 -0700 The autogen.sh is run with the following options that hadn't been a problem so far: --enable-introspection --disable-static --enable-introspection --disable-gtk-doc --disable-introspection --enable-silent-rules 929525 2 gustavo 2013-09-15 17:55:10 -0700 Hmm, is that using a specific tarball release? 930168 3 calvaris 2013-09-17 05:17:36 -0700 (In reply to comment #2) > Hmm, is that using a specific tarball release? No, I just use latest version and to not interfere with my upstream compilation, I just clone it by using git archive --format=tar svn-trunk | (cd /my/destination/inside/gnome/jhbuild/ && tar xf -) 930357 4 gustavo 2013-09-17 13:07:06 -0700 And I guess the tarball that's being generated by git archive (I have only recently been introduced to it) not including that file? 930382 5 calvaris 2013-09-17 14:09:39 -0700 (In reply to comment #4) > And I guess the tarball that's being generated by git archive (I have only recently been introduced to it) not including that file? Nope, the file is there. 930469 6 gustavo 2013-09-17 18:19:15 -0700 OK, I came up with a theory and it looks like it's correct: this only happens if you have srcdir == builddir (which is why we haven't noticed yet), and is caused by automake helpfully resolving the paths of the dependencies of a target, thus removing the useless ./ (that is $(srcdir), which is used to form $(WebInspectorUI)), but keeps that useless ./ in the actual WebInspectorUI variable, so my sed for normalizing the paths was not replacing anything in that case. Patch incoming, just testing with srcdir != builddir now. 930472 7 211962 gustavo 2013-09-17 18:24:34 -0700 Created attachment 211962 Patch 930536 8 211962 commit-queue 2013-09-17 23:50:55 -0700 Comment on attachment 211962 Patch Clearing flags on attachment: 211962 Committed r156027: <http://trac.webkit.org/changeset/156027> 930537 9 commit-queue 2013-09-17 23:50:57 -0700 All reviewed patches have been landed. Closing bug. 930547 10 ap 2013-09-18 00:09:28 -0700 I wonder why commit queue didn't use your e-mail for this commit: <http://trac.webkit.org/changeset/156027>. Perhaps your data in committers.json is not up to date? 930617 11 gustavo 2013-09-18 07:54:58 -0700 It seems to be up-to-date, that file has my name and the gns@gnome.org email. I wonder if the svn post-commit script is only rewriting the email used as username for committing (that would be kov@webkit.org) 211962 2013-09-17 18:24:34 -0700 2013-09-17 23:50:55 -0700 Patch bug-121405-20130917222432.patch text/plain 2149 gustavo U3VidmVyc2lvbiBSZXZpc2lvbjogMTU1OTg1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViSW5zcGVj dG9yVUkvQ2hhbmdlTG9nIGIvU291cmNlL1dlYkluc3BlY3RvclVJL0NoYW5nZUxvZwppbmRleCBi ZDk4MzE3ZGE5MmMyY2FmNzRhM2I5ODNmZWNiMGQwM2RkMDA0NWE3Li42ZTg0ZDdkNmE2OTcxZTNh MGZjZjgxMTg5NGJlOGQyNWQyZDUxMjhjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViSW5zcGVjdG9y VUkvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKQEAgLTEs MyArMSwxNiBAQAorMjAxMy0wOS0xNyAgR3VzdGF2byBOb3JvbmhhIFNpbHZhICA8Z25zQGdub21l Lm9yZz4KKworICAgICAgICBbR1RLXSBDYW5ub3QgZmluZCBXZWJJbnNwZWN0b3JVSS9Vc2VySW50 ZXJmYWNlL01haW4uaHRtbCByZWZlcmVuY2VkIGluIEdSZXNvdXJjZUJ1bmRsZS54bWwKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEyMTQwNQorCisgICAg ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogR05VbWFrZWZpbGUu YW06IGlmIHNyY2RpciA9PSBidWlsZGRpciB0aGUgV2ViSW5zcGVjdG9yVUkgdmFyaWFibGUgdGhh dCBpcyB1c2VkIGJvdGggdG8gbGlzdAorICAgICAgICB0aGUgZGVwZW5kZW5jaWVzIG9mIHRoZSBH UmVzb3VyY2VCdW5kbGUgYW5kIHRvIG5vcm1hbGl6ZSB0aGUgcGF0aHMgYWRkZWQgdG8gdGhlIFhN TCB0aHJvdWdoIHNlZAorICAgICAgICByZXBsYWNlbWVudCB3aWxsIGVuZCB1cCBzdGFydGluZyB3 aXRoIC4vLCB3aGljaCB3b3VsZCBiZSBmaW5lIGlmIGF1dG9tYWtlIGRpZCBub3QgcmVtb3ZlIHRo YXQKKyAgICAgICAgZnJvbSB0aGUgcGF0aHMgaW4gdGhlIHRhcmdldCdzIGRlcGVuZGVuY3kuIFdl IGRlYWwgd2l0aCB0aGF0IGJ5IG5vcm1hbGl6aW5nIHRoZSBwYXRoIHdlIHVzZSB0bworICAgICAg ICB0YWtlIHRoYXQgcXVpcmsgaW50byBhY2NvdW50LgorCiAyMDEzLTA5LTE3ICBUaW1vdGh5IEhh dGNoZXIgIDx0aW1vdGh5QGFwcGxlLmNvbT4KIAogICAgICAgICBDbGVhbiB1cCB0aGUgSW5zcGVj dG9yJ3MgV2ViU29ja2V0IGNvZGUgYW5kIHByZXZlbnQgaXQgZnJvbSBkcm9wcGluZyBtZXNzYWdl cy4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9HTlVtYWtlZmlsZS5hbSBiL1Nv dXJjZS9XZWJJbnNwZWN0b3JVSS9HTlVtYWtlZmlsZS5hbQppbmRleCA2OGU2MjFhNDJhNDA1ZDJm ODA1NDY0NzcyNmJlNzI1M2MzYTJkZDNjLi5kNmIyZGFmYmI5ZTYzMzQxN2ViOGE1YjM2YzI1ODJh NzllYjljNTZkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvR05VbWFrZWZpbGUu YW0KKysrIGIvU291cmNlL1dlYkluc3BlY3RvclVJL0dOVW1ha2VmaWxlLmFtCkBAIC0xMiw5ICsx MiwxMCBAQCAke0dFTlNPVVJDRVNfV0VCSU5TUEVDVE9SX1VJfS9HUmVzb3VyY2VCdW5kbGUueG1s OiBHTlVtYWtlZmlsZSBcCiAJJChBTV9WX2F0KWVjaG8gJzxncmVzb3VyY2VzPicgPj4gJHtHRU5T T1VSQ0VTX1dFQklOU1BFQ1RPUl9VSX0vR1Jlc291cmNlQnVuZGxlLnhtbAogCSQoQU1fVl9hdCll Y2hvICcgIDxncmVzb3VyY2UgcHJlZml4PSIvb3JnL3dlYmtpdGd0ay9pbnNwZWN0b3IiPicgPj4g JHtHRU5TT1VSQ0VTX1dFQklOU1BFQ1RPUl9VSX0vR1Jlc291cmNlQnVuZGxlLnhtbAogCSQoQU1f Vl9hdClcCisJCWJhc2VwYXRoPWBlY2hvICQoV2ViSW5zcGVjdG9yVUkpLyB8IHNlZCAncyxeLi8s LGcnYDsgXAogCQlmb3IgZmlsZSBpbiAkXjsgZG8gXAogCQkJaWYgWyAiJCRmaWxlIiA9ICJHTlVt YWtlZmlsZSIgXTsgdGhlbiBjb250aW51ZTsgZmk7IFwKLQkJCWZpbGU9YGVjaG8gJCRmaWxlIHwg c2VkIHMsJChXZWJJbnNwZWN0b3JVSSkvLCxnYDsgXAorCQkJZmlsZT1gZWNobyAkJGZpbGUgfCBz ZWQgInMsJCRiYXNlcGF0aCwsZyJgOyBcCiAJCQllY2hvICIgICAgPGZpbGU+JCRmaWxlPC9maWxl PiIgPj4gJHtHRU5TT1VSQ0VTX1dFQklOU1BFQ1RPUl9VSX0vR1Jlc291cmNlQnVuZGxlLnhtbDsg XAogCQlkb25lCiAJJChBTV9WX2F0KWVjaG8gJyAgPC9ncmVzb3VyY2U+JyA+PiAke0dFTlNPVVJD RVNfV0VCSU5TUEVDVE9SX1VJfS9HUmVzb3VyY2VCdW5kbGUueG1sCg==