Subject: Re: bug#27046: [PATCH 3/3] gnu: video: Add mediainfo.

From: Marius Bakke <[email-masked]>
Date: Wed, 24 May 2017 18:37:19 +0200

"Ethan R. Jones" <[email-masked]> writes:

> * gnu/packages/video.scm (mediainfo): New variable.

[...]

> +    (arguments
> +     '(#:tests? #f ; lacks tests
> +       #:phases
> +       ;; build scripts not in root of archive
> +       (modify-phases %standard-phases
> +         (add-before
> +             'configure 'pre-configure

Can you also move the 'configure line up after '(add-before'? Goes for
the other patches too.

> +           (lambda _
> +             (chdir "Project/GNU/CLI")))
> +         (add-before 'configure 'autogen
> +           (lambda _
> +             (zero? (system* "./autogen.sh")))))))
> +    (home-page "https://mediaarea.net/en/MediaInfo")
> +    (synopsis "Utility for reading media metadata")
> +    (description "MediaInfo is a utility used for retrieving technical
> +information and other metadata about audio or video files.
> +A non-exhaustive list of the information MediaInfo can retrieve from media files include:
> + - General: title, author, director, album, track number, date, duration...
> + - Video: codec, aspect, fps, bitrate...
> + - Audio: codec, sample rate, channels, language, bitrate...
> + - Text: language of subtitle
> + - Chapters: number of chapters, list of chapters
> +MediaInfo supports the following formats:
> + - Video: MKV, OGM, AVI, DivX, WMV, QuickTime, Real, MPEG-1, MPEG-2,
> +          MPEG-4, DVD (VOB)...
> + - Video Codecs: DivX, XviD, MSMPEG4, ASP, H.264, AVC...)
> + - Audio: OGG, MP3, WAV, RA, AC3, DTS, AAC, M4A, AU, AIFF...
> + - Subtitles: SRT, SSA, ASS, SAMI...
> +MediaInfo supplies technical and tag information about a video or audio file")

Please also convert this list to use @itemize. Can you send updated
patches? TIA! :-)