Page MenuHomePureOS Tracker

smplayer
Closed, ResolvedPublic

Description

Reference: https://bugs.debian.org/870233

smplayer recommends nonfree smtube optional installation

Parabola patch

Event Timeline

mladen created this task.May 16 2017, 11:20
mladen updated the task description. (Show Details)

this is not part of any dependency chain, so before I drop into source code of it, can you provide me some screenshots about that issue @mladen.pejakovic

mladen added a comment.EditedMay 20 2017, 05:47


Patch is already suggested:

Notes: Patch: https://git.parabola.nu/abslibre.git/tree/libre/smplayer/disable-smtube-support.patch

we just need to apply it.

ah that is easy one, just commenting lines (actually most of those gui suggestion will be solved by commenting lines). I'll see to patch this out today as well.

mladen added a comment.May 20 2017, 05:53
This comment was removed by mladen.

seems smplayer (authored by same upstream as smtube) is sloppy about executing external code: As an example, if youtube parsing fails then it offers to download updated javascript code for parsing - from insecure URL!

For security reasons I recommend to avoid smplayer (and smtube).

A less featureful alternative is gnome-mpv.

so, do you suggest to drop them both from archive (its not like we don't have plenty of other apps for such use case) and improvement in security and freedom of archive might be enough big to simply remove them from pureos archive (we can always change that in future if it changes)

Yes, I recommend to drop smplayer and smtube from PureOS.

If the mechanism to drop has the ability to include comments for those wondering why the Debian package is missing from PureOS, then I suggest mentioning gnome-mpv and minitube.

mak lowered the priority of this task from 95 to Freedom Issue.Aug 18 2017, 22:09
jonas.smedegaard closed this task as Resolved.Sep 18 2017, 03:58

Fixed in Debian.

jonas.smedegaard renamed this task from [FREEDOM ISSUE] smplayer to smplayer.Oct 24 2017, 05:40
jonas.smedegaard claimed this task.
jonas.smedegaard updated the task description. (Show Details)
jonas.smedegaard added a subscriber: hema.prathaban.