On Tue, Jan 19, 2016 at 1:49 PM, Michael Paquier
> On Tue, Jan 19, 2016 at 11:08 AM, Joe Conway <m...@joeconway.com> wrote:
>> On 01/18/2016 04:16 PM, Joe Conway wrote:
>>> Please see the attached. Duplication removed.
>> Actually please see this version instead.
> Thanks for the new patch.
> + tuplestore_puttuple(tupstore, tuple);
> + }
> + /*
> + * no longer need the tuple descriptor reference created by
> The patch has some whitespaces.
> +REVOKE ALL on pg_config FROM PUBLIC;
> +REVOKE EXECUTE ON FUNCTION pg_config() FROM PUBLIC;
> I guess that this portion is still under debate :)
> make: Nothing to be done for `all'.
> make -C ../backend submake-errcodes
> make: *** No rule to make target `config_info.o', needed by
> `libpgcommon.a'. Stop.
> make: *** Waiting for unfinished jobs....
> The patch is visibly forgetting to include config_info.c, which should
> be part of src/common.
> + * This function cleans up the paths for use with either cmd.exe or Msys
> + * on Windows. We need them to use filenames without spaces, for which a
> + * short filename is the safest equivalent, eg:
> + * C:/Progra~1/
> + */
> +cleanup_path(char *path)
> Perhaps this refactoring would be useful as a separate patch?
You need as well to update @pgcommonallfiles in Mkvcbuild.pm or the
compilation with MSVC is going to fail.
Sent via pgsql-hackers mailing list (email@example.com)
To make changes to your subscription: