RFR: 1236: Add jcheck option to check for large binary files [v3]

Erik Joelsson erikj at openjdk.java.net
Mon Dec 6 16:54:19 UTC 2021

On Mon, 6 Dec 2021 16:41:59 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> > I think size delta in the case of modified is fine. In my experience, it's very rare for binary patches to actually be partial in real situations. When replacing an image or archive, the whole file usually changes enough. If the binary patch is empty, due to file moved/rename, then we can't really do anything about that unfortunately. Have you looked at actual patch output from Git to see what it looks like for renamed binary files? Git doesn't track file renames explicitly, so I'm not sure what would actually happen here.
> Moved file is identifed as `renamed`. The `renamed` diff information is shown below.
> ```
> :100644 100644 c9eef64 c9eef64 R100     t/1.png t/2.png
> diff --git a/t/1.png b/t/2.png
> similarity index 100%
> rename from t/1.png
> rename to t/2.png
> ```
> It only has the source file and target file and have no information about size.

Then we can't check the size of files in such changes. We are limited to what Git will tell us here.


PR: https://git.openjdk.java.net/skara/pull/1247

More information about the skara-dev mailing list