1 |
On 04/10/2012 09:30 AM, Brian Harring wrote: |
2 |
> On Tue, Apr 10, 2012 at 08:43:55AM -0700, Zac Medico wrote: |
3 |
>> On 04/10/2012 03:53 AM, Ulrich Mueller wrote: |
4 |
>>> +^[ \t]*EAPI=(['"]?)([A-Za-z0-9+_.-]*)\1[ \t]*(#.*)?$ |
5 |
>> |
6 |
>> We could make the ([A-Za-z0-9+_.-]*) group "optional", but treat it as |
7 |
>> an error if the group is missing. That way, the package manager can |
8 |
>> detect invalid assignments earlier, and can skip sourcing the ebuild in |
9 |
>> this case. |
10 |
> |
11 |
> Or disallow EAPI= . |
12 |
|
13 |
My intention was to catch anything that starts with the ^[ \t]*EAPI= |
14 |
part. For my optional group thing to work, I guess you'd also have to |
15 |
add a (.*)? group to match whatever invalid characters may be there. |
16 |
-- |
17 |
Thanks, |
18 |
Zac |