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

GitHub Actions: fix Fedora CMake build #910

Merged
merged 4 commits into from
Jul 6, 2024

Conversation

ulmus-scott
Copy link
Contributor

The GitHub Actions Fedora CMake builder is now building libavdevice/iec61883.c which includes and uses FFmpeg’s mpegts.h, causing it to not compile.

Checklist

@ulmus-scott
Copy link
Contributor Author

I don't know if MythTV uses RTP or RTSP, so those still use our modified mpegts-mythtv.h.

This reduces the changes relative to upstream FFmpeg.
This reduces the changes relative to upstream FFmpeg in mpegts.*
and makes it clearer where in FFmpeg our version is used.

The impetus for this change is that the GitHub Actions Fedora CMake
builder is now building libavdevice/iec61883.c which includes and
uses FFmpeg’s mpegts.h, causing it to not compile.
Needed to run Qt moc on it.  Causes ld to fail otherwise.
This should have been added in db8c584
@ulmus-scott ulmus-scott force-pushed the GitHubActions-FedoraCMake branch from 62b3e76 to ce41d1b Compare June 22, 2024 17:05
@ulmus-scott
Copy link
Contributor Author

I rebased onto master without change to force a rebuild.

@hambre
Copy link
Contributor

hambre commented Jun 23, 2024

Thanks for these fixes, I need them also to compile on ArchLinux.

@ulmus-scott
Copy link
Contributor Author

Regarding avpriv_mpegts_parse_*(), MythTV does appear to use RTP and RTSP for (satellite) IPTV. mpegts-mythtv's versions are different due to changes in mpegts_open_section_filter() and write_section_data() (called via handle_packet()).

libavformat/mpegts.h is now unmodified.
@linuxdude42 linuxdude42 merged commit 9d5b6f0 into MythTV:master Jul 6, 2024
12 checks passed
@ulmus-scott ulmus-scott deleted the GitHubActions-FedoraCMake branch July 11, 2024 20:02
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.

3 participants