Summary: | gui-wm/wayfire-9999: Missing dependencies | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Alexander Weber <web.alexander> |
Component: | Current packages | Assignee: | No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | jstein, parona |
Priority: | Normal | Keywords: | PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://github.com/gentoo/gentoo/pull/33990 https://github.com/gentoo/gentoo/pull/35748 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Alexander Weber
2023-10-28 16:33:14 UTC
EDIT for issue 1: dev-cpp/nlohmann_json is required too for build The only real missing dependency is sys-auth/seatd which is for wlroots. doxygen and nlohmann_json are red herrings as doxygen is optional for wf-utils and nlohmman_json is for wayfire (it being automagic json support). You failure with system-wlroots is because wayfire was behind in its major version from live wlroots. This is not surprising for live package and you should be expecting this. You should know how to force a live package to version branch that is supported. Referenced this in the pull request due to seatd and nlohmman_json. Despite nlohmann_json being automagic i decided it was to best make it mandatory for the sake of avoiding patching and adding more flags. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a6da6e062fd05e637af7a70fe8644be36b3c579a commit a6da6e062fd05e637af7a70fe8644be36b3c579a Author: Alfred Wingate <parona@protonmail.com> AuthorDate: 2023-11-25 22:54:45 +0000 Commit: Arthur Zamarin <arthurzam@gentoo.org> CommitDate: 2023-12-17 19:55:17 +0000 gui-wm/wayfire: add 0.8.0 Bug: https://bugs.gentoo.org/915795 Bug: https://bugs.gentoo.org/916431 Closes: https://bugs.gentoo.org/831959 Closes: https://bugs.gentoo.org/859256 Signed-off-by: Alfred Wingate <parona@protonmail.com> Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org> gui-wm/wayfire/Manifest | 1 + gui-wm/wayfire/wayfire-0.8.0.ebuild | 158 ++++++++++++++++++++++++++++++++++++ gui-wm/wayfire/wayfire-9999.ebuild | 118 ++++++++++++++++++--------- 3 files changed, 239 insertions(+), 38 deletions(-) The second (versions) issue is still not solved. The wayfire-9999 requires wlroots-17 But wlroots-9999 is version 18 currently. Solved for me by adjusting dependency in ebuild, placed in my overlay https://github.com/bell07/gentoo-bell07_overlay/blob/master/gui-wm/wayfire/wayfire-9999.ebuild I don't think its a good solution for ::gentoo because 9999 its a moving target, best to keep it as 9999 so that users can have better freedom to handle it. If you insist on using the live package with system-wlroots then you can specify wlroots-9999 to use the 17 branch. EGIT_OVERRIDE_BRANCH_WLROOTS_WLROOTS="0.17" The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=49e20c85ca4aada2f761df4c74cc7b2087d49b45 commit 49e20c85ca4aada2f761df4c74cc7b2087d49b45 Author: Alfred Wingate <parona@protonmail.com> AuthorDate: 2024-03-14 11:37:48 +0000 Commit: Arthur Zamarin <arthurzam@gentoo.org> CommitDate: 2024-03-16 19:18:09 +0000 gui-wm/wayfire: update live ebuild * Remove system-wfconfig and system-wlroots use flags, the default will be to use system wf-config and system wlroots. Now that upstream is more active and targets specific wlroots major releases the point of system-wlroots has gone. wf-config has the same upstream so using a bundled version doesn't have much rationale. * Remove dependency on wlroots-9999 and instead require a specific major release of wlroots that could be updated as seen fit. Closes: https://bugs.gentoo.org/916431 Signed-off-by: Alfred Wingate <parona@protonmail.com> Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org> gui-wm/wayfire/wayfire-9999.ebuild | 79 ++++++++------------------------------ 1 file changed, 15 insertions(+), 64 deletions(-) |