Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 940901 - app-office/texstudio: dev-qt/qtmultimedia:6 is optional and unused as of current cmake args
Summary: app-office/texstudio: dev-qt/qtmultimedia:6 is optional and unused as of curr...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo Linux bug wranglers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 936956
  Show dependency tree
 
Reported: 2024-10-05 11:08 UTC by hsk17
Modified: 2024-10-05 13:22 UTC (History)
0 users

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 hsk17 2024-10-05 11:08:44 UTC
good thing that qt6 and cmake have arrived in texstudio ebuild.

a mminor issue:  texstudio-4.8.4-r1.eclass has dev-qt/qtmultimedia in DEPEND which is not used as now.
qtmultimedia library is required by texstudio only if it's mediaplayer parts are actually compiled and linked in, which by default is not the case.  cmake variable TEXSTUDIO_ENABLE_MEDIAPLAYER is OFF by default and is not set in the ebuild.

proposal:  (re-)add video USE flag, change dependency as
  video? ( dev-qt/qtmultimedia:6 )
and add in src_configure()
  use video && mycmakeargs+=( -DTEXSTUDIO_ENABLE_MEDIAPLAYER=ON )
Comment 1 Larry the Git Cow gentoo-dev 2024-10-05 13:22:38 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=40dd28ade54ab98409cd88a82e0e7a18206f8072

commit 40dd28ade54ab98409cd88a82e0e7a18206f8072
Author:     Florian Schmaus <flow@gentoo.org>
AuthorDate: 2024-10-05 12:59:52 +0000
Commit:     Florian Schmaus <flow@gentoo.org>
CommitDate: 2024-10-05 13:15:27 +0000

    app-office/texstudio: re-add IUSE=video making qtmultimedia dep optional
    
    Thanks to hsk17 for pointing this out.
    
    Closes: https://bugs.gentoo.org/940901
    Signed-off-by: Florian Schmaus <flow@gentoo.org>

 app-office/texstudio/metadata.xml                                    | 2 +-
 .../{texstudio-4.8.4-r1.ebuild => texstudio-4.8.4-r2.ebuild}         | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)