Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-12-04 Thread Peter Eisentraut
On 11/23/13, 7:12 AM, Mario Weilguni wrote: > Well, in that case and since this is a rarely used extension (I guess > so), maybe it would be the best to simply rename that extension to > uuidossp (or whatever) and don't make any special treatment for it? Why? This is a solved problem, and renamin

Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-11-23 Thread Oskari Saarenmaa
23.11.2013 14:12, Mario Weilguni kirjoitti: Am 22.11.2013 16:15, schrieb Tom Lane: [ memo to self: never, ever accept another contrib module whose name isn't a plain SQL identifier ] Well, in that case and since this is a rarely used extension (I guess so), maybe it would be the best to simply

Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-11-23 Thread Vik Fearing
On 11/23/2013 01:12 PM, Mario Weilguni wrote: > Am 22.11.2013 16:15, schrieb Tom Lane: >> [ memo to self: never, ever accept another contrib module whose name >> isn't a plain SQL identifier ] > > Well, in that case and since this is a rarely used extension (I guess > so), maybe it would be the bes

Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-11-23 Thread Mario Weilguni
Am 22.11.2013 16:15, schrieb Tom Lane: [ memo to self: never, ever accept another contrib module whose name isn't a plain SQL identifier ] Well, in that case and since this is a rarely used extension (I guess so), maybe it would be the best to simply rename that extension to uuidossp (or what

Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-11-22 Thread Andrew Dunstan
On 11/22/2013 10:19 AM, Alvaro Herrera wrote: Tom Lane wrote: roadrunn...@gmx.at writes: regression=# \echo Use '''CREATE EXTENSION "uuid-ossp"''' to load this file. Use 'CREATE EXTENSION "uuid-ossp"' to load this file. Does that look reasonable to people? +1 +1 cheers andrew -- Sent v

Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-11-22 Thread Alvaro Herrera
Tom Lane wrote: > roadrunn...@gmx.at writes: > regression=# \echo Use '''CREATE EXTENSION "uuid-ossp"''' to load this file. > Use 'CREATE EXTENSION "uuid-ossp"' to load this file. > > Does that look reasonable to people? +1 -- Álvaro Herrerahttp://www.2ndQuadrant.com/ PostgreS

Re: [HACKERS] Minor patch for the uuid-ossp extension

2013-11-22 Thread Tom Lane
roadrunn...@gmx.at writes: > When trying to add the extension with \i it writes an error message: > Use "CREATE EXTENSION uuid-ossp" to load this file. > Unfortunatly this does not work for extensions with dashes. Must CREATE > EXTENSION "uuid-ossp". Proposed patch is attached. [ memo to se

[HACKERS] Minor patch for the uuid-ossp extension

2013-11-22 Thread roadrunner6
When trying to add the extension with \i it writes an error message: Use "CREATE EXTENSION uuid-ossp" to load this file. Unfortunatly this does not work for extensions with dashes. Must CREATE EXTENSION "uuid-ossp". Proposed patch is attached. Regards Mario diff -Nurb contrib.orig/uuid-os