Bug 230288

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

Myles C. Maxfield
Reported 2021-09-14 18:23:44 PDT
Allow WPT exporter to know about an existing WPT checkout
Attachments
WIP (1.77 KB, patch)
2021-09-14 18:24 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2021-09-14 18:24:48 PDT
Created attachment 438199 [details] WIP
youenn fablet
Comment 2 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.
Myles C. Maxfield
Comment 3 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.
Radar WebKit Bug Importer
Comment 4 2021-09-21 18:24:16 PDT
<rdar://problem/83378849>
Sam Sneddon [:gsnedders]
Comment 5 2025-08-26 10:37:53 PDT
*** Bug 277444 has been marked as a duplicate of this bug. ***
Sam Sneddon [:gsnedders]
Comment 6 2025-08-26 10:40:22 PDT
I believe this is effectively fixed by https://github.com/WebKit/WebKit/pull/48812. Brianna?
Note You need to log in before you can comment on or make changes to this bug.