Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 821721 - app-editors/vis: seems to need dev-lua/lpeg at runtime too
Summary: app-editors/vis: seems to need dev-lua/lpeg at runtime too
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Haelwenn (lanodan) Monnier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-04 21:18 UTC by Sam James
Modified: 2021-11-25 06:16 UTC (History)
1 user (show)

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 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-11-04 21:18:08 UTC
vis seems to use dev-lua/lpeg at runtime (not just for tests). Noticed when a user reported the issue in #gentoo (at runtime the program wouldn't allow using lpeg features like syntax highlighting(?)).

https://github.com/martanne/vis/search?q=lpeg seems consistent too.
Comment 1 Haelwenn (lanodan) Monnier 2021-11-16 23:36:38 UTC
Yes, vis uses (but doesn't requires) lpeg at runtime, which is advertised in the ebuild at install time via optfeature.

I originally put lpeg as dependency of the lua USE flag because it's a small dependency anyway but it got changed during review.
Comment 2 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-11-25 06:16:53 UTC
(In reply to Haelwenn (lanodan) Monnier from comment #1)
> Yes, vis uses (but doesn't requires) lpeg at runtime, which is advertised in
> the ebuild at install time via optfeature.
> 
> I originally put lpeg as dependency of the lua USE flag because it's a small
> dependency anyway but it got changed during review.

I'm sorry, I hadn't noticed. For small dependencies, if it's a big quality of life improvement, I generally prefer to do the flag (if the main package doesn't take long to build either, e.g. pure Python/Lua/etc). Up to you though.