Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 863614 - media-libs/suil: reworked ebuild (fixes: IUSE="gtk, qt5", HOMEPAGE, SRC_URI)
Summary: media-libs/suil: reworked ebuild (fixes: IUSE="gtk, qt5", HOMEPAGE, SRC_URI)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Professional Audio Applications Maintainers
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2022-08-04 14:01 UTC by Olivier Huber
Modified: 2022-09-18 13:34 UTC (History)
0 users

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


Attachments
proposed changes (suil.patch,4.70 KB, patch)
2022-08-04 14:01 UTC, Olivier Huber
Details | Diff
changes for suil 0.10.16 (suil.patch,2.49 KB, patch)
2022-08-15 08:50 UTC, Olivier Huber
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Olivier Huber 2022-08-04 14:01:09 UTC
Created attachment 797656 [details, diff]
proposed changes

The attached patch does the following:
- Fix copypasta longdescription in metadata.xml
- Fix HOMEPAGE and SRC_URI
- Remove python dependency: there is no python involved in suil. For building the docs, the dependency should be fine (?)
- add gtk2 useflag and dependencies
- add X useflag and dependencies
- Add missing Qt5 dependencies
- simplify dependencies management

Tested with pkgdev manifest, pkgcheck scan --net and qa-vdb suil

I can also open a pull request on github if that helps.
Comment 1 Andreas Sturmlechner gentoo-dev 2022-08-05 11:49:38 UTC
It is good to fix bogus IUSE=gtk ( x11-libs/gtk+:2 ) enabling $(meson_feature gtk gtk3) but please do not introduce new gtk+:2 dependencies at this point anymore.
Comment 2 Olivier Huber 2022-08-05 12:02:30 UTC
(In reply to Andreas Sturmlechner from comment #1)
> It is good to fix bogus IUSE=gtk ( x11-libs/gtk+:2 ) enabling
> $(meson_feature gtk gtk3) but please do not introduce new gtk+:2
> dependencies at this point anymore.

Sadly Ardour, one of the major DAW, is still using gtk+:2 and as such gtk2 support in suil is much needed. In the past, the devs stated their lack of interest in updating :(

Btw if this gets merged, the dependency in media-sound/ardour must be adjusted to media-libs/suil[gtk2]
Comment 3 Andreas Sturmlechner gentoo-dev 2022-08-05 13:02:11 UTC
(In reply to Olivier Huber from comment #2)
> Sadly Ardour, one of the major DAW, is still using gtk+:2 and as such gtk2
> support in suil is much needed.
How does that even work right now, since IUSE=gtk is actually enabling gtk3?
Comment 4 Olivier Huber 2022-08-05 13:04:18 UTC
(In reply to Andreas Sturmlechner from comment #3)
> (In reply to Olivier Huber from comment #2)
> > Sadly Ardour, one of the major DAW, is still using gtk+:2 and as such gtk2
> > support in suil is much needed.
> How does that even work right now, since IUSE=gtk is actually enabling gtk3?

The build system detects gtk+:2 and enables the module
Comment 5 Olivier Huber 2022-08-15 08:50:35 UTC
Created attachment 799651 [details, diff]
changes for suil 0.10.16

suil 0.10.16 was added to the tree, updating the patch
Comment 6 Larry the Git Cow gentoo-dev 2022-09-18 13:34:32 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=690b0c79e8f921246347194e62a9a6de6b35ea85

commit 690b0c79e8f921246347194e62a9a6de6b35ea85
Author:     Miroslav Šulc <fordfrog@gentoo.org>
AuthorDate: 2022-09-18 13:17:08 +0000
Commit:     Miroslav Šulc <fordfrog@gentoo.org>
CommitDate: 2022-09-18 13:34:28 +0000

    media-libs/suil: improvements to 0.10.16
    
    - Fix copypasta longdescription in metadata.xml
    - Fix HOMEPAGE and SRC_URI
    - Remove python dependency: there is no python involved in suil. For building the docs, the dependency should be fine (?)
    - add gtk2 useflag and dependencies
    - add X useflag and dependencies
    - Add missing Qt5 dependencies
    - simplify dependencies managemen
    
    Patch by Olivier Huber <oli.huber@gmail.com>
    
    Closes: https://bugs.gentoo.org/863614
    Signed-off-by: Miroslav Šulc <fordfrog@gentoo.org>

 media-libs/suil/metadata.xml                       | 18 +++++++----
 ...{suil-0.10.16.ebuild => suil-0.10.16-r1.ebuild} | 37 ++++++++++++++--------
 2 files changed, 34 insertions(+), 21 deletions(-)

Additionally, it has been referenced in the following commit(s):

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

commit b92a8a2f9a01b5ca49218444ab133e887c7ccfec
Author:     Miroslav Šulc <fordfrog@gentoo.org>
AuthorDate: 2022-09-18 13:34:02 +0000
Commit:     Miroslav Šulc <fordfrog@gentoo.org>
CommitDate: 2022-09-18 13:34:28 +0000

    media-sound/ardour: updated python versions, dropped eutils.eclass, updated to changes in media-libs/suil
    
    Bug: https://bugs.gentoo.org/863614
    Signed-off-by: Miroslav Šulc <fordfrog@gentoo.org>

 media-sound/ardour/ardour-6.9-r2.ebuild | 6 +++---
 media-sound/ardour/ardour-9999.ebuild   | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)