Page MenuHomePhabricator

sidedata: send the correct revision data for wireproto v2
ClosedPublic

Authored by joerg.sonnenberger on Nov 28 2020, 9:04 AM.

Details

Summary

When no sidedata is present, rawdata() and revision() are the same. But
as soon as sidedata is present, the way it is currently stored will
change the rawdata and that is not desired here, so switch to the
correct data accessor.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Alphare accepted this revision.Dec 7 2020, 5:18 AM
pulkit accepted this revision.Dec 7 2020, 8:45 AM
This revision is now accepted and ready to land.Dec 7 2020, 8:45 AM

I think this will behave badly if any other processing flag is involved
(something like lfs, which thankfully is not used on changelog
revision). What happens to the associated flags ? where they preserved
by wireprotov2 in the first place ?

I don't think they are. The main motivation for pushing this one is to ensure that later changes to result in more sidedata doesn't create spurious failures with wireproto v2. As discussed on IRC around the sprint, it is dead at this point, but I'd still want to avoid introducing extra breakage.