nano-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] syntax: gentoo: some cleaning and modernizing


From: Benno Schulenberg
Subject: Re: [PATCH] syntax: gentoo: some cleaning and modernizing
Date: Fri, 25 Jun 2021 17:19:58 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0

Op 24-06-2021 om 15:28 schreef David Michael:
> It looks like the games functions were just removed last month, so
> yes, they could be dropped.

Dropped, since its eclass has been deprecated for five years:
    https://bugs.gentoo.org/574082

>> Also, "dojar" and "dopython" get highlighted, but I don't see them in the 
>> specs
>> either.  Obsolete too?
> 
> They seem to have been builtins long ago that were replaced by other
> methods, so they can be dropped.

Dropped.

With these changes I've pushed your patch:
    https://git.savannah.gnu.org/cgit/nano.git/commit/?id=7b4a835d

>  Note that some of the other
> highlighted install functions (dopam, domenu, etc.) will not be found
> in the spec either, but they are defined in eclasses.

I do see domenu and doicon in the eclasses, but not dopam -- only dopamd,
dopammod and dopamsecurity.  So probably 'dopam' should be dropped?  And
maybe 'docron' and some others should be added instead?

Looking further, I don't see 'epause', 'ebeep' and 'epunt_cxx' anywhere.
Should they be dropped too?  If 'ecvs_clean' and 'esvn_clean' are colored,
why not also 'egit_clean'?  Or maybe these should not be colored at all
(when they are only rarely used, for example)?

The "inherit.*" regex looks too broad: it will color also the semicolons
that occur after its arguments in several eclasses.

And probably there's other stuff that needs adjustment.  So you may want
to supply a further patch?

Benno

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]