Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 880755 - app-misc/mc videofile view error message is missing portage-specific information
Summary: app-misc/mc videofile view error message is missing portage-specific information
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Marek Szuba
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-10 16:22 UTC by lexofleviafan
Modified: 2023-01-24 11:00 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description lexofleviafan 2022-11-10 16:22:57 UTC
When trying to view (F3) a videofile in Midnight Commander and it can't locate a program to extract information with, it shows a message saying you need to have mplayer or mpv installed; however, the file that it's looking for (in case of mpv) is `mpv_identify.sh`, which is only installed with mpv when the `tools` USE flag is enabled.

Thus, anyone who has mpv installed without `+tools` will see a confusing message suggesting to install mpv.

Reproducible: Always

Steps to Reproduce:
1. Have `app-misc/mc` installed (without `media-video/mplayer` or `media-video/mpv +tools`), with default configuration for handling videofiles
2. Run `mc` and navigate to a videofile (e.g. an MP4 file)
3. Press F3 to view the file
Actual Results:  
The viewer contains the following message:
`Please install either mplayer or mpv to get information for this file`

Expected Results:  
The message should clarify that mpv should be installed with `+tools` USE flag.

The file containing the message is `/usr/libexec/mc/ext.d/video.sh`; there should be no problem in patching it during the installation.
Comment 1 Piotr Karbowski (RETIRED) gentoo-dev 2022-11-10 18:51:25 UTC
This is Gentoo-specific so we cannot really bring it upstream. Our only options would be to patch /usr/libexec/mc/ext.d/video.sh and replace the current message's 'mpv' with 'mpv[tools]'.
Comment 2 Larry the Git Cow gentoo-dev 2023-01-24 11:00:59 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2b77588e1eca99bfa6f72fa241cbe6107a85984c

commit 2b77588e1eca99bfa6f72fa241cbe6107a85984c
Author:     Marek Szuba <marecki@gentoo.org>
AuthorDate: 2023-01-24 10:08:53 +0000
Commit:     Marek Szuba <marecki@gentoo.org>
CommitDate: 2023-01-24 11:00:44 +0000

    app-misc/mc: add 4.8.29
    
    Closes: https://bugs.gentoo.org/880755
    Signed-off-by: Marek Szuba <marecki@gentoo.org>

 app-misc/mc/Manifest                           |   1 +
 app-misc/mc/files/mc-4.8.29-gentoo-tools.patch |  11 +++
 app-misc/mc/mc-4.8.29.ebuild                   | 126 +++++++++++++++++++++++++
 3 files changed, 138 insertions(+)