WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
230288
Allow WPT exporter to know about an existing WPT checkout
https://bugs.webkit.org/show_bug.cgi?id=230288
Summary
Allow WPT exporter to know about an existing WPT checkout
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
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug