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

Add external player support for Baka, Celluloid & Haruna #4067

Merged

Conversation

trostboot
Copy link
Contributor

Add external player support for Baka, Celluloid & Haruna

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

closes #3990

Description

Adds basic support for Baka & Haruna and extended support for Celluloid

Testing

Haruna and Celluloid have been fully tested, all options should work. Baka does not appear to be in active development anymore and requires replacing mpv and youtube-dl on Windows to even work, and it segfaults for me in Linux.
If someone can confirm it works for them, I'll leave it in, otherwise I'll remove it.

Desktop

  • OS: openSUSE Tumbleweed
  • OS Version: 20230921-2505.1
  • FreeTube version: 0.19

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Sep 24, 2023
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) September 24, 2023 17:22
@PikachuEXE
Copy link
Collaborator

Waiting Baka to be tested first

#3990 (comment)
image

@trostboot
Copy link
Contributor Author

In addition to the testing in #3990 (comment), I've also spun up a Manjaro VM and Baka works as expected there. So I'm inclined to leave it in for now.

PikachuEXE
PikachuEXE previously approved these changes Oct 7, 2023
Copy link
Collaborator

@PikachuEXE PikachuEXE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if tested then approve

@efb4f5ff-1298-471a-8973-3d47447115dc

leaving last review for @absidue

@absidue
Copy link
Member

absidue commented Oct 8, 2023

Is it worth adding Baka, a player, that according to the comments on the issue, struggles to play YouTube videos and hasn't seen development in years, so is unlikely to get fixed?

@trostboot
Copy link
Contributor Author

I'll leave that decision up to you. It technically works in that it will play streams fine, if used with a modern yt-dlp and mpv binary, whether that's supplied manually on Windows or provided automatically through your distro's package manager on Linux. It just doesn't do much more than that.

Personally, I'd say it's abandoned at this point and I wouldn't have included it if not asked for in the request.

auto-merge was automatically disabled October 10, 2023 16:46

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) October 10, 2023 16:46
@FreeTubeBot FreeTubeBot merged commit 1cc1ec6 into FreeTubeApp:development Oct 11, 2023
10 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 11, 2023
@trostboot trostboot deleted the ext-baka-celluloid-haruna branch October 11, 2023 12:17
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Oct 17, 2023
* development:
  Bump webpack from 5.88.2 to 5.89.0 (FreeTubeApp#4168)
  Fix playlists not showing up on artist topic channels with the local API (FreeTubeApp#4129)
  Bump undici from 5.19.1 to 5.26.3 (FreeTubeApp#4173)
  Bump marked from 9.1.0 to 9.1.2 (FreeTubeApp#4171)
  Bump sass from 1.69.0 to 1.69.3 (FreeTubeApp#4169)
  Bump the eslint group with 2 updates (FreeTubeApp#4166)
  Bump the babel group with 2 updates (FreeTubeApp#4165)
  Truncate title of playlist in queue window (FreeTubeApp#4134)
  Wrap channel tabs as soon as there isn't enough screen space (FreeTubeApp#4139)
  Keep download folder path when changing to 'ask for download path' and back (FreeTubeApp#4101)
  Add external player support for Baka, Celluloid & Haruna (FreeTubeApp#4067)
  Fix error when clicking on the current quality in the DASH quality selector (FreeTubeApp#4130)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants