search

jconway / plr

PL/R - R Procedural Language for PostgreSQL
http://www.joeconway.com
90 stars 42 forks source link

Support PostgreSQL 9.6 #14

Closed petere closed 8 years ago

petere commented 8 years ago

GetConfigOptionByName() has an additional argument.

jconway commented 8 years ago

Thanks!

df7cb commented 8 years ago

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 commented 8 years ago

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: https://github.com/jconway/plr/pull/14#issuecomment-249697966

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

df7cb commented 8 years ago

Hi,

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

davecramer commented 8 years ago

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 https://github.com/jconway/plr/pull/14#issuecomment-249791052, or mute the thread https://github.com/notifications/unsubscribe-auth/AAYz9uAbv3pZxtq9aUL-kfMn84CNvN0hks5quMoegaJpZM4GXs5o .

davecramer commented 8 years ago

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 https://github.com/jconway/plr/pull/14#issuecomment-249791052, or mute the thread https://github.com/notifications/unsubscribe-auth/AAYz9uAbv3pZxtq9aUL-kfMn84CNvN0hks5quMoegaJpZM4GXs5o .

df7cb commented 8 years ago

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 commented 8 years ago

+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 https://github.com/jconway/plr/pull/14#issuecomment-249842947, or mute the thread https://github.com/notifications/unsubscribe-auth/AAYz9nH3KROy4UvBxShCh5r8jkkXmub8ks5quQMfgaJpZM4GXs5o .

devrimgunduz commented 8 years ago

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 commented 8 years ago

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 https://github.com/jconway/plr/pull/14#issuecomment-250130617, or mute the thread https://github.com/notifications/unsubscribe-auth/AAYz9ouHxmsccZKsWpzdgFtTNHjQF7Zeks5qukGxgaJpZM4GXs5o .