Summary: | [Mac] REGRESSION (r164823): Building JavaScriptCore creates files under /tmp/JavaScriptCore.dst | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> | ||||||||
Component: | New Bugs | Assignee: | Csaba Osztrogonác <ossy> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Blocker | CC: | commit-queue, ddkilzer, joepeck, mitz, mrowe, ossy | ||||||||
Priority: | P1 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=232940 | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 129399 | ||||||||||
Attachments: |
|
Description
Csaba Osztrogonác
2014-04-23 02:45:45 PDT
Created attachment 229969 [details] Patch I propose to revert r164823 and r164828, because now build-webkit always try to remove and recreate a broken and useless symlink unconditionally. It is a huge regression that now only one user can build WebKit on a Mac. Comment on attachment 229969 [details]
Patch
Rolling those changes out reintroduces a much more serious issue. This should be trivial to fix properly, so let's just do that instead.
(In reply to comment #2) > (From update of attachment 229969 [details]) > Rolling those changes out reintroduces a much more serious issue. This should be trivial to fix properly, so let's just do that instead. Could you give us any hint how can it be fixed properly? Created attachment 229970 [details]
Patch
Created attachment 229971 [details]
Patch
(In reply to comment #5) > Created an attachment (id=229971) [details] > Patch Thanks for the patch, it fixes the multi user issue I mentioned in the description. But I don't have expertise in Mac deployment, I let others to review the fix. Comment on attachment 229971 [details] Patch Clearing flags on attachment: 229971 Committed r167709: <http://trac.webkit.org/changeset/167709> All reviewed patches have been landed. Closing bug. |