On 04/10/2012 09:58 AM, Ulrich Mueller wrote:
>>>>>> On Tue, 10 Apr 2012, Brian Harring wrote:
>
>> On Tue, Apr 10, 2012 at 08:43:55AM -0700, Zac Medico wrote:
>>> We could make the ([A-Za-z0-9+_.-]*) group "optional", but treat it
>>> as an error if the group is missing. That way, the package manager
>>> can detect invalid assignments earlier, and can skip sourcing the
>>> ebuild in this case.
>
> Currently there isn't any ebuild in the tree that assigns an empty
> EAPI, so I think we can neglect any performance issues.
My intention was to detect EAPI assignments containing invalid
characters such as spaces or unicode characters. Either way, I agree
it's not really a performance issue. If it makes the specification too
complex for your liking, then I agree it's not worth doing.
--
Thanks,
Zac
|