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

libcperciva import, fix Makefile.am, and fix test #394

Merged
merged 3 commits into from
Dec 23, 2024
Merged

libcperciva import, fix Makefile.am, and fix test #394

merged 3 commits into from
Dec 23, 2024

Conversation

gperciva
Copy link
Member

No description provided.

@gperciva gperciva changed the title libcperciva import and fix Makefile.am libcperciva import, fix Makefile.am, and fix test Dec 21, 2024
@gperciva
Copy link
Member Author

There was one report of this test failing because it successfully decrypted the 10-second file in 1 second [*]. This was on a Debian build server, so I suspect that the system was heavily loaded while it created the 10-second file, so scrypt enc underestimated the parameters.

[*] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1086624

Ideally, there would always be 0 errors.   However, due to porting
issues, some platforms report a non-zero number of errors even for
trivial programs.
Same rationale as:
    2022-10-03 build: add missing files to EXTRA_DIST
    af5fed9
There was one report of this test failing because it successfully
decrypted the 10-second file in 1 second [*].  This was on a Debian
build server, so I suspect that the system was heavily loaded while it
created the 10-second file, so `scrypt enc` underestimated the
parameters.

[*] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1086624

Reported by:	Sebastian Ramacher
@cperciva cperciva merged commit 2bbd6dc into master Dec 23, 2024
2 checks passed
@gperciva gperciva deleted the misc branch December 23, 2024 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants