Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating diffs for some files fails #1101

Closed
joshaber opened this issue Jul 14, 2017 · 3 comments
Closed

Creating diffs for some files fails #1101

joshaber opened this issue Jul 14, 2017 · 3 comments

Comments

@joshaber
Copy link

This might be a relatively recently introduced bug.

We're trying to enable delta updates for GitHub Desktop (desktop/desktop#2239). With Squirrel.Windows 1.5.2, it works. With Squirrel.Windows 1.7.6, an exception is thrown creating deltas for some files:

DeltaPackageBuilder: We really couldn't create a delta for package.json: System.DivideByZeroException: Attempted to divide by zero.
   at Squirrel.Bsdiff.BinaryPatchUtility.Create(Byte[] oldData, Byte[] newData, Stream output)
   at Squirrel.DeltaPackageBuilder.createDeltaForSingleFile(FileInfo targetFile, DirectoryInfo workingDirectory, Dictionary`2 baseFileListing)

@Keboo mentioned hitting this same bug in #1087.

@anaisbetts
Copy link
Contributor

I think we're just gonna revert all of these bsdiff changes, they're breaking stuff

@Thieum
Copy link
Contributor

Thieum commented May 3, 2019

@joshaber are you still experiencing this issue with the latest version of Squirrel?

@shiftkey
Copy link
Contributor

shiftkey commented May 4, 2019

I'm tracking desktop/desktop#5197 to upgrade our version of electron-winstaller (that uses Squirrel) and I'll report back if we still see this issue later on.

@shiftkey shiftkey closed this as completed May 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants