Re: getTablePrivileges empty if table owned by group role

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