pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Nikita Glukhov <n.gluhov@postgrespro.ru>
To: Thomas Munro <thomas.munro@enterprisedb.com>
Cc: David Steele <david@pgmasters.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Cc: Alexander Korotkov <a.korotkov@postgrespro.ru>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Oleg Bartunov <obartunov@gmail.com>
Cc: Pavel Stehule <pavel.stehule@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: jsonpath
Date: Mon, 2 Jul 2018 14:56:02 +0300
Message-ID: <3bc15ece-5219-e4ca-0b5c-fcd2b276aa26@postgrespro.ru> (raw)
In-Reply-To: <CAEepm=0jJ4fkEWLHc2Rqt5F4Und6qd+60pxFQmCnbf0nYYjrcg@mail.gmail.com>
References: <fcc6fc6a-b497-f39a-923d-aa34d0c588e8@2ndQuadrant.com>
	<48f13b75-0be7-c356-ff26-1db743add56e@postgrespro.ru>
	<7b4b8973-43f5-71c5-bf86-51cb5798dd88@2ndQuadrant.com>
	<f0b44fe7-d47a-eff2-592c-d9dbe4a63f75@2ndQuadrant.com>
	<5290ddd1-f23e-a53a-cae0-830dbde0a5fe@2ndQuadrant.com>
	<604ffe9c-b42e-7c42-e16b-aa44c3a12c3c@postgrespro.ru>
	<927f72f0-6832-a4ca-5676-b0ab5a2895d4@postgrespro.ru>
	<CAF4Au4wxHuEeZrMxxKXUYcs1kDws-f0oW0-sEyun388-U-NOxw@mail.gmail.com>
	<9a1ac993-7608-797c-2095-c735605da82f@postgrespro.ru>
	<CA+TgmoYXtoPPaV-OEUfqujCcfKp+ZT=KOuPHkhRoQL_iYzQ8yQ@mail.gmail.com>
	<94ace92c-714c-903c-fab2-31fb838204ad@postgrespro.ru>
	<CAA8=A796mY7kJdKARt7B+faW70-c8mRh+Ff_joBJ+NyCFLv3Gg@mail.gmail.com>
	<19506.1521524178@sss.pgh.pa.us>
	<880bcbc6-1dba-cf82-7945-c10528f602c4@postgrespro.ru>
	<892ae4c4-e549-ab21-c57c-37c5fd47fda0@pgmasters.net>
	<8dd1cd76-cd4e-9be7-9515-ef0b8b21cb67@postgrespro.ru>
	<CAEepm=0jJ4fkEWLHc2Rqt5F4Und6qd+60pxFQmCnbf0nYYjrcg@mail.gmail.com>

On 28.06.2018 05:39, Thomas Munro wrote:

> On Thu, Jun 28, 2018 at 11:38 AM, Nikita Glukhov
> <n.gluhov@postgrespro.ru> wrote:
>> Attached 15th version of the patches.
> Hi Nikita,
>
> I wonder why the Windows build scripts are not finding and processing
> your new .y and .l files:
>
> https://ci.appveyor.com/project/postgresql-cfbot/postgresql/build/1.0.3672
>
> Perhaps Mkvcbuild.pm needs to be told about them?  For example, it has this:
>
>      $postgres->AddFiles('src/backend/parser', 'scan.l', 'gram.y');
Yes, generation of jsonpath_gram.* was missing in Mkvcbuild.pm.

I have fixed it in the new 16th version of patches, and now it seems to 
work on
Windows (https://ci.appveyor.com/project/NikitaGlukhov/postgres/build/1.0.4)

> PS If you need a way to test on Windows but don't have a local Windows
> build system, this may be useful:
>
> https://wiki.postgresql.org/wiki/Continuous_Integration

Thank, this is really helpful.

-- 
Nikita Glukhov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

Attachments:

  [application/gzip] sqljson_jsonpath_v16.tar.gz (85.1K, ../3bc15ece-5219-e4ca-0b5c-fcd2b276aa26@postgrespro.ru/2-sqljson_jsonpath_v16.tar.gz)
  download

view thread (198+ messages)  latest in thread

Message-ID: <3bc15ece-5219-e4ca-0b5c-fcd2b276aa26@postgrespro.ru>
Permalink:  ../3bc15ece-5219-e4ca-0b5c-fcd2b276aa26@postgrespro.ru/
Also on:    postgresql.org/message-id/3bc15ece-5219-e4ca-0b5c-fcd2b276aa26@postgrespro.ru

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: n.gluhov@postgrespro.ru, thomas.munro@enterprisedb.com, david@pgmasters.net, tgl@sss.pgh.pa.us, andrew.dunstan@2ndquadrant.com, a.korotkov@postgrespro.ru, robertmhaas@gmail.com, obartunov@gmail.com, pavel.stehule@gmail.com
  Subject: Re: jsonpath
  In-Reply-To: <3bc15ece-5219-e4ca-0b5c-fcd2b276aa26@postgrespro.ru>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox