Bug 215393 - REGRESSION(r?): build-webkit --inspector-frontend always builds ImageDiff
Summary: REGRESSION(r?): build-webkit --inspector-frontend always builds ImageDiff
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: BJ Burg
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-08-11 14:34 PDT by BJ Burg
Modified: 2020-08-12 10:23 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.38 KB, patch)
2020-08-11 15:00 PDT, BJ Burg
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2020-08-11 14:34:24 PDT
--inspector-frontend is for engineers' use and should not compile non-inspector stuff.
Comment 1 Radar WebKit Bug Importer 2020-08-11 14:37:01 PDT
<rdar://problem/66862975>
Comment 2 BJ Burg 2020-08-11 15:00:17 PDT
Created attachment 406422 [details]
Patch
Comment 3 Darin Adler 2020-08-11 15:08:20 PDT
Comment on attachment 406422 [details]
Patch

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

> Tools/Scripts/build-webkit:372
> +    if (isInspectorFrontend()) {
> +        exit exitStatus($result);
> +    }

This *is* perl, so you could us the perl one-liner style.
Comment 4 Darin Adler 2020-08-11 15:08:52 PDT
Comment on attachment 406422 [details]
Patch

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

>> Tools/Scripts/build-webkit:372
>> +    }
> 
> This *is* perl, so you could us the perl one-liner style.

use

(or not, or rewrite the script in Python)
Comment 5 Jonathan Bedard 2020-08-11 16:29:01 PDT
This should be fine, WebInspector's front-end is a configuration I overlooked when adding the code Brian is skipping, this should be safe!
Comment 6 BJ Burg 2020-08-11 16:51:56 PDT
(In reply to Darin Adler from comment #4)
> Comment on attachment 406422 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=406422&action=review
> 
> >> Tools/Scripts/build-webkit:372
> >> +    }
> > 
> > This *is* perl, so you could us the perl one-liner style.
> 
> use
> 
> (or not, or rewrite the script in Python)

I have successfully resisted the urge for 8 years. 😅
Comment 7 EWS 2020-08-11 16:57:20 PDT
Committed r265537: <https://trac.webkit.org/changeset/265537>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 406422 [details].
Comment 8 Philippe Normand 2020-08-12 02:09:20 PDT
Comment on attachment 406422 [details]
Patch

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

>>>> Tools/Scripts/build-webkit:372
>>>> +    }
>>> 
>>> This *is* perl, so you could us the perl one-liner style.
>> 
>> use
>> 
>> (or not, or rewrite the script in Python)
> 
> I have successfully resisted the urge for 8 years. 😅

I haven't ;) I'm using a local Python port of build-webkit. It's very minimal though. Re-implementing all the features of the Perl version would be a large task :(
Comment 9 BJ Burg 2020-08-12 10:23:47 PDT
(In reply to Philippe Normand from comment #8)
> Comment on attachment 406422 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=406422&action=review
> 
> >>>> Tools/Scripts/build-webkit:372
> >>>> +    }
> >>> 
> >>> This *is* perl, so you could us the perl one-liner style.
> >> 
> >> use
> >> 
> >> (or not, or rewrite the script in Python)
> > 
> > I have successfully resisted the urge for 8 years. 😅
> 
> I haven't ;) I'm using a local Python port of build-webkit. It's very
> minimal though. Re-implementing all the features of the Perl version would
> be a large task :(

I don't think anyone would object to checking in what you have (as long as it's named something else and clearly marked as experimental). It's easier to improve once it's been shared ;-)