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

Option to install proprietary codecs #92

Draft
wants to merge 44 commits into
base: main
Choose a base branch
from

Conversation

meisenzahl
Copy link
Member

Fixes #91

Copy link
Member

@jlnr jlnr left a comment

Choose a reason for hiding this comment

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

I'm not sure what the best way to word this is, but in my opinion it would be helpful to point out where these codecs are going to be used. Just Music/Movie apps? Will this affect flatpak'd third-party apps?

src/Views/ThirdPartySoftwareView.vala Outdated Show resolved Hide resolved
src/MainWindow.vala Outdated Show resolved Hide resolved
@davidmhewitt
Copy link
Member

This doesn't actually install the packages, correct?

It seems you just append them to a list, and then this list isn't used anywhere.

@meisenzahl
Copy link
Member Author

@davidmhewitt correct.

I will rearrange the views. The SoftwareView should be the last to be displayed, then install the software and display a progress bar.

Is there an easy way to install apt packages via Vala and check the progress?

@alatiera
Copy link

@jlnr

I'm not sure what the best way to word this is, but in my opinion it would be helpful to point out where these codecs are going to be used. Just Music/Movie apps?

Given this installs gst bad/ugly and libav, it will also pull down all of ffmpeg and its deps along with plenty of other useful codecs. So anything that uses GStreamer will have the codecs show up in the registry and be able to use them. Additionally there are plenty of apps that use ffmpeg directly and check for it at runtime, like Firefox.

Will this affect flatpak'd third-party apps?

No, Flatpak toolchains are completely isolated, usually applications pull down the org.freedesktop.Platform.ffmpeg-full extension during installation.

@cassidyjames cassidyjames requested a review from a team March 8, 2021 22:31
@davidmhewitt
Copy link
Member

For devices that use WiFi to connect to the internet, the likelihood of being connected during this initial setup period is pretty low, so this page won't work. I wonder if we should detect an internet connection and only show the page if there is one.

And depending on the thoughts of @elementary/ux , maybe it would make sense to show it on the next login (if there was an internet connection then), if it was missed the first time?

@danirabbit
Copy link
Member

Yeah we need a network connection step in initial setup before we can do this

I guess since we expect apps to be flatpak, we don't show debs in AppCenter anymore, we only need to pull down the flatpak extension here

@Marukesu
Copy link

Marukesu commented Apr 6, 2021

I guess since we expect apps to be flatpak, we don't show debs in AppCenter anymore, we only need to pull down the flatpak extension here

But, we use the system or user installation? Do extensions on system installation works with apps on user installation?

@meisenzahl meisenzahl marked this pull request as draft January 6, 2023 14:49
@meisenzahl
Copy link
Member Author

I tried to refactor this branch to install org.freedesktop.Platform.ffmpeg-full as Flatpak runtime.

Videos would use version 22.08 of org.freedesktop.Platform.ffmpeg-full as an extension if available: https://github.com/elementary/videos/blob/2.9.0/io.elementary.videos.yml#L24

Videos is installed system-wide. To test if the runtime effects the capability of Videos to play other video formats I installed org.freedesktop.Platform.ffmpeg-full system-wide (flatpak install --system runtime/org.freedesktop.Platform.ffmpeg-full/x86_64/22.08):

Name Application ID Version Branch Origin Installation
Videos io.elementary.videos 2.9.0 stable appcenter system
ffmpeg-full org.freedesktop.Platform.ffmpeg-full 22.08 freedesktop system

I tested if the installation of the extension affects which video formats can be played:

format without runtime with runtime
AVI
MOV
MP4
OGG
WMV
WEBM

I tested this after the installation of the extension and a reboot of my device. Unfortunately, this makes no difference.

@Marukesu @alatiera do you have any other ideas about what the issue could be?

@alatiera
Copy link

Seems like something goes wrong with the videos flatpak, I will need to debug it to investigate if the extension gets mounted, etc. I will try to find some time to do so during the weekend or else sometime next week

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.

Option to install proprietary codecs
7 participants