Horton

Thread

Commits

  1. On all Windows platforms, not just Cygwin, use _timezone and _tzname.

  1. Patch missing from back branches

    Haroon <muhammad.haroon@2ndquadrant.com> — 2018-04-26T04:06:09Z

    Apparently the following patch[1]  got committed to head only (9.6 at the
time) and never made it into back branches i.e. 9.5, 9.4 and 9.3. Is it an
oversight ?

[1]
http://git.postgresql.org/pg/commitdiff/868628e4fd44d75987d6c099ac63613cc5417629


Regards,
Haroon
-- 
Haroon                http://www.2ndQuadrant.com/
<http://www.2ndquadrant.com/>
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

  2. Re: Patch missing from back branches

    Michael Paquier <michael@paquier.xyz> — 2018-04-26T05:29:50Z

    On Thu, Apr 26, 2018 at 09:06:09AM +0500, Haroon wrote:
> Apparently the following patch[1]  got committed to head only (9.6 at the
> time) and never made it into back branches i.e. 9.5, 9.4 and 9.3. Is it an
> oversight ?
> 
> [1]
> http://git.postgresql.org/pg/commitdiff/868628e4fd44d75987d6c099ac63613cc5417629

The support of VS 2015 being back-patched down to 9.5 is intentional,
as the ideawhen integrating a new integration with visual studio is to
patch HEAD plus the latest major version already released, at least
that's the move respected for the last couple of years.

Now I think that you are right: we should have this patch present as
well in REL9_5_STABLE in order to be able to get the combination
REL9_5_STABLE/VS2015 working correctly.  At least we have made efforts
to claim support of VS2015 down to 9.5, so I would suggest to
cherry-pick it and make the solution complete.

The discussion about _timezone and _tzdata is here by the way:
https://www.postgresql.org/message-id/CAB7nPqTEkqF7E7nWcdQSvyOUAV3OGjX%3DrKSxFTkk03TOJGF5ng%40mail.gmail.com

There is dory in the buildfarm which compiles using VS 2015, but it runs
only 9.6 and newer versions.

Robert?  Or Magnus perhaps?
--
Michael

  3. Re: Patch missing from back branches

    Stephen Frost <sfrost@snowman.net> — 2018-04-28T14:46:22Z

    Greetings,

* Michael Paquier (michael@paquier.xyz) wrote:
> There is dory in the buildfarm which compiles using VS 2015, but it runs
> only 9.6 and newer versions.

That would be specifically because compiling 9.5 didn't work..  We'd be
happy to have dory running on older major versions, if someone is willing
to put in the effort to make it work.

Thanks!

Stephen

  4. Re: Patch missing from back branches

    Andrew Dunstan <andrew.dunstan@2ndquadrant.com> — 2018-04-30T20:12:06Z

    
On 04/26/2018 01:29 AM, Michael Paquier wrote:
> On Thu, Apr 26, 2018 at 09:06:09AM +0500, Haroon wrote:
>> Apparently the following patch[1]  got committed to head only (9.6 at the
>> time) and never made it into back branches i.e. 9.5, 9.4 and 9.3. Is it an
>> oversight ?
>>
>> [1]
>> http://git.postgresql.org/pg/commitdiff/868628e4fd44d75987d6c099ac63613cc5417629
> The support of VS 2015 being back-patched down to 9.5 is intentional,
> as the ideawhen integrating a new integration with visual studio is to
> patch HEAD plus the latest major version already released, at least
> that's the move respected for the last couple of years.
>
> Now I think that you are right: we should have this patch present as
> well in REL9_5_STABLE in order to be able to get the combination
> REL9_5_STABLE/VS2015 working correctly.  At least we have made efforts
> to claim support of VS2015 down to 9.5, so I would suggest to
> cherry-pick it and make the solution complete.
>
> The discussion about _timezone and _tzdata is here by the way:
> https://www.postgresql.org/message-id/CAB7nPqTEkqF7E7nWcdQSvyOUAV3OGjX%3DrKSxFTkk03TOJGF5ng%40mail.gmail.com
>
> There is dory in the buildfarm which compiles using VS 2015, but it runs
> only 9.6 and newer versions.
>
> Robert?  Or Magnus perhaps?


Well, the commit suggests that we should be guided by what the buildfarm
did, and then we never followed that up. I'd say in the light of
experience we should backpatch it to the remaining live branches.

cheers

andrew


-- 
Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

  5. Re: Patch missing from back branches

    Tom Lane <tgl@sss.pgh.pa.us> — 2018-04-30T20:27:26Z

    Andrew Dunstan <andrew.dunstan@2ndquadrant.com> writes:
> Well, the commit suggests that we should be guided by what the buildfarm
> did, and then we never followed that up. I'd say in the light of
> experience we should backpatch it to the remaining live branches.

There are other VS2015 patches that were not pushed back further than
9.5, e.g. 0fb54de9a.  So I don't think we should push this one further
back than that.  But it'd be consistent to put it in 9.5.

			regards, tom lane

  6. Re: Patch missing from back branches

    Tom Lane <tgl@sss.pgh.pa.us> — 2018-05-01T16:04:48Z

    Stephen Frost <sfrost@snowman.net> writes:
> * Michael Paquier (michael@paquier.xyz) wrote:
>> There is dory in the buildfarm which compiles using VS 2015, but it runs
>> only 9.6 and newer versions.

> That would be specifically because compiling 9.5 didn't work..  We'd be
> happy to have dory running on older major versions, if someone is willing
> to put in the effort to make it work.

I've pushed this patch into the 9.5 branch, so it should be possible to
spin dory up on that branch now.

			regards, tom lane

  7. Re: Patch missing from back branches

    Haroon <muhammad.haroon@2ndquadrant.com> — 2018-05-02T07:47:19Z

    On 1 May 2018 at 21:04, Tom Lane <tgl@sss.pgh.pa.us> wrote:

> I've pushed this patch into the 9.5 branch, so it should be possible to
> spin dory up on that branch now.

Thanks Tom!

Thanks everyone else for your input on the issue!

Regards, Haroon

-- 
Haroon                http://www.2ndQuadrant.com/
<http://www.2ndquadrant.com/>
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services