New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 754643 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 753487
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[WPT Export] Exporter failed to export all changes; were silently reverted by later import

Project Member Reported by foolip@chromium.org, Aug 11 2017

Issue description

https://chromium-review.googlesource.com/c/582862 was exported in https://github.com/w3c/web-platform-tests/pull/6687, but the final hunk of the diff is missing. That was later reverted by import:
https://chromium-review.googlesource.com/602030

There should not have been ant conflicts, the wpt copy of the file was byte-for-byte the same before this was exported. If there was, out-of-order exports might be part of the explanation.

I will manually export the missing hunk, but we also need to understand the cause, and look to see if it's happened in other cases.
 

Comment 1 by foolip@chromium.org, Aug 11 2017

The missed hunk may also be the cause of the conflict in  issue 754616 .

Comment 2 by foolip@chromium.org, Aug 11 2017

Exporting missing bit in https://github.com/w3c/web-platform-tests/pull/6845
The hunk was added in patchset v5: https://chromium-review.googlesource.com/c/582862/4..5/third_party/WebKit/LayoutTests/external/wpt/css-fonts/variations/font-parse-numeric-stretch-style-weight.html

which was right before the CL was submitted.

Suspecting issue 753487 might be the root cause, even though the commit queue did take hours to finish and we would expect the exporter to pick up this revision. Looking for logs.
Cc: -robertma@chromium.org
Owner: robertma@chromium.org
Status: Assigned (was: Untriaged)
Mergedinto: 753487
Status: Duplicate (was: Assigned)
Unfortunately, I believe the logs have expired. The best guess now is issue 753487. Marking as dup.

Sign in to add a comment