Bug 116079 – [Windows] Update ORWT to know about the Windows 8 SDK and Debug Tools

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Brent Fulgham]

Many of the Windows debugging tools referenced in our build instructions are no longer easily accessible to people just starting to create a build environment. For example, the "Debugging Tools" download package is now replaced with a "Windows 8 SDK" installer that includes the debugging tools as part of its payload.

Since these paths have changed, and most new users will not be able to run tests, we should update old-run-webkit-tests to find the debugging tools in the appropriate spots.

[Radar WebKit Bug Importer]

<rdar://problem/13881971>

[Brent Fulgham]

Created attachment 201650 [details]
Patch

[Benjamin Poulain]

Comment on attachment 201650 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=201650&action=review

> Tools/ChangeLog:24
> -2013-05-13  Zan Dobersek  <[e-mail]>
> +

Uh?

> Tools/Scripts/old-run-webkit-tests:2846
> +    my $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{PROGRAMFILES}), "Windows Kits", "8.0", "Debuggers", "x64", "ntsd.exe");
>      unless (-f $ntsdPath) {
> -        $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{ProgramW6432}), "Debugging Tools for Windows (x64)", "ntsd.exe");
> +        $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{PROGRAMFILES}), "Windows Kits", "8.0", "Debuggers", "x86", "ntsd.exe");
>          unless (-f $ntsdPath) {
> -            $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{SYSTEMROOT}), "system32", "ntsd.exe");
> +            $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{PROGRAMFILES}), "Debugging Tools for Windows (x86)", "ntsd.exe");
>              unless (-f $ntsdPath) {
> -                print STDERR "Can't find ntsd.exe. Crash logs will not be saved.\nSee <http://trac.webkit.org/wiki/BuildingOnWindows#GettingCrashLogs>.\n";
> -                return;
> +                $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{ProgramW6432}), "Debugging Tools for Windows (x64)", "ntsd.exe");
> +                unless (-f $ntsdPath) {
> +                    $ntsdPath = File::Spec->catfile(toCygwinPath($ENV{SYSTEMROOT}), "system32", "ntsd.exe");
> +                    unless (-f $ntsdPath) {
> +                        print STDERR "Can't find ntsd.exe. Crash logs will not be saved.\nSee <http://trac.webkit.org/wiki/BuildingOnWindows#GettingCrashLogs>.\n";
> +                        return;
> +                    }
> +                }
>              }
>          }
>      }

Instead of this crazy nesting, I think we should have a loop going over a list of stuff we are looking for.

> Tools/Scripts/webkitdirs.pm:1657
> +return;

Uh?

[Brent Fulgham]

Created attachment 201729 [details]
Patch

[Brent Fulgham]

Cleaned up the messy patch, and switched to a loop over the possible locations for ntsd.exe.

[David Kilzer (:ddkilzer)]

Comment on attachment 201729 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=201729&action=review

r=me

> Tools/Scripts/old-run-webkit-tests:2844
> +    my $ntsdPath = shift @possiblePaths;
> +
> +    while (not -f $ntsdPath) {
> +        if (!@possiblePaths) {
> +            print STDERR "Can't find ntsd.exe. Crash logs will not be saved.\nSee <http://trac.webkit.org/wiki/BuildingOnWindows#GettingCrashLogs>.\n";
> +            return;
>          }
> +
> +        $ntsdPath = shift @possiblePaths;

I think this might be a little clearer written as a for() loop, but I'm also fine landing this version:

my $ntsdPath;
for ($ntsdPath = shift @possiblePaths; not -f $ntsdPath; $ntsdPath = shift @possiblePaths) {
    if (…) { …}
}

[David Kilzer (:ddkilzer)]

Comment on attachment 201729 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=201729&action=review

> Tools/Scripts/old-run-webkit-tests:2834
> +    my @possiblePaths = (File::Spec->catfile(toCygwinPath($ENV{PROGRAMFILES}), "Windows Kits", "8.0", "Debuggers", "x64", "ntsd.exe"),
> +        File::Spec->catfile(toCygwinPath($ENV{PROGRAMFILES}), "Windows Kits", "8.0", "Debuggers", "x86", "ntsd.exe"),
> +        File::Spec->catfile(toCygwinPath($ENV{PROGRAMFILES}), "Debugging Tools for Windows (x86)", "ntsd.exe"),
> +        File::Spec->catfile(toCygwinPath($ENV{ProgramW6432}), "Debugging Tools for Windows (x64)", "ntsd.exe"),
> +        File::Spec->catfile(toCygwinPath($ENV{SYSTEMROOT}), "system32", "ntsd.exe"));

May be slightly easier to read if each File::Spec->catfile() is on its own line:

    my @possiblePaths = (
        File::Spec->catfile(…),
        File::Spec->catfile(…),
        File::Spec->catfile(…),
        File::Spec->catfile(…),
        File::Spec->catfile(…),
    );

But again, it's fine this way as well.

[Brent Fulgham]

Committed r150077: <http://trac.webkit.org/changeset/150077>