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

Lintian reports #10

Open
szepeviktor opened this issue Jan 6, 2023 · 10 comments
Open

Lintian reports #10

szepeviktor opened this issue Jan 6, 2023 · 10 comments

Comments

@szepeviktor
Copy link
Contributor

szepeviktor commented Jan 6, 2023

lintian --display-info --display-experimental --pedantic --show-overrides php8.2-relay-0.5.1-php8.2-debian_amd64.deb

kép

I know FPM is convenient but I'm a Debian user.
@tillkruss spelling-error-in-binary errors are interesting :)
Is there a public changelog for the closed-source PHP extension?

@tillkruss
Copy link
Member

We publish all changes here: https://github.com/cachewerk/relay/releases

@michael-grunder
Copy link
Member

@tillkruss spelling-error-in-binary errors are interesting :)

For the life of me I can't stop spelling retrieve wrong 😂

@tillkruss
Copy link
Member

See #3 and #4 as well @szepeviktor

@szepeviktor
Copy link
Contributor Author

For the life of me I can't stop spelling retrieve wrong 😂

It was necessary for me 😃

@szepeviktor
Copy link
Contributor Author

szepeviktor commented Jan 6, 2023

@tillkruss I would gladly check every byte you enter into the computer in my spare time, that is ~24 hours/day nowadays.

@tillkruss
Copy link
Member

We don't have any references of certifcate or lengH in our code base, it must be coming from a bundled library.

Screenshot 2023-01-06 at 12 02 29 PM

@szepeviktor
Copy link
Contributor Author

it must be coming from a bundled library

You can ask them to correct it. 👉🏻 redis/hiredis#1154

Failed to open system certifcate store

@szepeviktor
Copy link
Contributor Author

szepeviktor commented Jan 6, 2023

or lengH in our code base

That message is cut in three in the binary.
https://github.com/redis/hiredis/blob/acd09461d3c796751a0ea07ea69d81e7ce264320/read.c#L418

@szepeviktor
Copy link
Contributor Author

@tillkruss What tool do you use in the above comment??

@tillkruss
Copy link
Member

codespell

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

3 participants