On 2011-08-23 20:19, Kris Jurka wrote:
>
>
> On Tue, 23 Aug 2011, Jonas Sundman wrote:
>
>> after some digging and further reading in the todo list I learned that acl
>> parsing is known to be uncompatible with versions after 7.3.
>>
>> I decided to have a look into the code and turned up with a working
>> version of the acl parsing, which I would like to share.
>
> Great. Can you resend as a context diff (diff -c), the default format
> isn't really usable. Also we need to able to support the older ACL
> formats as well. I haven't read your patch, but my ancient notes say 7.3,
> 7.4, and 8.0 all have different formats. I know you aren't likely to have
> these old versions handy, but can you test with the attached ACL entries
> and ensure they get parsed correctly?
Thank you for your response, I checked the acl:s, and they all parse correctly.
The interetation of the acl which consists of just one equals sign is a bit
unclear to me, for the moment it gives no grants at all.
I tried to follow the style of the original code but it can certainly be much
more enchanced.
Please find a context diff attached (with some debugging calls in main).
Jonas Sundman