Bug 230288

Summary: Allow WPT exporter to know about an existing WPT checkout
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: Tools / TestsAssignee: Myles C. Maxfield <mmaxfield>
Status: NEW ---    
Severity: Normal CC: ews-watchlist, glenn, jbedard, simon.fraser, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
WIP none

Description Myles C. Maxfield 2021-09-14 18:23:44 PDT
Allow WPT exporter to know about an existing WPT checkout
Comment 1 Myles C. Maxfield 2021-09-14 18:24:48 PDT
Created attachment 438199 [details]
WIP
Comment 2 youenn fablet 2021-09-15 01:23:03 PDT
The WPT exporter is currently using a separate WPT checkout as it is adding/deleting branches, remotes and so on.
It is also checking out branches, resetting hard so it can overwrite changes in the WPT repository.
If we make use of an existing WPT checkout through webkit-patch, we should probably make sure this is ok to modify the WPT checkout, or ask user for confirmation.
Comment 3 Myles C. Maxfield 2021-09-15 03:35:30 PDT
(In reply to youenn fablet from comment #2)
> The WPT exporter is currently using a separate WPT checkout as it is
> adding/deleting branches, remotes and so on.
> It is also checking out branches, resetting hard so it can overwrite changes
> in the WPT repository.
> If we make use of an existing WPT checkout through webkit-patch, we should
> probably make sure this is ok to modify the WPT checkout, or ask user for
> confirmation.

Good idea.
Comment 4 Radar WebKit Bug Importer 2021-09-21 18:24:16 PDT
<rdar://problem/83378849>