Re: [HACKERS] ISN extension bug? (with patch)

2014-01-15 Thread Fabien COELHO
With this caveat, the one-liner patch (4 characters removed) reattached does compile for me: Thanks, applied. Thanks! -- Fabien. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] ISN extension bug? (with patch)

2014-01-13 Thread Heikki Linnakangas
On 01/03/2014 07:53 PM, Fabien COELHO wrote: If so, there is only the one-liner patch to consider. This patch doesn't apply anymore. Please submit an updated patch for the commit fest. In src/include/utils/elog.h there is an include for utils/errcodes.h which is generated somehow when

Re: [HACKERS] ISN extension bug? (with patch)

2014-01-03 Thread Peter Eisentraut
On 12/24/13, 10:29 AM, Fabien COELHO wrote: On 12/22/13, 2:36 AM, Fabien COELHO wrote: I'm not sure whether the policy is to update the version number of the extension for such a change. As the library is always isn.so, two versions cannot live in parallel anyway. If it is useful, the second

Re: [HACKERS] ISN extension bug? (with patch)

2014-01-03 Thread Fabien COELHO
If so, there is only the one-liner patch to consider. This patch doesn't apply anymore. Please submit an updated patch for the commit fest. In src/include/utils/elog.h there is an include for utils/errcodes.h which is generated somehow when compiling postgresql but not present by

Re: [HACKERS] ISN extension bug? (with patch)

2013-12-24 Thread Peter Eisentraut
On 12/22/13, 2:36 AM, Fabien COELHO wrote: I'm not sure whether the policy is to update the version number of the extension for such a change. As the library is always isn.so, two versions cannot live in parallel anyway. If it is useful, the second patch attached also upgrade the version

Re: [HACKERS] ISN extension bug? (with patch)

2013-12-24 Thread Fabien COELHO
On 12/22/13, 2:36 AM, Fabien COELHO wrote: I'm not sure whether the policy is to update the version number of the extension for such a change. As the library is always isn.so, two versions cannot live in parallel anyway. If it is useful, the second patch attached also upgrade the version