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

Fix linux detection #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix linux detection #11

wants to merge 1 commit into from

Conversation

ferferga
Copy link

Linux wasn't detected correctly in my Raspberry Pi device as the returned string in OSTYPE is linux-gnueabihf instead of linux-gnu

Removed also the author block as I saw that there are other contributions from other people throughout the commit history, so the authors of the code are not only you but also @incrementing and @VarunBatraIT. People looking for the full history of contributions can directly check in https://github.com/zevilz/zImageOptimizer/graphs/contributors

@ferferga
Copy link
Author

ferferga commented Jun 1, 2021

Hello @zevilz

Any news about this?

@VarunBatraIT
Copy link
Contributor

I wouldn't mind his signature on Author name. He is already taking full responsibility for this repository. At least my contribution was not a lot nor I think I am an expert in answering any questions regarding this repository. Next, Linux detection codes look good to me.

@zevilz zevilz closed this Jul 8, 2024
@ferferga
Copy link
Author

ferferga commented Jul 8, 2024

@zevilz Why this was closed? The issue this is aiming to fix still exists.

If it's for the copyright thing, why not tell me so I can drop that commit?

@zevilz zevilz reopened this Jul 8, 2024
@ferferga
Copy link
Author

ferferga commented Jul 8, 2024

@zevilz Dropped the commit and rebased with latest master

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

Successfully merging this pull request may close these issues.

3 participants