[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#859776: sponsorship-requests: vis/0.3 [ITP] -- A a modern and legacy free Vim-like text editor



On 2017-04-07 19:54, Adam Borowski wrote:
> On Fri, Apr 07, 2017 at 05:52:21PM +0200, Paride Legovini wrote:
>> On 2017-04-07 15:39, Adam Borowski wrote:
>>> On Fri, Apr 07, 2017 at 12:54:22PM +0200, Paride Legovini wrote:
>>>>  * Package name    : vis
>>>>    Version         : 0.3
> 
> Seems to be fine, except for two error messages I get:
> 
> /usr/share/vis/themes/default-256.lua:23: attempt to index a nil value (local 'lexers')
> WARNING: could not find lpeg module
> 

The package now Recommends: lua-lpeg, that is the dependency you are
missing. The upstream README.md lists it as an "optional runtime
dependency", so I think Recommends: is appropriate.

I also changed the priorities for the alternatives for 'editor' and 'vi'
(see debian/postinst). The rationale is that it's OK to prefer vis to
very basic flavors of vim, lettings the ones with more features win. I
also didn't want to override nano, as it is the editor suggested to less
experienced users. Seems reasonable to me, but I'd like to have some
feedback.

Paride


Reply to: