Re: Remove utils/acl.h from catalog/objectaddress.h

2020-03-10 Thread Peter Eisentraut
On 2020-03-09 17:07, Alvaro Herrera wrote: On 2020-Mar-07, Peter Eisentraut wrote: I noticed that catalog/objectaddress.h includes utils/acl.h for no apparent reason. It turns out this used to be needed but not anymore. So removed it and cleaned up the fallout. Patch attached. parser/parse_

Re: Remove utils/acl.h from catalog/objectaddress.h

2020-03-09 Thread Alvaro Herrera
On 2020-Mar-07, Peter Eisentraut wrote: > I noticed that catalog/objectaddress.h includes utils/acl.h for no apparent > reason. It turns out this used to be needed but not anymore. So removed it > and cleaned up the fallout. Patch attached. parser/parse_nodes.h already includes nodes/parsenodes

Re: Remove utils/acl.h from catalog/objectaddress.h

2020-03-08 Thread Tom Lane
Peter Eisentraut writes: > I noticed that catalog/objectaddress.h includes utils/acl.h for no > apparent reason. It turns out this used to be needed but not anymore. > So removed it and cleaned up the fallout. Patch attached. Seems reasonable. One thing I noticed is that if you are including

Remove utils/acl.h from catalog/objectaddress.h

2020-03-06 Thread Peter Eisentraut
, Remote DBA, Training & Services From 6a2603fbaaf7029f1171e87c396b673ec80e7077 Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Sat, 7 Mar 2020 08:19:18 +0100 Subject: [PATCH] Remove utils/acl.h from catalog/objectaddress.h The need for this was remove