Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 937117 - app-editors/imhex should not have hardcoded dependency to sys-apps/xdg-desktop-portal
Summary: app-editors/imhex should not have hardcoded dependency to sys-apps/xdg-deskto...
Status: RESOLVED FIXED
Alias: None
Product: GURU
Classification: Unclassified
Component: Package issues (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gasc Henri
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-08-02 07:49 UTC by spacok
Modified: 2024-08-31 13:05 UTC (History)
1 user (show)

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


Attachments
updated ebuild for imhex-1.35.4 (imhex-1.35.4.ebuild,2.72 KB, text/plain)
2024-08-02 07:51 UTC, spacok
Details
Simplified ebuild including fix (imhex-1.35.4.ebuild,2.64 KB, text/plain)
2024-08-03 17:23 UTC, spacok
Details

Note You need to log in before you can comment on or make changes to this bug.
Description spacok 2024-08-02 07:49:08 UTC
imhex-1.35.4 has hardcoded dependency to xdg-desktop-portal and it's not necessary (it's controllable by cmake parameters). We should have a use flag.

Reproducible: Always
Comment 1 spacok 2024-08-02 07:51:21 UTC
Created attachment 898804 [details]
updated ebuild for imhex-1.35.4

Proposed solution to have use flag controling xdg-desktop-portal dependency
Comment 2 spacok 2024-08-02 07:51:45 UTC
Proposed fix is attached
Comment 3 spacok 2024-08-03 17:13:26 UTC
Comment on attachment 898804 [details]
updated ebuild for imhex-1.35.4

># Copyright 1999-2024 Gentoo Authors
># Distributed under the terms of the GNU General Public License v2
>
>EAPI=8
>
>LLVM_COMPAT=( {15..18} )
>
>inherit cmake llvm-r1 toolchain-funcs
>
>DESCRIPTION="A hex editor for reverse engineers, programmers, and eyesight"
>HOMEPAGE="https://github.com/WerWolv/ImHex"
>SRC_URI="
>	https://github.com/WerWolv/ImHex/releases/download/v${PV}/Full.Sources.tar.gz -> ${P}.gh.tar.gz
>	https://github.com/WerWolv/ImHex-Patterns/archive/refs/tags/ImHex-v${PV}.tar.gz -> ${PN}-patterns-${PV}.gh.tar.gz
>"
>S="${WORKDIR}/ImHex"
>S_PATTERNS="${WORKDIR}/ImHex-Patterns-ImHex-v${PV}"
>
>LICENSE="GPL-2"
>SLOT="0"
>KEYWORDS="~amd64"
>IUSE="+system-llvm test lto desktop-portal"
>RESTRICT="!test? ( test )"
>
>PATCHES=(
>	# If virtual/dotnet-sdk is installed on your system, then cmake
>	# will use it at some point and try to access internet.
>	# Because it did not cause any issue, we can disable it
>	"${FILESDIR}/remove_dotnet.patch"
>	# Remove the different -Werror flags
>	"${FILESDIR}/remove_Werror.patch"
>)
>
>DEPEND="
>	app-arch/bzip2
>	app-arch/xz-utils
>	app-arch/zstd:=
>	app-forensics/yara:=
>	>=dev-cpp/nlohmann_json-3.10.2
>	dev-libs/capstone:=
>	>=dev-libs/nativefiledialog-extended-1.2.0:=
>	>=dev-libs/libfmt-8.0.0:=
>	media-libs/freetype
>	>=media-libs/glfw-3.4[X]
>	media-libs/glm
>	media-libs/libglvnd
>	net-libs/mbedtls:=
>	net-misc/curl
>	sys-apps/file
>	sys-libs/zlib
>	virtual/libiconv
>	virtual/libintl
>"
>RDEPEND="${DEPEND}"
>BDEPEND="
>	system-llvm? ( sys-devel/llvm )
>	app-admin/chrpath
>	gnome-base/librsvg
>	desktop-portal? ( sys-apps/xdg-desktop-portal )
>"
>
>pkg_pretend() {
>	if tc-is-gcc && [[ $(gcc-major-version) -lt 12 ]]; then
>		die "${PN} requires GCC 12 or newer"
>	fi
>}
>
>src_unpack() {
>	default
>
>	mv "${WORKDIR}/ImHex-Patterns-ImHex-v${PV}" "${S}/ImHex-Patterns"
>}
>
>src_configure() {
>	if use test; then
>		sed -ie "s/tests EXCLUDE_FROM_ALL/tests ALL/" "${S}/CMakeLists.txt"
>	fi
>
>	local mycmakeargs=(
>		-D IMHEX_PLUGINS_IN_SHARE=OFF \
>		-D IMHEX_STRIP_RELEASE=OFF \
>		-D IMHEX_OFFLINE_BUILD=ON \
>		-D IMHEX_IGNORE_BAD_CLONE=ON \
>		-D IMHEX_PATTERNS_PULL_MASTER=OFF \
>		-D IMHEX_IGNORE_BAD_COMPILER=OFF \
>		-D IMHEX_DISABLE_STACKTRACE=ON \
>		-D IMHEX_BUNDLE_DOTNET=OFF \
>		-D IMHEX_ENABLE_LTO=$(usex lto) \
>		-D IMHEX_USE_DEFAULT_BUILD_SETTINGS=OFF \
>		-D IMHEX_STRICT_WARNINGS=OFF \
>		-D IMHEX_ENABLE_UNIT_TESTS=$(usex test) \
>		-D IMHEX_ENABLE_PRECOMPILED_HEADERS=OFF \
>               -D IMHEX_USE_GTK_FILE_PICKER=$(usex desktop-portal) \
>		-D IMHEX_VERSION="${PV}" \
>		-D PROJECT_VERSION="${PV}" \
>		-D USE_SYSTEM_CAPSTONE=ON \
>		-D USE_SYSTEM_FMT=ON \
>		-D USE_SYSTEM_LLVM=$(usex system-llvm) \
>		-D USE_SYSTEM_NFD=ON \
>		-D USE_SYSTEM_NLOHMANN_JSON=ON \
>		-D USE_SYSTEM_YARA=ON
>	)
>
>	cmake_src_configure
>}
Comment 4 spacok 2024-08-03 17:23:03 UTC
Created attachment 898908 [details]
Simplified ebuild including fix

Sorry for mess in this bug but the UI is... let's say horrible :)
Comment 5 Larry the Git Cow gentoo-dev 2024-08-31 13:05:56 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=4455c1b4b4aa869bd921d357753ae9c84f8dc3d8

commit 4455c1b4b4aa869bd921d357753ae9c84f8dc3d8
Author:     spacok <spacok@protonmail.com>
AuthorDate: 2024-08-31 11:51:05 +0000
Commit:     Henri Gasc <gasc@eurecom.fr>
CommitDate: 2024-08-31 11:51:44 +0000

    app-editors/imhex: add desktop-portal USE flag
    
    Closes: https://bugs.gentoo.org/937117
    Signed-off-by: Henri Gasc <gasc@eurecom.fr>

 app-editors/imhex/{imhex-1.35.4.ebuild => imhex-1.35.4-r1.ebuild} | 8 ++++----
 app-editors/imhex/metadata.xml                                    | 1 +
 2 files changed, 5 insertions(+), 4 deletions(-)