137450 2014-10-06 09:36:46 -0700 Fix Windows dependency tracking related to JSInternals.h/cpp 2014-10-22 02:47:53 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 ossy webkit-unassigned bfulgham lforschler ossy rniwa roger_fong thorton oldest_to_newest 1039743 0 ossy 2014-10-06 09:36:46 -0700 Windows EWS were stucked for 2.5 days after http://trac.webkit.org/changeset/174315 with this error: 1>c1xx : fatal error C1083: Cannot open source file: 'C:\cygwin\home\buildbot\WebKit\WebKitBuild\Release\obj32\WebCore\DerivedSources\JSInternals.cpp': No such file or directory 1>..\testing\js\WebCoreTestSupport.cpp(33): fatal error C1083: Cannot open include file: 'JSInternals.h': No such file or directory 1>Done Building Project "C:\cygwin\home\buildbot\WebKit\Source\WebCore\WebCore.vcxproj\WebCoreTestSupport.vcxproj" (Build target(s)) -- FAILED. Buildfix attempt / workaround landed in r174344. (Similar patch landed in r144570 to fix a similar strange Win failure.) 1039762 1 ossy 2014-10-06 10:39:02 -0700 It seems r174344 didn't help. It would be great if anybody from Apple can fix this bug to unbreak the Windows EWS bots. 1040002 2 ossy 2014-10-07 05:26:46 -0700 One more attempt from Roger - http://trac.webkit.org/changeset/174378 , but the EWS is still broken. 1040838 3 ossy 2014-10-10 00:42:54 -0700 Fixed by https://trac.webkit.org/changeset/174541. Just out of curiosity, could you tell us how is it possible that the problem happened on the EWS bots only, but not on the buildbots? What was the exact problem why we needed to move the enum declaration to the top of the file? If it is design problem in the generator or if it is a Windows issue, we should add a new rule to the idl style checker to force folks to put the enum declaration to the proper place. 1040840 4 thorton 2014-10-10 00:48:09 -0700 Ha! Yeah, I'd really like to understand this better. 1041827 5 ossy 2014-10-15 02:09:28 -0700 (In reply to comment #2) > One more attempt from Roger - http://trac.webkit.org/changeset/174378 , > but the EWS is still broken. Roger, could you explain us, why this unreviewed patch fixed the issue? What was the problem, and why was this the proper fix? 1041828 6 ossy 2014-10-15 02:10:09 -0700 (In reply to comment #5) > (In reply to comment #2) > > One more attempt from Roger - http://trac.webkit.org/changeset/174378 , > > but the EWS is still broken. > > Roger, could you explain us, why this unreviewed patch fixed the issue? > What was the problem, and why was this the proper fix? I meant r174541. 1042116 7 ossy 2014-10-16 06:50:37 -0700 ping 1042681 8 ossy 2014-10-19 03:41:46 -0700 Roger, please don't ignore our reqeust to explain why your unreviewed and uncommented patch - http://trac.webkit.org/changeset/174541 fixed the Windows issue. Please answer ASAP. Thanks. 1042925 9 roger_fong 2014-10-20 12:28:30 -0700 (In reply to comment #8) > Roger, please don't ignore our reqeust to explain why your unreviewed > and uncommented patch - http://trac.webkit.org/changeset/174541 fixed > the Windows issue. Please answer ASAP. Thanks. Apologies, I've neglected my bugzilla inbox recently. Added a few too many people to my watch list and it's been getting ultra flooded. Unfortunately I don't have a particularly good answer for you. The only hunch that I have is that it might have to do with needing to upgrade the vs2013 express service packs on those machines but I found my actual solution via a sort of last resort guess and check. The next chance I get I'll try to upgrade those bots (which I have to manually right now), locally revert my fix, and see if the problem still occurs. 1043395 10 ossy 2014-10-22 02:47:53 -0700 (In reply to comment #9) > (In reply to comment #8) > > Roger, please don't ignore our reqeust to explain why your unreviewed > > and uncommented patch - http://trac.webkit.org/changeset/174541 fixed > > the Windows issue. Please answer ASAP. Thanks. > > Apologies, I've neglected my bugzilla inbox recently. Added a few too many > people to my watch list and it's been getting ultra flooded. No problem. You can setup a more sophisticated mail filter for bugzilla mails to separate important / non-important mails. There are many reason why do you get a bugzilla mail, you can find them at the end of the mail: "You are receiving this mail because:" Here is the list of these reasons: http://trac.webkit.org/browser/trunk/Websites/bugs.webkit.org/template/en/default/global/reason-descs.none.tmpl?order=name#L20 > Unfortunately I don't have a particularly good answer for you. > The only hunch that I have is that it might have to do with needing to > upgrade the vs2013 express service packs on those machines but I found my > actual solution via a sort of last resort guess and check. The next chance I > get I'll try to upgrade those bots (which I have to manually right now), > locally revert my fix, and see if the problem still occurs. It's strange. Maybe not r174541 fixed the issue, but some tweak on the servers. Otherwise it would be great to have exactly the same configuration on the EWS and the buildbots to avoid similar issues in the future.