On 2014/06/04 19:47, Christian Weisgerber wrote: > Here's a port for MediaInfo 0.7.69: > > | MediaInfo is a convenient unified display of the most relevant technical > | and tag data for video and audio files. > > This is the CLI version only. As far as I understand, the GUI > version just presents the same information in a window. Not worth > the effort. > > This is software that is also intended for non-POSIX platforms, so > it's all a bit icky. The tarball I picked is a convenience package > of ZenLib, MediaInfoLib, and MediaInfo proper. They all need to > be configured and built individually and it took some hammering to > get this into the ports framework. The components are also distributed > separately and we _could_ split it all up into three ports, but I > doubt there's much interest in freestanding ZenLib and MediaInfoLib. > > -- > Christian "naddy" Weisgerber [email protected]
Missing NO_TEST. I would prefer to fix line-endings in the files which are patched, it's not strictly necessary for ports, but otherwise emailed patches tend to break, I normally use "perl -i -pe 's/\r$$//' <file...>" for this. Personally I'd use DISTNAME/EXTRACT_SUFX rather than setting DISTFILES, either way works though. Otherwise: works fine for me, and provides a bit more information and is easier to read than ffprobe so I'm in favour of it going in. I agree with using the bundle in this case; if something else turns up that needs any of the libraries we could look at separating them then.
