docker-tools icon indicating copy to clipboard operation
docker-tools copied to clipboard

Add libmythtv-perl to mythtv-backend build to make mythlink.pl usable

Open d0ct0rvenkman opened this issue 4 years ago • 3 comments
trafficstars

This commit will add in the MythTV perl libraries into the mythtv-backend container to facilitate the use of mythlink.pl.

https://github.com/MythTV/mythtv/blob/master/mythtv/contrib/user_jobs/mythlink.pl

d0ct0rvenkman avatar Jan 17 '21 16:01 d0ct0rvenkman

Wow, that's an old/forgotten script! But indeed it's still in the source repo - see wiki doc for mythlink.pl and source on github. Do any of the installable packages at Mythbuntu ppa include the mythlink.pl script?

instantlinux avatar Jan 18 '21 18:01 instantlinux

I haven't checked to see if it's present in any of the Myth packages. I just stuck the script into my DVR recording directory and ran it from there. I'll see if I can find out when I take some free time later in the week.

Also, the script is not as ill-maintained as the docs would suggest. Its been updated at some point to allow renaming in the Plex-friendly "Title - SXXEYY - Title.ext" format, so it's at least a bit useful. It seems like Myth is likely a lot more robust of a DVR solution than the inbuilt Plex DVR, so I'm trying to see how well they can work together.

d0ct0rvenkman avatar Jan 18 '21 22:01 d0ct0rvenkman

It looks like the file is supposed to be contained within the mythtv-backend package, but I don't see the file in my container.

mythtv@mythtv-backend:~$ dpkg -S mythlink.pl
mythtv-backend: /usr/share/doc/mythtv-backend/contrib/user_jobs/mythlink.pl
mythtv@mythtv-backend:~$ ls -al /usr/share/doc/mythtv-backend/contrib/user_jobs/mythlink.pl
ls: cannot access '/usr/share/doc/mythtv-backend/contrib/user_jobs/mythlink.pl': No such file or directory

d0ct0rvenkman avatar Feb 03 '21 02:02 d0ct0rvenkman

Ahoy! Just curious if the above change can be merged in. I'm getting ready to upgrade to a v32 container and it would be wonderful if I didn't have to manually customize my container image.

d0ct0rvenkman avatar Jun 21 '23 13:06 d0ct0rvenkman

Can you add the mythlink,pl script to this PR? If it's not part of any system package, then add a fetch operation (with checksum verification) to pull it from github. Something like

ARG PPA_BRANCH=32
ARG MYTHLINK_SHA=459cb8b60adae4b631a95a9cfb1b41dcb959cc4a0b9053582a711d58b8d8a0d2
wget https://raw.githubusercontent.com/MythTV/mythtv/fixes/${PPA_BRANCH}/mythtv/contrib/user_jobs/mythlink.pl
echo "$MYTHLINK_SHA  mythlink.pl" | sha256sum -c

That way the build will have similar robustness as an apt package.

instantlinux avatar Jun 21 '23 18:06 instantlinux

The PR has been updated with the commands you suggested. I needed to bump the mythtv package version against the latest shown here to get the container to build.

d0ct0rvenkman avatar Jul 02 '23 18:07 d0ct0rvenkman

@d0ct0rvenkman the image tag 32.0-fixes.202305160103.72024244b8 is published with this update (I've also set permissions of mythlink.pl to rwxr-xr-x).

Version 33.1 is out so I'll get that published when I get some time.

instantlinux avatar Jul 15 '23 23:07 instantlinux