mirror of
https://github.com/aome510/spotify-player.git
synced 2026-04-26 01:15:55 +03:00
[GH-ISSUE #189] Homebrew support #99
Labels
No labels
bug
documentation
enhancement
good first issue
help wanted
pull-request
question
third-party
third-party
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-player#99
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @flexwie on GitHub (May 5, 2023).
Original GitHub issue: https://github.com/aome510/spotify-player/issues/189
Is your feature request related to a problem? Please describe.
Using cargo to install the player is the easiest way for Mac users, but still requires to have the rust toolchain installed on the machine.
Describe the solution you'd like
Many if not most applications for Mac can be installed with Homebrew. It would be nice to have a formula to do so.
Describe alternatives you've considered
I guess a simple bash script that grabs the binaries from the releases would work as well, but its less elegant and won't support updating and uninstalling.
@aome510 commented on GitHub (May 5, 2023):
I don't have any experience dealing with
homebrewbefore. Any help on this will be greatly appreciated.@imNel commented on GitHub (May 7, 2023):
Basically you'll have to create a ruby file for
spotify-playerand then PR into homebrew-core. You can use the docs for adding a new formula or view the formula cookbook for more info :)@flexwie commented on GitHub (May 8, 2023):
I don't have any experience either so far, but I would be open to trying it out!
@tresni commented on GitHub (May 16, 2023):
I did this 3 weeks ago 😁 https://github.com/Homebrew/homebrew-core/pull/128997
Please test and 👍 that formula to try to get it bumped up for inclusion.
@flexwie commented on GitHub (May 21, 2023):
Looks great! One thing I'd love to have is the ability to choose between the different features instead of installing them all. Maybe that could be done with options?
@tresni commented on GitHub (May 21, 2023):
That is possible. At one point I know it was being discouraged to have options in
homebrew-coreand instead to do that in a separate tap (similar to the ffmpeg example they give in the documentation.) I don't know if that's still the case, I'll do some grepping through the core formulas and see.@aome510 commented on GitHub (May 21, 2023):
IMO, it's better to install the application with default features. Not all non-default features work well without setting up or further configurations.
@aome510 commented on GitHub (May 21, 2023):
Also thanks for the PR in the core repo, look forward to seeing it getting merged.
@tresni commented on GitHub (May 21, 2023):
Okay, confirming that you are not allowed to use options in homebrew/core:
Using the ffmpeg formula as a guide, it would appear that enabling some optional features would generally be the recommendation. I would suggest that we leave the formula as is unless I'm missing something and there are cases where enabling
images,lyric-finder,notifycould actually cause the application to not work. In my (all be it) limited testing, I didn't find any issues with having them enabled if the underlying system didn't support the feature (e.g. running in Terminal vs iTerm withimagesjust lead to block display, I haven't found a way to makelyric-findernot work, etc)@aome510 commented on GitHub (May 22, 2023):
One case that
lyric-finderdoesn't work: https://github.com/aome510/spotify-player/issues/155. Andimagedoesn't display a "nice-looking" image unless you usekittyoriTerm2.notifycan be quite annoying if you don't really need one.For me, I didn't find any problems with
imageandlyric-finderas my two additional features.My point is that not all people need the features listed above and sometimes they may not work as expected. Using the default features provide a good and comfortable baseline. Users can always install the app with more features enabled if they want.
That said, the PR is still yours so feel free to leave as-is if that's what you think the best.
@aome510 commented on GitHub (Jun 10, 2023):
Close this as https://github.com/Homebrew/homebrew-core/pull/128997 has been merged.