Support PostgreSQL 9.6 #14

petere · 2015-10-28T20:28:14Z

GetConfigOptionByName() has an additional argument.

jconway · 2016-05-19T16:53:34Z

Thanks!

GetConfigOptionByName() has an additional argument.

* Update compilingplr.md * Update compilingplr.md * fix: issue#2 server process terminated when argument is null to as.numeric(NULL) * automated testing * Update compilingplr.md * fix: issue#2 server process terminated when argument is null to as.numeric(NULL) * added travis-ci file * fixed install of r * fixed install of r * fixed install of r * fixed install of r * fixed install of r * fixed install of r * added tests * added tests * added tests * added tests * added tests * added tests * added tests * added tests * added tests * find psql * find psql * set bindir * try 9.5 * try 9.5 * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * try pg_createcluster * explicitly include pg_config path (#13) * Support PostgreSQL 9.6 (#14) GetConfigOptionByName() has an additional argument. * Starting in pg9.6 SPI_processed is uint64 rather than int.

df7cb · 2016-09-26T21:08:07Z

Hi Joe, hi Dave,

can we have a new release that supports 9.6? I just pulled this patch for the Debian package, but a shiny new tarball would be nicer to base on. (A tag on github is enough.)

Thanks!

jconway · 2016-09-26T21:13:11Z

I'm just boarding a plane to start a holiday. If Dave can create the tag that's great, but if not it will be a few days at least before I can.

Joe

On September 26, 2016 4:08:08 PM CDT, Christoph Berg notifications@github.com wrote:

Hi Joe, hi Dave,

can we have a new release that supports 9.6? I just pulled this patch
for the Debian package, but a shiny new tarball would be nicer to base
on. (A tag on github is enough.)

Thanks!

You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
#14 (comment)

Sent from my Android device with K-9 Mail. Please excuse my brevity.

df7cb · 2016-09-27T07:47:42Z

Hi,

it's not urgent, as long as it happens eventually :). Happy holidays!

davecramer · 2016-09-27T11:02:18Z

I can do it, the problem is which repo to do it out of.

We want to deprecate jconway/plr in favour of
https://github.com/postgres-plr/plr. I'll tag it on the latter. I still
have to make sure the code is the same in both.

Cheers

Dave Cramer

On 27 September 2016 at 03:47, Christoph Berg notifications@github.com
wrote:

Hi,

it's not urgent, as long as it happens eventually :). Happy holidays!

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#14 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYz9uAbv3pZxtq9aUL-kfMn84CNvN0hks5quMoegaJpZM4GXs5o
.

davecramer · 2016-09-27T11:38:50Z

Christoph,

This is more difficult than just a tag. The release version has to be
changed as well. Either way we want you to pull the code from
https://github.com/postgres-plr/plr

Thanks,

Dave Cramer

On 27 September 2016 at 07:01, Dave Cramer davecramer@gmail.com wrote:

I can do it, the problem is which repo to do it out of.

We want to deprecate jconway/plr in favour of https://github.com/
postgres-plr/plr. I'll tag it on the latter. I still have to make sure
the code is the same in both.

Cheers

Dave Cramer

On 27 September 2016 at 03:47, Christoph Berg notifications@github.com
wrote:

Hi,

it's not urgent, as long as it happens eventually :). Happy holidays!

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#14 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYz9uAbv3pZxtq9aUL-kfMn84CNvN0hks5quMoegaJpZM4GXs5o
.

df7cb · 2016-09-27T11:50:54Z

Re: Dave Cramer 2016-09-27 jconway/plr/pull/14/c249840570@github.com

Christoph,

This is more difficult than just a tag. The release version has to be
changed as well.

Aye. I meant to say we (Debian) don't need a full-fledged download
site, the "tags as tarball" feature of github works. (Unless there's
some autoconf or whatnot step involved, of course.)

Either way we want you to pull the code from
https://github.com/postgres-plr/plr

Now that you are mentioning it, you had actually told me before that
you want to move there. I blame github for not showing that repository
when searching for it: https://github.com/search?utf8=%E2%9C%93&q=plr

Maybe update the 1-line note at the top of Joe's repo with a pointer
to the new one?

Thanks,
Christoph

davecramer · 2016-09-27T12:34:10Z

+Devrim

Dave Cramer

On 27 September 2016 at 07:50, Christoph Berg notifications@github.com
wrote:

Re: Dave Cramer 2016-09-27 jconway/plr/pull/14/c249840570@github.com

Christoph,

This is more difficult than just a tag. The release version has to be
changed as well.

Aye. I meant to say we (Debian) don't need a full-fledged download
site, the "tags as tarball" feature of github works. (Unless there's
some autoconf or whatnot step involved, of course.)

Either way we want you to pull the code from
https://github.com/postgres-plr/plr

Now that you are mentioning it, you had actually told me before that
you want to move there. I blame github for not showing that repository
when searching for it: https://github.com/search?utf8=%E2%9C%93&q=plr

Maybe update the 1-line note at the top of Joe's repo with a pointer
to the new one?

Thanks,
Christoph

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#14 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYz9nH3KROy4UvBxShCh5r8jkkXmub8ks5quQMfgaJpZM4GXs5o
.

devrimgunduz · 2016-09-28T10:30:08Z

Hi,

Yeah, I need the new tarball, preferably today, please. Otherwise plr RPM users won't be able to upgrade to 9.6.

Regards, Devrim

davecramer · 2016-09-28T12:55:03Z

I will get it out today but it will be coming from
https://github.com/postgres-plr/plr

Thanks

Dave Cramer

On 28 September 2016 at 06:30, Devrim Gündüz notifications@github.com
wrote:

Hi,

Yeah, I need the new tarball, preferably today, please. Otherwise plr RPM
users won't be able to upgrade to 9.6.

Regards, Devrim

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#14 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYz9ouHxmsccZKsWpzdgFtTNHjQF7Zeks5qukGxgaJpZM4GXs5o
.

Support PostgreSQL 9.6

e516cbc

GetConfigOptionByName() has an additional argument.

jconway merged commit 38ce481 into jconway:master May 19, 2016

davecramer referenced this pull request in postgres-plr/plr May 19, 2016

Support PostgreSQL 9.6 (#14)

f1cc6fa

GetConfigOptionByName() has an additional argument.

petere deleted the support-9.6 branch October 26, 2018 07:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Support PostgreSQL 9.6 #14

Support PostgreSQL 9.6 #14

petere commented Oct 28, 2015

jconway commented May 19, 2016

df7cb commented Sep 26, 2016

jconway commented Sep 26, 2016

df7cb commented Sep 27, 2016

davecramer commented Sep 27, 2016

davecramer commented Sep 27, 2016

df7cb commented Sep 27, 2016

davecramer commented Sep 27, 2016

devrimgunduz commented Sep 28, 2016

davecramer commented Sep 28, 2016

Support PostgreSQL 9.6 #14

Support PostgreSQL 9.6 #14

Conversation

petere commented Oct 28, 2015

jconway commented May 19, 2016

df7cb commented Sep 26, 2016

jconway commented Sep 26, 2016

df7cb commented Sep 27, 2016

davecramer commented Sep 27, 2016

davecramer commented Sep 27, 2016

df7cb commented Sep 27, 2016

davecramer commented Sep 27, 2016

devrimgunduz commented Sep 28, 2016

davecramer commented Sep 28, 2016