"Tool/Scripts/run-launcher --gtk" stopped working after 74855 (JavaScriptCore/ moved to Source/JavaScriptCore) should fix Tools/Scripts/webkitdirs.pm
Created attachment 77790 [details] proposed patch this is kinda trivial
(In reply to comment #1) > Created an attachment (id=77790) [details] > proposed patch > > this is kinda trivial I must admit I don't get what the patch is supposed to achieve or how it's related to moving JSC to Source/. Can you explain?
(In reply to comment #2) > (In reply to comment #1) > > Created an attachment (id=77790) [details] [details] > > proposed patch > > > > this is kinda trivial > > I must admit I don't get what the patch is supposed to achieve or how it's related to moving JSC to Source/. Can you explain? From talking to Philippe (who also sees the brokeness), it seems that the problem is that this is called with JavaScriptCore as $libraryName, which does not exist now as a directory, so the whole thing will fail. I'm not sure what was the point of doing $dir/$foo/../.libs, but I think just $dir/.libs is OK as you suggest. I wonder if this has changed recently, our solution (and Qt, Elf,...) does not seem so robust anymore.
Comment on attachment 77790 [details] proposed patch The patch is good, but you need to explain a bit more what's going on in the ChangeLog. Thanks!
it used to work before JavascriptCore moved to Source/: /path/to/WebKitBuild/JavascriptCore/../.libs existed but now indeed: /path/to/WebkitBuild/Source/JavascriptCore/../.libs doesn't exist. Using ../../.libs would work but I agree with Xan that the proposed patch is better
(In reply to comment #5) > it used to work before JavascriptCore moved to Source/: > > /path/to/WebKitBuild/JavascriptCore/../.libs existed > > but now indeed: > > /path/to/WebkitBuild/Source/JavascriptCore/../.libs doesn't exist. > > Using ../../.libs would work but I agree with Xan that the proposed patch is better yeah, I don't know why that was "JavascriptCore/../" which doesn't really make sense to me, so i removed it
*** Bug 51816 has been marked as a duplicate of this bug. ***
Created attachment 77804 [details] proposed patch add some explanation as Xan suggested
Comment on attachment 77804 [details] proposed patch Thank you!
The commit-queue encountered the following flaky tests while processing attachment 77804 [details]: http/tests/xmlhttprequest/access-control-preflight-credential-async.html bug 51817 (author: jchaffraix@webkit.org) The commit-queue is continuing to process your patch.
Comment on attachment 77804 [details] proposed patch Clearing flags on attachment: 77804 Committed r74889: <http://trac.webkit.org/changeset/74889>
All reviewed patches have been landed. Closing bug.