Bug 169522 - REGRESSION (r211657): bisect-builds doesn't work
Summary: REGRESSION (r211657): bisect-builds doesn't work
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-11 22:21 PST by Alexey Proskuryakov
Modified: 2017-05-15 16:08 PDT (History)
3 users (show)

See Also:


Attachments
fix (doesn't make it work) (2.53 KB, patch)
2017-03-11 22:42 PST, Alexey Proskuryakov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2017-03-11 22:21:39 PST
$ bisect-builds 
Undefined subroutine &main::safariPathFromSafariBundle called at Tools/Scripts/bisect-builds line 132.
Comment 1 mitz 2017-03-11 22:23:04 PST
For what it’s worth, I’m not sure it was working prior to that revision either (I think it was just failing differently).
Comment 2 Alexey Proskuryakov 2017-03-11 22:42:43 PST
Created attachment 304189 [details]
fix (doesn't make it work)

This looks like what it should take to fix it, but the script still doesn't work:

$ bisect-builds 
Requires Mac OS X 10.4 (Tiger) or later at /Users/ap/Safari/OpenSource/Tools/Scripts/bisect-builds line 391.

I don't understand the code that produces this error. Confirmed that $osxVersionString and $safariVersionString are as expected (10.12.3 and 10.0.3).
Comment 3 mitz 2017-03-11 22:44:40 PST
Looks like the script hasn’t been updated for macOS Sierra.
Comment 4 Alexey Proskuryakov 2017-03-11 22:45:18 PST
Comment on attachment 304189 [details]
fix (doesn't make it work)

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

> Tools/Scripts/bisect-builds:75
> +my $safariPath = safariPath();

Actually, "/Applications/Safari.app/Contents/MacOS/SafariForWebKitDevelopment" may be even more appropriate here. When bisecting with nightlies, we don't want a locally built Safari.
Comment 5 Alexey Proskuryakov 2017-03-11 22:56:13 PST
> Looks like the script hasn’t been updated for macOS Sierra.

Good point. Adding the support still doesn't make it work though. I confirmed that this nightly build works when opened manually.

    if ($osxVersion ge v10.12 && $osxVersion lt v10.13) {
        @files = grep { $_->{rev} >= 205582 } @files;
    } elsif ($osxVersion ge v10.11 && $osxVersion lt v10.12) {

$ bisect-builds

Works: r205582  Fails: r213762

Checking revision r209345 (200 builds left to test after this)...
Downloading WebKit-SVN-r209345.dmg to /Users/ap/Library/Caches/WebKit-Nightlies/trunk...
######################################################################## 100.0%
Mounting disk image and running WebKit...
LSOpenURLsWithRole() failed for the application /Applications/Safari.app/Contents/MacOS/SafariForWebKitDevelopment with error -10810.
Did the bug reproduce in r209345 (yes/no/retry/broken)? ^C
Comment 6 Alexey Proskuryakov 2017-03-11 22:57:08 PST
"This" being r209345. I didn't confirm whether r205582 works, I just know that this is when the nightlies got larger.
Comment 7 mitz 2017-03-11 22:59:24 PST
(In reply to comment #5)

> LSOpenURLsWithRole() failed for the application
> /Applications/Safari.app/Contents/MacOS/SafariForWebKitDevelopment with
> error -10810.

This is probably because $isBundle is true even thought the above path isn’t a bundle.