17691 2008-03-05 18:25:34 -0800 REGRESSION: FindSafari doesn't work 2008-06-25 14:24:43 -0700 1 1 1 Unclassified WebKit Platform 528+ (Nightly build) PC Windows XP RESOLVED FIXED PlatformOnly, Regression P1 Normal --- 1 spam_hole webkit-unassigned aroben dev+webkit grfgguvf mrowe webkit oldest_to_newest 72862 0 spam_hole 2008-03-05 18:25:34 -0800 I downloaded the very latest nightly (r30768) and found that the FindSafari.exe program doesn't start with this error: This application has failed to start because the application configuration is incorrect. Reinstalling the application may fix this problem. I have reinstalled Safari 3.0.4 and redownloaded/unpacked the package, to no avail. The older FindSafari from r30628 works fine, even with the new nightly. Also, the old one is 10.5 KB; the new one is 39.0 KB. The second issue (cosmetic) is that after starting WebKit with run-nightly-webkit.cmd, the version number as reported by Safari is still 523.15. I know I am running WebKit, though, because it gets a high score (88) on Acid3 (among other things), while regular Safari (when run directly) gets 39. 72865 1 mrowe 2008-03-05 19:15:47 -0800 How are you checking the WebKit version number? Safari's "About" box displays Safari's version number, not WebKit's, so it will be unaffected by running a nightly build. 72866 2 spam_hole 2008-03-05 19:24:59 -0800 I am using Help->About. I couldn't find out how to do it any other way. Is there an about: page for it? 72867 3 aroben 2008-03-05 19:49:47 -0800 (In reply to comment #2) > I am using Help->About. I couldn't find out how to do it any other way. Is > there an about: page for it? No, the WebKit version number is not shown anywhere in Safari's UI. It is, however, included in the user agent string, and sites like <http://whatsmyuseragent.com/> make it easy to see the contents of that string. 72868 4 spam_hole 2008-03-05 20:08:32 -0800 OK, so that's not a bug. Thanks for letting me know. The FindSafari thing is still problematic, though. 72869 5 aroben 2008-03-05 20:25:26 -0800 The r30768 nightly works just fine for me on XP Pro SP 2 (I'm using it right now to write this comment). Perhaps you should try deleting the %TMP%\WebKitNightly directory and running the nightly again? 72870 6 spam_hole 2008-03-05 20:27:55 -0800 Cleared out my temp directory. Same error, if I use the new FindSafari. 72871 7 spam_hole 2008-03-05 20:29:32 -0800 I should add that it works (but seems to crash on RSS feeds, something I can't seem to reproduce always). 72872 8 aroben 2008-03-05 20:39:19 -0800 (In reply to comment #7) > I should add that it works (but seems to crash on RSS feeds, something I can't > seem to reproduce always). Does this mean that Safari now launches and you no longer get the error dialog? 72873 9 aroben 2008-03-05 20:44:56 -0800 (In reply to comment #0) > I downloaded the very latest nightly (r30768) and found that the FindSafari.exe > program doesn't start with this error: > > This application has failed to start because the application configuration is > incorrect. Reinstalling the application may fix this problem. > > The older FindSafari from r30628 works fine, even with the new nightly. > Also, the old one is 10.5 KB; the new one is 39.0 KB. It's been suggested that this is all caused by r30394 <http://trac.webkit.org/projects/webkit/changeset/30394>, which caused FindSafari to always link against the debug C runtime. This makes sense to me, though it doesn't explain why r30628 works for you. Perhaps we should change FindSafari to always link against the release C runtime. > > The second issue (cosmetic) is that after starting WebKit with > run-nightly-webkit.cmd, the version number as reported by Safari is still > 523.15. I know I am running WebKit, though, because it gets a high score (88) > on Acid3 (among other things), while regular Safari (when run directly) gets > 39. > 72874 10 spam_hole 2008-03-05 20:50:16 -0800 Safari only launches (and works, and crashes randomly on RSS...) if the FindSafari.exe file is the one from the r30628 package (posted in the Windows nightlies archive). Simply using the run-nightly-webkit.cmd script with the newer (broken) FindSafari results in the console window appearing momentarily; no dialog error is shown, and Safari doesn't start (as verified by the Process Manager). If I add a pause statement, the output is: 1 file(s) copied. The system cannot execute the specified program. Setting environment for using Microsoft Visual Studio 2005 x86 tools. 72876 11 webkit 2008-03-05 23:08:36 -0800 I can confirm the problem. After executing a script "run-webkit-nightly.cmd" I get "the system cannot execute the specified program". 72877 12 webkit 2008-03-05 23:09:21 -0800 Marked as regression P1 bug. 72907 13 dev+webkit 2008-03-06 06:51:37 -0800 (In reply to comment #9) > (In reply to comment #0) > > I downloaded the very latest nightly (r30768) and found that the FindSafari.exe > > program doesn't start with this error: > > > > This application has failed to start because the application configuration is > > incorrect. Reinstalling the application may fix this problem. > > > > The older FindSafari from r30628 works fine, even with the new nightly. > > Also, the old one is 10.5 KB; the new one is 39.0 KB. > > It's been suggested that this is all caused by r30394 > <http://trac.webkit.org/projects/webkit/changeset/30394>, which caused > FindSafari to always link against the debug C runtime. This makes sense to me, > though it doesn't explain why r30628 works for you. > > Perhaps we should change FindSafari to always link against the release C > runtime. > I just changed it to use our release vsprops instead of the debug ones and the nightly works fine now on my non-dev Vista work box. 72908 14 19569 dev+webkit 2008-03-06 07:05:35 -0800 Created attachment 19569 s/Debug/Release 72911 15 19569 aroben 2008-03-06 07:13:52 -0800 Comment on attachment 19569 s/Debug/Release r=me 72950 16 dev+webkit 2008-03-06 16:30:46 -0800 Fix landed in r30864, it will be included in the next nightly. 72962 17 spam_hole 2008-03-06 20:15:40 -0800 FindSafari still doesn't work in r30865, giving the exact same problem as before; the file is 39.0 KB (same size as the not-working r30768 version, but the md5sums are different). The old version from r30628 still works fine. 72963 18 dev+webkit 2008-03-06 20:19:06 -0800 (In reply to comment #17) > FindSafari still doesn't work in r30865, giving the exact same problem as > before; the file is 39.0 KB (same size as the not-working r30768 version, but > the md5sums are different). > > The old version from r30628 still works fine. > It's an issue with the way the nightlies are being built/packaged. 72964 19 spam_hole 2008-03-06 20:28:26 -0800 Alright, I see. Well, I'll continue to use the old FindSafari for now, then. Is there a possibility that using this "outdated" version is resulting in my non-reproducible random crashes, or might that be something else? 72965 20 spam_hole 2008-03-06 20:28:47 -0800 Sorry, I checked the wrong box. It's not fixed. 72966 21 dev+webkit 2008-03-06 20:31:50 -0800 (In reply to comment #19) > Alright, I see. Well, I'll continue to use the old FindSafari for now, then. Is > there a possibility that using this "outdated" version is resulting in my > non-reproducible random crashes, or might that be something else? > No - there haven't been any changes to FindSafari is awhile so the "old" version you have will be identical to the "correct" one in ToT. 72967 22 mrowe 2008-03-06 20:53:00 -0800 Grab the latest nightly build (r30868). It was just fixed. 72968 23 spam_hole 2008-03-06 20:56:02 -0800 I can verify that it is fixed. Thanks! 84397 24 grfgguvf 2008-06-25 07:50:55 -0700 No it is not fixed! I have just downloaded the r34752 nightly. It still exhibits the same problem on Windows XP! 84434 25 mrowe 2008-06-25 14:00:41 -0700 Given that this bug report was closed *three months ago*, any issue that you are seeing should be filed as a separate bug report as it's likely to be an unrelated issue. 84435 26 grfgguvf 2008-06-25 14:24:43 -0700 Oops... Not having Safari installed results in the same symptoms as this bug. The WebKit nightly download site should tell the rest of us that it needs a Safari installation, I think... 19569 2008-03-06 07:05:35 -0800 2008-03-06 07:13:52 -0800 s/Debug/Release Bug17691 text/plain 3667 dev+webkit SW5kZXg6IFdlYktpdC93aW4vQ2hhbmdlTG9nDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQotLS0gV2ViS2l0L3dpbi9D aGFuZ2VMb2cJKHJldmlzaW9uIDMwODI3KQ0KKysrIFdlYktpdC93aW4vQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpDQpAQCAtMSwzICsxLDEyIEBADQorMjAwOC0wMy0wNiAgTWF0dCBMaWxlayAgPHdl YmtpdEBtYXR0bGlsZWsuY29tPg0KKw0KKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuDQorDQorICAgICAgICBDaGFuZ2UgRmluZFNhZmFyaSB0byBsaW5rIGFnYWluc3QgdGhlIHJl bGVhc2UgbGlicmFyaWVzIGluc3RlYWQgb2YgZGVidWcNCisgICAgICAgIGFzIHNvbWUgbWFjaGlu ZXMgZG8gbm90IGhhdmUgdGhlIGRlYnVnIG9uZXMgaW5zdGFsbGVkLg0KKw0KKyAgICAgICAgKiBX ZWJLaXQudmNwcm9qL1dlYktpdC5zbG46DQorDQogMjAwOC0wMy0wNSAgQW5kZXJzIENhcmxzc29u ICA8YW5kZXJzY2FAYXBwbGUuY29tPg0KIA0KICAgICAgICAgQnVpbGQgZml4Lg0KSW5kZXg6IFdl YktpdC93aW4vV2ViS2l0LnZjcHJvai9XZWJLaXQuc2xuDQo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQotLS0gV2ViS2l0 L3dpbi9XZWJLaXQudmNwcm9qL1dlYktpdC5zbG4JKHJldmlzaW9uIDMwNzYwKQ0KKysrIFdlYktp dC93aW4vV2ViS2l0LnZjcHJvai9XZWJLaXQuc2xuCSh3b3JraW5nIGNvcHkpDQpAQCAtMTQ0LDEy ICsxNDQsMTIgQEANCiAJCXtBQThBNUE4NS01OTJCLTQzNTctQkM2MC1FMEU5MUUwMjZBRjZ9LkRl YnVnfFdpbjMyLkJ1aWxkLjAgPSBEZWJ1Z3xXaW4zMg0KIAkJe0FBOEE1QTg1LTU5MkItNDM1Ny1C QzYwLUUwRTkxRTAyNkFGNn0uUmVsZWFzZXxXaW4zMi5BY3RpdmVDZmcgPSBSZWxlYXNlfFdpbjMy DQogCQl7QUE4QTVBODUtNTkyQi00MzU3LUJDNjAtRTBFOTFFMDI2QUY2fS5SZWxlYXNlfFdpbjMy LkJ1aWxkLjAgPSBSZWxlYXNlfFdpbjMyDQotCQl7REEzMURBNTItNjY3NS00OEQ0LTg5RTAtMzMz QTcxNDQzOTdDfS5EZWJ1Z19JbnRlcm5hbHxXaW4zMi5BY3RpdmVDZmcgPSBEZWJ1Z3xXaW4zMg0K LQkJe0RBMzFEQTUyLTY2NzUtNDhENC04OUUwLTMzM0E3MTQ0Mzk3Q30uRGVidWdfSW50ZXJuYWx8 V2luMzIuQnVpbGQuMCA9IERlYnVnfFdpbjMyDQotCQl7REEzMURBNTItNjY3NS00OEQ0LTg5RTAt MzMzQTcxNDQzOTdDfS5EZWJ1Z3xXaW4zMi5BY3RpdmVDZmcgPSBEZWJ1Z3xXaW4zMg0KLQkJe0RB MzFEQTUyLTY2NzUtNDhENC04OUUwLTMzM0E3MTQ0Mzk3Q30uRGVidWd8V2luMzIuQnVpbGQuMCA9 IERlYnVnfFdpbjMyDQotCQl7REEzMURBNTItNjY3NS00OEQ0LTg5RTAtMzMzQTcxNDQzOTdDfS5S ZWxlYXNlfFdpbjMyLkFjdGl2ZUNmZyA9IERlYnVnfFdpbjMyDQotCQl7REEzMURBNTItNjY3NS00 OEQ0LTg5RTAtMzMzQTcxNDQzOTdDfS5SZWxlYXNlfFdpbjMyLkJ1aWxkLjAgPSBEZWJ1Z3xXaW4z Mg0KKwkJe0RBMzFEQTUyLTY2NzUtNDhENC04OUUwLTMzM0E3MTQ0Mzk3Q30uRGVidWdfSW50ZXJu YWx8V2luMzIuQWN0aXZlQ2ZnID0gUmVsZWFzZXxXaW4zMg0KKwkJe0RBMzFEQTUyLTY2NzUtNDhE NC04OUUwLTMzM0E3MTQ0Mzk3Q30uRGVidWdfSW50ZXJuYWx8V2luMzIuQnVpbGQuMCA9IFJlbGVh c2V8V2luMzINCisJCXtEQTMxREE1Mi02Njc1LTQ4RDQtODlFMC0zMzNBNzE0NDM5N0N9LkRlYnVn fFdpbjMyLkFjdGl2ZUNmZyA9IFJlbGVhc2V8V2luMzINCisJCXtEQTMxREE1Mi02Njc1LTQ4RDQt ODlFMC0zMzNBNzE0NDM5N0N9LkRlYnVnfFdpbjMyLkJ1aWxkLjAgPSBSZWxlYXNlfFdpbjMyDQor CQl7REEzMURBNTItNjY3NS00OEQ0LTg5RTAtMzMzQTcxNDQzOTdDfS5SZWxlYXNlfFdpbjMyLkFj dGl2ZUNmZyA9IFJlbGVhc2V8V2luMzINCisJCXtEQTMxREE1Mi02Njc1LTQ4RDQtODlFMC0zMzNB NzE0NDM5N0N9LlJlbGVhc2V8V2luMzIuQnVpbGQuMCA9IFJlbGVhc2V8V2luMzINCiAJCXs1OUND MDU0Ny03MEFDLTQ5OUMtOUIxOS1FQzAxQzZGNjExMzd9LkRlYnVnX0ludGVybmFsfFdpbjMyLkFj dGl2ZUNmZyA9IERlYnVnfFdpbjMyDQogCQl7NTlDQzA1NDctNzBBQy00OTlDLTlCMTktRUMwMUM2 RjYxMTM3fS5EZWJ1Z19JbnRlcm5hbHxXaW4zMi5CdWlsZC4wID0gRGVidWd8V2luMzINCiAJCXs1 OUNDMDU0Ny03MEFDLTQ5OUMtOUIxOS1FQzAxQzZGNjExMzd9LkRlYnVnfFdpbjMyLkFjdGl2ZUNm ZyA9IERlYnVnfFdpbjMyDQpJbmRleDogV2ViS2l0VG9vbHMvQ2hhbmdlTG9nDQo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 DQotLS0gV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCShyZXZpc2lvbiAzMDgyNykNCisrKyBXZWJLaXRU b29scy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkNCkBAIC0xLDMgKzEsMTIgQEANCisyMDA4LTAz LTA2ICBNYXR0IExpbGVrICA8d2Via2l0QG1hdHRsaWxlay5jb20+DQorDQorICAgICAgICBSZXZp ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4NCisNCisgICAgICAgIENoYW5nZSBGaW5kU2FmYXJpIHRv IGxpbmsgYWdhaW5zdCB0aGUgcmVsZWFzZSBsaWJyYXJpZXMgaW5zdGVhZCBvZiBkZWJ1Zw0KKyAg ICAgICAgYXMgc29tZSBtYWNoaW5lcyBkbyBub3QgaGF2ZSB0aGUgZGVidWcgb25lcyBpbnN0YWxs ZWQuDQorDQorICAgICAgICAqIEZpbmRTYWZhcmkvRmluZFNhZmFyaS52Y3Byb2o6DQorDQogMjAw OC0wMy0wNSAgQWxwIFRva2VyICA8YWxwQGF0b2tlci5jb20+DQogDQogICAgICAgICBBdHRlbXB0 IHRvIGdldCB0aGUgYXV0b3Rvb2xzIGJ1aWxkIHdvcmtpbmcgYWdhaW4gb24gdGhlIGJvdCB3aXRo DQpJbmRleDogV2ViS2l0VG9vbHMvRmluZFNhZmFyaS9GaW5kU2FmYXJpLnZjcHJvag0KPT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PQ0KLS0tIFdlYktpdFRvb2xzL0ZpbmRTYWZhcmkvRmluZFNhZmFyaS52Y3Byb2oJKHJldmlz aW9uIDMwNzYwKQ0KKysrIFdlYktpdFRvb2xzL0ZpbmRTYWZhcmkvRmluZFNhZmFyaS52Y3Byb2oJ KHdvcmtpbmcgY29weSkNCkBAIC0xNSw5ICsxNSw5IEBADQogCTwvVG9vbEZpbGVzPg0KIAk8Q29u ZmlndXJhdGlvbnM+DQogCQk8Q29uZmlndXJhdGlvbg0KLQkJCU5hbWU9IkRlYnVnfFdpbjMyIg0K KwkJCU5hbWU9IlJlbGVhc2V8V2luMzIiDQogCQkJQ29uZmlndXJhdGlvblR5cGU9IjEiDQotCQkJ SW5oZXJpdGVkUHJvcGVydHlTaGVldHM9IiQoV2ViS2l0TGlicmFyaWVzRGlyKVx0b29sc1x2c3By b3BzXGNvbW1vbi52c3Byb3BzOyQoV2ViS2l0TGlicmFyaWVzRGlyKVx0b29sc1x2c3Byb3BzXGRl YnVnLnZzcHJvcHM7JChXZWJLaXRMaWJyYXJpZXNEaXIpXHRvb2xzXHZzcHJvcHNcZGVidWdfaW50 ZXJuYWwudnNwcm9wcyINCisJCQlJbmhlcml0ZWRQcm9wZXJ0eVNoZWV0cz0iJChXZWJLaXRMaWJy YXJpZXNEaXIpXHRvb2xzXHZzcHJvcHNcY29tbW9uLnZzcHJvcHM7JChXZWJLaXRMaWJyYXJpZXNE aXIpXHRvb2xzXHZzcHJvcHNccmVsZWFzZS52c3Byb3BzIg0KIAkJCUNoYXJhY3RlclNldD0iMSIN CiAJCQk+DQogCQkJPFRvb2wNCg==