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

Pi-Apps installer broken #201

Closed
Botspot opened this issue Dec 13, 2023 · 4 comments
Closed

Pi-Apps installer broken #201

Botspot opened this issue Dec 13, 2023 · 4 comments
Assignees
Labels

Comments

@Botspot
Copy link

Botspot commented Dec 13, 2023

Pi-Apps runs this command to install piKiss:

curl -sSL https://git.io/JfAPE | bash || error 'Failed to install piKiss!'

Your commit here, "👌 IMPROVE: run piKiss.sh when install it" broke the installer.
Now when installing PiKISS from Pi-Apps, users are greeted with the pikiss interface. This is bad because installs should be automatic and noninteractive.
Also, your internet connection check is failing on some systems. We have had two error reports sent in where pikiss installs fine but once it is run, your check_internet_available function incorrectly detects a lost internet connection.

This issue is requesting three changes:

  1. To avoid running piKiss.sh at the end of installation, at least for pi-apps users.
  2. To improve your check_internet_available function.
  3. To verify that in the future, your changes do not break pi-apps again. This is the second time it was happened in the past few weeks (the other occasion was for issue Not installing on Bookworm 64-bit today #199)
@Botspot
Copy link
Author

Botspot commented Dec 13, 2023

Attached are the two logfiles we received.
install-fail-piKiss (3).txt
install-fail-piKiss (2).txt

@jmcerrejon jmcerrejon self-assigned this Dec 13, 2023
@jmcerrejon jmcerrejon added the bug label Dec 13, 2023
@jmcerrejon
Copy link
Owner

I don't know when was the moment in the time PiKISS depended on Pi-Apps. I don't want to be rude, but I had to add some code to make it compatible with YOUR software, but is not my business. You always can remove my software from Pi-Apps.

So, I'm going to fix for Pi-Apps this time, but If It's broken again in the future, I'm afraid you must come here again and notify me, and maybe I'll see what I can do.

@Botspot
Copy link
Author

Botspot commented Dec 13, 2023

I don't know when was the moment in the time PiKISS depended on Pi-Apps. I don't want to be rude, but I had to add some code to make it compatible with YOUR software, but is not my business.

I would clarify that for both times something broke, it was due to a change you made that broke noninteractive installation. Ideally there would not need to be any specific pi-apps workarounds. For example, you would have encountered the same two problems if you tried packaging pikiss into a deb package, as in some cases those can be noninteractive.
Pi-Apps is not the issue, but Pi-Apps is affected by the underlying issue. I just wanted to clarify that.

You always can remove my software from Pi-Apps.

I could, but before you start suggesting such things, please be reminded that Pi-Apps has brought in 27,393 users to your project. source
I try to evaluate apps on the basis of usefulness without playing favorites. PiKISS is useful so it stays. Of course if it keeps breaking and users cannot install it, then fairness would force me to disable it or even remove it. Yes I have done this before to multiple other apps.
There is no reason it should have to come to that, though I am making the assumption that you value your userbase and donors, some of which only found out about PiKISS because it was on Pi-Apps.

@jmcerrejon
Copy link
Owner

Fixed at d2ecd27 & 617360b.

I could, but before you start suggesting such things, please be reminded that Pi-Apps has brought in 27,393 users to your project.

I repeat I am not going to test that the changes I make will work in Pi-Apps. You are free to do whatever you want in consequence. Maybe I don't sound fair towards users who bounce my software through you, and I feel sorry for them, but I will not tolerate any threats.

theofficialgman added a commit to Botspot/pi-apps that referenced this issue Jan 2, 2024
due to upstream breakage for the third time recently jmcerrejon/PiKISS#201
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants