Bug 238004 - New port: multimedia/makemkv video converter reading from DVD and Bluray
Summary: New port: multimedia/makemkv video converter reading from DVD and Bluray
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs mailing list
Depends on:
Reported: 2019-05-20 19:04 UTC by Felix Palmen
Modified: 2019-08-20 16:57 UTC (History)
3 users (show)

See Also:

add multimedia/makemkv (16.70 KB, patch)
2019-05-20 19:04 UTC, Felix Palmen
no flags Details | Diff
add multimedia/makemkv (16.94 KB, patch)
2019-05-22 16:40 UTC, Felix Palmen
no flags Details | Diff
add multimedia/makemkv (16.94 KB, patch)
2019-06-10 08:55 UTC, Felix Palmen
no flags Details | Diff
add multimedia/makemkv (16.46 KB, patch)
2019-06-16 13:20 UTC, Felix Palmen
no flags Details | Diff
add multimedia/makemkv (16.49 KB, patch)
2019-06-16 13:29 UTC, Felix Palmen
no flags Details | Diff
add multimedia/makemkv (16.52 KB, patch)
2019-08-20 16:57 UTC, Felix Palmen
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Palmen 2019-05-20 19:04:56 UTC
Created attachment 204487 [details]
add multimedia/makemkv

Unfortunately, this port leaves out the MakeMKV GUI. Although the GUI can be built with a few patches on FreeBSD, it's useless because makemkvcon (the closed source Linux binary) won't run in "guiserver" mode. See <https://www.makemkv.com/forum/viewtopic.php?f=3&t=19486> for my attempt to get upstream help with that.

Still, after being on the wanted ports list for so long, I think console-only usage is better than nothing, and it works quite well for me.
Comment 1 Felix Palmen 2019-05-22 16:40:50 UTC
Created attachment 204545 [details]
add multimedia/makemkv

Small improvements:

- add hint to pkg-message about needed permissions of /compat/linux/tmp, so makemkvcon runs as non-root
- linted manpage
Comment 2 Felix Palmen 2019-06-10 08:55:25 UTC
Created attachment 204939 [details]
add multimedia/makemkv

Update to new upstream version 1.14.4

I also noticed poudriere testport issues
> Warning: you need USES=ssl
This is probably bogus, the port only links OpenSSL from the linux-c7 ports.
Comment 3 Sergey A. Osokin freebsd_committer 2019-06-14 00:29:10 UTC
Hi Felix,

so pretty impressive job, thanks very much for sharing this!

I've got next question when I've done with my review of your job, it is related to '/opt' the port uses.  Can we avoid this by moving needful binaries to another location?  FreeBSD Handbook basically recommends to use ${PREFIX} for most cases, so that would be great to improve the port.

Please let me know if you have any questions.

Again, well done!  Thanks very much!
Comment 4 Felix Palmen 2019-06-14 05:18:39 UTC
Hello Sergey,

The "makemkvcon" binary uses /sys/bus/scsi (which isn't provided by linsysfs) to find optical drives. Therefore I included a script to fake the relevant structure based on camcontrol output. I can't do that in the original location, because that wouldn't work on machines that have linsysfs mounted. The pathname used must not be longer than the original because I'm patching it into the makemkvcon binary with sed.

/opt/makemkv looked like an obvious choice, as I could put almost everything right there and people will know it's a software package with its own directory. Maybe /compat/linux/opt/makemkv would be a better choice, but I'm not sure it's a good idea to put files there that don't come from a linux-c* port?

I could also separate the linux binaries and libs (putting them in their "standard" dirs) from the faked sysfs parts. But then, where to put the latter? /etc/makemkv would be short enough, but the generated files neither are really "configuration" nor do they belong to base ...

I'll gladly try to incorporate better ideas of course :) thank you very much.

Best regards
Comment 5 Felix Palmen 2019-06-16 13:20:47 UTC
Created attachment 205124 [details]
add multimedia/makemkv

After some brief email discussion about using /opt (which definitely isn't an ideal solution), I think I came up with the best alternative here.

The faked sysfs entries are now created in ${LINUXBASE}/etc/makemkv, everything else goes to its normal destination in ${LOCALBASE}. The libraries are installed to ${LOCALBASE}/lib/makemkv, to avoid cluttering the normal lib dir with "foreign" (Linux) libs.

This also allows for easy migration away from "faking" sysfs stuff, once linsysfs(5) supports the files needed for makemkvcon.
Comment 6 Felix Palmen 2019-06-16 13:29:04 UTC
Created attachment 205127 [details]
add multimedia/makemkv

Sorry for the noise, accidentally uploaded a patch missing my last bugfix.

This one is fine and works.
Comment 7 Felix Palmen 2019-07-03 16:29:57 UTC
Any news on this? If there's anything else I could (or should) improve, please let me know, thanks!

BR, Felix
Comment 8 Johannes Jost Meixner freebsd_committer 2019-08-02 12:45:57 UTC
Hi Felix,

I've looked through the attachment a bit and have some questions:

* Why do you bundle OpenSSL -- twice, once with 1.0.1e (available in ports, security/linux-c6-openssl within the CentOS 6 userland) and once with 1.0.2k (available in ports, security/linux-c7-openssl within the CentOS 7 userland)?
* Related: Have you considered submitting ports for ffmpeg (based on the CentOS RPMs in "EPEL") and fdk-aac (maybe via RpmFusion) ?

Now, to the Makefile.
I'm a bit wormed by calling configure three times in pre-configure, and even calling `make` there. This will probably go away once the prerequisites are outsourced, right?

Comment 9 Felix Palmen 2019-08-02 13:01:40 UTC
- OpenSSL is NOT bundled, it's just downloaded and extracted, so the correct headers can be used during compilation. The libraries provided by linux ports are linked.

- for the same reason, providing packages for ffmpeg / fdk-aac would only complicate things. they would be in different versions, depening on the linux port version installed, ffmpeg would not link fdk-aac (as recommended by makemkv), of course they would not include headers either and makemkv is typically tested against the latest versions of ffmpeg. I think static linking is pretty appropriate here.
Comment 10 Felix Palmen 2019-08-20 16:57:37 UTC
Created attachment 206736 [details]
add multimedia/makemkv

build was broken by r506352, so here's an updated patch.

BR, Felix