Summary: | [Qt] Incremental build problem revealed by r102981 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> | ||||
Component: | Tools / Tests | Assignee: | Ádám Kallai <kadam> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, eric, ossy, rniwa, vestbo, webkit.review.bot | ||||
Priority: | P2 | Keywords: | Qt, QtTriaged | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 76528 | ||||||
Attachments: |
|
Description
Csaba Osztrogonác
2011-12-15 22:47:55 PST
Created attachment 123550 [details]
proposed patch
It is necessary to set mathattrs.in dependency for generating MathMLNames.cpp file. I added depend this file /Source/WebCore/DerivedSources.pri.
Comment on attachment 123550 [details]
proposed patch
While the fix is correct it unfortunately doesn't solve the general problem, for exxample is $$PWD/mathml/mathtags.in still left out of depends. And there's a bunch of other generators that have the same problem.
The better approach I think would be to add code in default_post.prf that walks every argument of commands and checks if it's a file that exists, and if so, adds it to depends. That should ideally fix it for all of these cases.
Feel free to have a stab at it, or assign the bug to me and I'll have a look. Thanks!
(In reply to comment #2) > (From update of attachment 123550 [details]) > While the fix is correct it unfortunately doesn't solve the general problem, for exxample is $$PWD/mathml/mathtags.in still left out of depends. And there's a bunch of other generators that have the same problem. The correct is absolutely correct for this bug. And $$PWD/mathml/mathtags.in isn't missing from the dependency list, because MATHML_NAMES is $$PWD/mathml/mathtags.in which is the input of this generator. > The better approach I think would be to add code in default_post.prf that walks every argument of commands and checks if it's a file that exists, and if so, adds it to depends. That should ideally fix it for all of these cases. > Feel free to have a stab at it, or assign the bug to me and I'll have a look. Thanks! The idea is good, but I'm not sure if it works always. Let's try it. But I don't think if we shouldn't land this trivial, necssarry and correct fix because maybe we can do a general fix sometimes in the future. I filed a new bug report for this future development: https://bugs.webkit.org/show_bug.cgi?id=76902 (In reply to comment #3) > The correct is absolutely correct for this bug. And $$PWD/mathml/mathtags.in isn't missing from the dependency list, because MATHML_NAMES is $$PWD/mathml/mathtags.in which is the input of this generator. I should have reread my comment. :) s/The correct/The fix/g Comment on attachment 123550 [details]
proposed patch
r=me to decrease the number of possible incremental build bugs in the future.
Comment on attachment 123550 [details] proposed patch Clearing flags on attachment: 123550 Committed r105731: <http://trac.webkit.org/changeset/105731> All reviewed patches have been landed. Closing bug. |