Bug 26323 – bisect builds does not know about Safari 4

• Status: RESOLVED
• Resolution: CONFIGURATION CHANGED
• Component: Tools / Tests

[Gavin Sherlock]

I was trying to use bisect builds (latest version downloaded as of 6/11/09) to determine if bug 26316 is a regression, but all the builds it downloaded (before I gave up) crash with Safari 4 installed, as bisect-builds doesn't know the minimum build number to use.

[Alexey Proskuryakov]

Is this still an issue?

[Gavin Sherlock]

Doesn't look like the code has been updated:

323    if (eval "v$macOSXVersion" ge v10.5) {
324	        if ($safariVersion eq "4 Public Beta") {
325	            @files = grep { $_->{rev} >= 39682 } @files;
326	        } elsif (eval "v$safariVersion" ge v3.2) {
327	            @files = grep { $_->{rev} >= 37348 } @files;
328	        } elsif (eval "v$safariVersion" ge v3.1) {
329	            @files = grep { $_->{rev} >= 29711 } @files;
330	        } elsif (eval "v$safariVersion" ge v3.0) {
331	            @files = grep { $_->{rev} >= 25124 } @files;
332	        } elsif (eval "v$safariVersion" ge v2.0) {
333	            @files = grep { $_->{rev} >= 19594 } @files;
334	        } else {
335	            die "Requires Safari 2.0 or newer";
336	        }
337	    }

It knows about the beta, but not the release version.  I believe the most recent nightly to work with Safari 4 is r40534.

[Ahmad Saleem]

Can't find reference to this code via searchfox.org and GitHub.

Should we mark this as "RESOLVED CONFIGURATION CHANGED"?

[Alexey Proskuryakov]

We've long stopped trying to maintain a compatibility list for this.