Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 901311 - [guru] mpv-plugin/mpv-mpris-0.9-r2 fails tests: test metadata: stderr not empty!
Summary: [guru] mpv-plugin/mpv-mpris-0.9-r2 fails tests: test metadata: stderr not empty!
Status: RESOLVED FIXED
Alias: None
Product: GURU
Classification: Unclassified
Component: Package issues (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Oliver Freyermuth
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-03-15 08:47 UTC by Agostino Sarubbo
Modified: 2023-03-23 09:11 UTC (History)
1 user (show)

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


Attachments
build.log (build.log,119.55 KB, text/plain)
2023-03-15 08:47 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2023-03-15 08:47:45 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: mpv-plugin/mpv-mpris-0.9-r2 fails tests.
Discovered on: amd64 (internal ref: guru_tinderbox)
Comment 1 Agostino Sarubbo gentoo-dev 2023-03-15 08:47:48 UTC
Created attachment 857803 [details]
build.log

build log and emerge --info
Comment 2 Oliver Freyermuth 2023-03-15 23:47:36 UTC
Thanks for the report!

The cause is that media-video/mpv[-lua,-javascript] does not support the CLI options --load-scripts and --scripts, which are used by the tests of mpv-plugin/mpv-mpris. 

I believe the actual issue is an mpv bug, since "cplugin" (which mpv-plugin/mpv-mpris relies on) is also a "scripting backend" in mpv, so cplugins can also be loaded via these options. 

I'll delve a bit deeper, report an upstream issue and consider a workaround.
Comment 3 Oliver Freyermuth 2023-03-18 20:17:13 UTC
I have fixed this upstream in media-video/mpv via:
https://github.com/mpv-player/mpv/pull/11468

As a workaround for currently released mpv versions, I have added a dependency on media-video/mpv[lua] or media-video/mpv[javascript] or a to-be-released mpv version via:
https://gitweb.gentoo.org/repo/proj/guru.git/commit/?h=dev&id=486f989f33290842fdf31906ae33e14ceb99a544
This should fix this issue once merged. 

Thanks again for the report!
Comment 4 Larry the Git Cow gentoo-dev 2023-03-23 09:11:56 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=486f989f33290842fdf31906ae33e14ceb99a544

commit 486f989f33290842fdf31906ae33e14ceb99a544
Author:     Oliver Freyermuth <o.freyermuth@googlemail.com>
AuthorDate: 2023-03-18 20:13:07 +0000
Commit:     Oliver Freyermuth <o.freyermuth@googlemail.com>
CommitDate: 2023-03-18 20:13:07 +0000

    mpv-plugin/mpv-mpris: depend on mpv script backends for tests
    
    Current media-video/mpv does not support "scripts" option
    unless lua or javascript support is enabled.
    This is fixed upstream via:
    https://github.com/mpv-player/mpv/pull/11468
    for the next release.
    
    Closes: https://bugs.gentoo.org/901311
    Signed-off-by: Oliver Freyermuth <o.freyermuth@googlemail.com>

 mpv-plugin/mpv-mpris/mpv-mpris-0.9-r2.ebuild | 8 +++++++-
 mpv-plugin/mpv-mpris/mpv-mpris-9999.ebuild   | 8 +++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)