Bug 230288 - Allow WPT exporter to know about an existing WPT checkout
Summary: Allow WPT exporter to know about an existing WPT checkout
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-09-14 18:23 PDT by Myles C. Maxfield
Modified: 2021-09-21 18:24 PDT (History)
6 users (show)

See Also:

Attachments
WIP (1.77 KB, patch)
2021-09-14 18:24 PDT, Myles C. Maxfield 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>