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