dtcooper / raspotify

A Spotify Connect client that mostly Just Works™
https://dtcooper.github.io/raspotify
MIT License
4.63k stars 213 forks source link

Raspotify doesen't install on Raspberry Pi Buster #517

Closed bobrathbone closed 2 years ago

bobrathbone commented 2 years ago

Compatible OS

Latest Version

Due Diligence

What happened?

Installation on a Raspberry Pi 4 with RPi OS Bullseye

curl -sL https://dtcooper.github.io/raspotify/install.sh | sh Fails with syntax error

Solution: run with bash curl -sL https://dtcooper.github.io/raspotify/install.sh | bash

Once the above is cured the installation fails with a apt-transport-https dependency error. Solution: Run

sudo apt-get install apt-transport-https The corrected curl program then runs OK

Relevant log output

curl -sL https://dtcooper.github.io/raspotify/install.sh | sh
sh: 60: Syntax error: "(" unexpected (expecting "then")
bobrathbone commented 2 years ago

I don't think that the above is a very useful answer especially when I have already supplied the solution. Would it not be more sensible to put the correct instructions in "The easy way" on the wiki at https://github.com/dtcooper/raspotify Namely: sudo apt-get install apt-transport-https curl -sL https://dtcooper.github.io/raspotify/install.sh | bash

JasonLG1979 commented 2 years ago

I haven't given you a solution?

JasonLG1979 commented 2 years ago

Thanks for the report. As far as I can tell I've fixed the issue.

bobrathbone commented 2 years ago

Many thanks for the update. I have checked the script today and the syntax error has disappeared.

The reason for my my comment about the usefulness of your first reply was that the the first update stated I saw was something like:

"This issue can probably be easily resolved by reading the docs/wiki" https://github.com/dtcooper/raspotify/labels/read-the-field-manual Status: Closed

I took this to mean RTFM. The above reply has now disappeared from this post. My apologies if I misunderstood this although I am still confused why such a reply was posted.

JasonLG1979 commented 2 years ago

Many thanks for the update. I have checked the script today and the syntax error has disappeared.

The reason for my my comment about the usefulness of your first reply was that the the first update stated I saw was something like:

"This issue can probably be easily resolved by reading the docs/wiki" read-the-field-manual Status: Closed

I took this to mean RTFM. The above reply has now disappeared from this post. My apologies if I misunderstood this although I am still confused why such a reply was posted.

You have crossed issues. This issue has never been tagged. Your other issue was tagged because if you would have read the config file you could have figured it out yourself.

Google RTFM. One of the possible origins to RTFM is Read the field manual. So yes that tag means exactly that.