brickadia / p4-fast-reconcile

A tool to speed up "Reconcile Offline Work" in Perforce Helix Core.
MIT License
18 stars 2 forks source link

Error cleanup #3

Closed jorgenpt closed 2 months ago

jorgenpt commented 2 months ago

Some minor cleanup to a couple of error handling scenarios

Zeblote commented 2 months ago

We can't make the apple type an actual error because there is at least one of these in the Unreal Engine streams offered by Epic Games. Currently the behavior is that this file is incorrectly identified as modified, because I do not know how to compute the digest. An error in this case would make the tool unusable on such repository rather than misidentifying one file

I've so far been unable to find any documentation on what the apple thing refers to, it did not seem to match any common digests. Perhaps it is something cursed like how the utf8 file type requires you to normalize line endings before calling md5 (who comes up with that??)

jorgenpt commented 2 months ago

@Zeblote Yeah, I couldn’t figure it out at first glance, so the other pull request instead falls back to using p4 reconcile on them.