Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Flatpak changes #42

Merged
merged 3 commits into from
Oct 29, 2019
Merged

Flatpak changes #42

merged 3 commits into from
Oct 29, 2019

Conversation

p1u3o
Copy link
Contributor

@p1u3o p1u3o commented Oct 7, 2019

@TingPing Do these look good to you?

This removes the un-needed gst-plugins-ugly and disables a bunch of plugins that aren't used by Twitter for gst-plugins-bad, which reduces the disk usage by around 50%.

@p1u3o p1u3o mentioned this pull request Oct 7, 2019
@TingPing
Copy link
Contributor

TingPing commented Oct 7, 2019

Instead of maintaining a massive list of what not to build I suggest just building the single plugin you want like so: https://github.com/flathub/io.github.Pithos/blob/0a149e953c59b9410b5282a01b0e0e23e9f230f1/io.github.Pithos.json#L32-L44

@TingPing
Copy link
Contributor

TingPing commented Oct 7, 2019

Also if its a 1.16.1 specific regression (which sounds odd), add a comment.

@p1u3o
Copy link
Contributor Author

p1u3o commented Oct 8, 2019

Is there a way of checking what modules are currently being used? I guess otherwise it's going to be trial and error to see which one being replaced "fixes" things.

@p1u3o
Copy link
Contributor Author

p1u3o commented Oct 8, 2019

Nm, it seems building just gst/mpegtsdemux/libgstmpegtsdemux.so is what was "fixing it".

This makes no sense.

@p1u3o
Copy link
Contributor Author

p1u3o commented Oct 8, 2019

Some videos of the glitch I've been experiencing.

with old mpegtsdemux (1.16.0) - https://drive.google.com/open?id=1l1IA99HnChbwwJI-Yj9zzOzFoeJwe-1r
with current mpegtsdemux (1.16.1) https://drive.google.com/open?id=1PRFxNorNNMVF4vKdA8NHX1xFaM_kmNYa

in the latter, the video freezes at 4 seconds.

@p1u3o
Copy link
Contributor Author

p1u3o commented Oct 8, 2019

@TingPing
Copy link
Contributor

TingPing commented Oct 8, 2019

Good job tracking it down 😃

Is this regression reported upstream?

@p1u3o
Copy link
Contributor Author

p1u3o commented Oct 9, 2019

Good job tracking it down smiley

Is this regression reported upstream?

I couldn't find it upstream. I'll file it when I get a minute.

In the mean time I've attached a patch to this MR that reverts the troubling commit.

@TingPing
Copy link
Contributor

TingPing commented Oct 9, 2019

@IBBoard
Copy link
Owner

IBBoard commented Oct 12, 2019

Is this good to merge now?

@p1u3o
Copy link
Contributor Author

p1u3o commented Oct 21, 2019

Is this good to merge now?

This should be fine to merge into this repo, it doesn't conflict with anything. I've been busy the last couple of weeks and haven't got around to debugging the output to go onto a bug report upstream.

@IBBoard IBBoard merged commit f1578b9 into IBBoard:master Oct 29, 2019
@IBBoard IBBoard mentioned this pull request Nov 17, 2019
@sdroege
Copy link

sdroege commented Nov 24, 2019

Did someone report this against GStreamer in the end? I can't find any issue that looks related.

@p1u3o
Copy link
Contributor Author

p1u3o commented Nov 25, 2019

I've been super busy with work the last few weeks and haven't had the chance to look into this, as much I would've liked (sorry).

I'm not that clued on the GStreamer stack, but I see some changes upstream on GStreamer where this bug is, I'll today compile with the latest upstream changes and see if there's any difference.

@p1u3o
Copy link
Contributor Author

p1u3o commented Nov 25, 2019

This seems fixed on the upstream gst-plugins-bad 1.16 branch.

Edit: Was fixed by https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/commit/cfa0f1858075a3390def50250eafc89d270178d3, updated manifest coming up.

@IBBoard
Copy link
Owner

IBBoard commented Nov 25, 2019

Thanks for finding that! Hopefully the backport will be released (and packaged by distros) soon.

IBBoard added a commit that referenced this pull request Feb 8, 2020
Apparently it's meant to be done, there's no way to stop gettext
picking it up, and not translating just messes up the completion
percentages. My name is now pre-translated in all languages instead.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants