>> grant/revoke does not work in NetBSD/m68k. This is due to the wrong
>> assumption that sizeof(AclItem) is equal to 8 in all platforms. I am
>> going to fix this by replacing all occurrence of sizeof(AclItem) to
>> ACLITEM_SIZE (newly defined as 8 in catalog/pg_type.h). See included
>> patches. If there's no objection, I will commit them. Comments?
>
>I do not like this patch at *all*. Why is sizeof(AclItem) not the
>correct thing to use?
In NetBSD/m68k sizeof(AclItem) = 6, not 8.
>Replacing it with a hardwired "8" seems like
>a step backwards --- not to mention a direct contradiction of what
>you claim the patch is doing.
It's already hard wired in pg_type.h, isn't it.
>Perhaps the real problem is that the AclItem struct definition needs
>modification? Or maybe we need a way to put a machine-dependent size
>into the pg_type entry for type aclitem? The latter seems like a
>good thing to be able to do on general principles.
Glad to hear you have better idea. Anyway, NetBSD/m68k users need some
way to fix the problem now, since the problem seems very serious.
--
Tatsuo Ishii
