Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 831441 - [guru] app-editors/helix-0.6.0: runtime directory is configured incorrectly
Summary: [guru] app-editors/helix-0.6.0: runtime directory is configured incorrectly
Status: RESOLVED FIXED
Alias: None
Product: GURU
Classification: Unclassified
Component: Package issues (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: degni.public
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-01-19 03:13 UTC by feathecutie
Modified: 2023-03-16 08:41 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 feathecutie 2022-01-19 03:13:50 UTC
The runtime directory that grammars and themes used by app-editors/helix are stored in is configured incorrectly in the ebuild.

When using the ebuild-installed version of heli, grammars and themes are not availible at all, so it seems like helix doesn't detect the runtime directory that gets installed at /usr/share/helix/runtime.

Manually setting the HELIX_RUNTIME environment variable to /usr/share/helix/runtime makes helix work as expected.

From briefly looking at the ebuild, it seems that detecting the runtime directory is implemented using a custom patch, while the official helix README states that packages solve the issue of finding the runtime directory "by wrapping the hx binary with a wrapper that sets the variable to the install dir."

It might be worth considering following this claim and if possible, setting the HELIX_RUNTIME environment variable for helix instead of using a custom patch, but I don't know enough about ebuilds and Gentoo packages to know if this is a reasonable suggestion or not.
Comment 1 Mike Rivnak 2022-10-21 03:52:24 UTC
Confirming that this report is accurate and still relevant.

As for a solution I feel that patching helix to look for the runtime directory in /usr/share/helix is preferred, allowing HELIX_RUNTIME to be set by the user if necessary/desired. That said I don't have experience with packaging to make the determination myself. Bumping this to hopefully get some attention again.
Comment 2 Larry the Git Cow gentoo-dev 2023-03-16 08:41:42 UTC
The bug has been closed via the following commit(s):

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

commit ef332c32ede95f702cb22ebe806c60b44aad18bf
Author:     Florian Schmaus <flow@gentoo.org>
AuthorDate: 2023-03-16 08:37:41 +0000
Commit:     Florian Schmaus <flow@gentoo.org>
CommitDate: 2023-03-16 08:37:41 +0000

    app-editors/helix: treeclean, now in ::gentoo
    
    Closes: https://bugs.gentoo.org/819096
    Closes: https://bugs.gentoo.org/819099
    Closes: https://bugs.gentoo.org/831441
    Closes: https://bugs.gentoo.org/861443
    Closes: https://bugs.gentoo.org/887227
    Closes: https://bugs.gentoo.org/901007
    Signed-off-by: Florian Schmaus <flow@gentoo.org>

 app-editors/helix/Manifest             | 544 ---------------------------------
 app-editors/helix/helix-22.05.ebuild   | 294 ------------------
 app-editors/helix/helix-22.08.1.ebuild | 326 --------------------
 app-editors/helix/helix-22.12.ebuild   | 431 --------------------------
 app-editors/helix/metadata.xml         |  21 --
 5 files changed, 1616 deletions(-)