Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-10 Thread Pavel Stehule
2014-09-10 0:13 GMT+02:00 Andres Freund : > On 2014-09-09 22:22:45 +0200, Andres Freund wrote: > > I plan to push this soon. > > Done. > > Thanks for the patch! > Thank you very much Pavel > > Andres Freund > > -- > Andres Freund http://www.2ndQuadrant.com/ > PostgreSQL D

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-09 Thread Andres Freund
On 2014-09-09 22:22:45 +0200, Andres Freund wrote: > I plan to push this soon. Done. Thanks for the patch! Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-09 Thread Andres Freund
Hi, Given we already have three topics for --help and I can see others I went with my --help= proposal. On 2014-08-28 13:20:07 +0200, Andres Freund wrote: > Some stuff I changed: > * I rephrased the sgml changes > * s/Printing options/Display options/. Or maybe "Display influencing > variables"

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-09 Thread Andres Freund
On 2014-08-28 13:54:28 +0200, Andres Freund wrote: > On 2014-08-28 13:20:07 +0200, Andres Freund wrote: > > I've attached a incremental patch. > > Apparently I didn't attach the patch, as so much a file containing the > name of the patchfile... Which you obviously didn't integrate. And didn't com

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-04 Thread Pavel Stehule
Hi here is a second variant with support --help=variables Regards Pavel 2014-09-04 4:25 GMT+02:00 Robert Haas : > On Thu, Aug 28, 2014 at 11:20 AM, Andres Freund > wrote: > >> >* How about making it --help=variables instead of --help-variables? > >> > >> -1, help is not a variable to be assi

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-03 Thread Robert Haas
On Thu, Aug 28, 2014 at 11:20 AM, Andres Freund wrote: >> >* How about making it --help=variables instead of --help-variables? >> >> -1, help is not a variable to be assigned imho > > I don't think variable assignment is a good mental model for long > commandline arguments. And it's not like I'm t

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-03 Thread Pavel Stehule
Hello fixed ECHO, ECHO_HIDDEN, PROPMPT Regards Pavel 2014-09-01 11:52 GMT+02:00 Fujii Masao : > On Thu, Aug 28, 2014 at 9:34 PM, Pavel Stehule > wrote: > > > > > > > > 2014-08-28 14:22 GMT+02:00 Fujii Masao : > > > >> On Thu, Aug 28, 2014 at 5:48 AM, Pavel Stehule > > >> wrote: > >> > comm

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-09-01 Thread Fujii Masao
On Thu, Aug 28, 2014 at 9:34 PM, Pavel Stehule wrote: > > > > 2014-08-28 14:22 GMT+02:00 Fujii Masao : > >> On Thu, Aug 28, 2014 at 5:48 AM, Pavel Stehule >> wrote: >> > comments? >> >> +fprintf(output, _(" ECHO control what input is >> written to standard output [all, queries]

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Andres Freund
On 2014-08-28 14:04:27 +0200, Petr Jelinek wrote: > On 28/08/14 13:20, Andres Freund wrote: > >Hi, > > > >Stuff I wondered about: > >* How about making it --help=variables instead of --help-variables? > > -1, help is not a variable to be assigned imho I don't think variable assignment is a good m

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Pavel Stehule
2014-08-28 14:22 GMT+02:00 Fujii Masao : > On Thu, Aug 28, 2014 at 5:48 AM, Pavel Stehule > wrote: > > comments? > > +fprintf(output, _(" ECHO control what input is > written to standard output [all, queries]\n")); > > The valid values in the help messages should be consistent

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Fujii Masao
On Thu, Aug 28, 2014 at 5:48 AM, Pavel Stehule wrote: > comments? +fprintf(output, _(" ECHO control what input is written to standard output [all, queries]\n")); The valid values in the help messages should be consistent with the values that the tab-completion displays. So in

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Petr Jelinek
On 28/08/14 13:20, Andres Freund wrote: Hi, Stuff I wondered about: * How about making it --help=variables instead of --help-variables? -1, help is not a variable to be assigned imho * Do we really need the 'Report bugs to' blurb for --help-variables? Probably not. * Since we're not exha

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Pavel Stehule
2014-08-28 13:20 GMT+02:00 Andres Freund : > Hi, > > On 2014-08-27 22:48:54 +0200, Pavel Stehule wrote: > > Hi > > > > I chose \? xxx, because it is related to psql features. I wrote commands: > > > > \? options > > \? variables > > > > comments? > > Generall I like it. > > Some stuff I changed: >

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Andres Freund
On 2014-08-28 13:20:07 +0200, Andres Freund wrote: > I've attached a incremental patch. Apparently I didn't attach the patch, as so much a file containing the name of the patchfile... Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Develo

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Fujii Masao
On Thu, Aug 28, 2014 at 8:20 PM, Andres Freund wrote: > Hi, > > On 2014-08-27 22:48:54 +0200, Pavel Stehule wrote: >> Hi >> >> I chose \? xxx, because it is related to psql features. I wrote commands: >> >> \? options >> \? variables >> >> comments? > > Generall I like it. > > Some stuff I changed

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-28 Thread Andres Freund
Hi, On 2014-08-27 22:48:54 +0200, Pavel Stehule wrote: > Hi > > I chose \? xxx, because it is related to psql features. I wrote commands: > > \? options > \? variables > > comments? Generall I like it. Some stuff I changed: * I rephrased the sgml changes * s/Printing options/Display options/.

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-27 Thread Pavel Stehule
Hi I chose \? xxx, because it is related to psql features. I wrote commands: \? options \? variables comments? Regards Pavel 2014-08-26 13:48 GMT+02:00 Andres Freund : > On 2014-08-26 13:44:16 +0200, Pavel Stehule wrote: > > 2014-08-26 13:30 GMT+02:00 Petr Jelinek : > > > > > On 26/08/14 1

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-26 Thread Andres Freund
On 2014-08-26 13:44:16 +0200, Pavel Stehule wrote: > 2014-08-26 13:30 GMT+02:00 Petr Jelinek : > > > On 26/08/14 13:20, Andres Freund wrote: > > > >> > >> I'm looking at committing this, but I wonder: Shouldn't this be > >> accessible from inside psql as well? I.e. as a backslash command? > >> > >

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-26 Thread Pavel Stehule
2014-08-26 13:30 GMT+02:00 Petr Jelinek : > On 26/08/14 13:20, Andres Freund wrote: > >> >> I'm looking at committing this, but I wonder: Shouldn't this be >> accessible from inside psql as well? I.e. as a backslash command? >> >> > +1 > have you idea about command name? \?+ Pavel > > > --

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-26 Thread Petr Jelinek
On 26/08/14 13:20, Andres Freund wrote: I'm looking at committing this, but I wonder: Shouldn't this be accessible from inside psql as well? I.e. as a backslash command? +1 -- Petr Jelinek http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Servic

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-26 Thread Andres Freund
On 2014-06-29 18:54:50 +0530, Abhijit Menon-Sen wrote: > At 2014-06-29 20:35:04 +0900, maumau...@gmail.com wrote: > > > > Thanks, I marked it as ready for committer. I hope Fujii san or > > another committer will commit this, refining English expression if > > necessary. > > Since it was just a m

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-08-19 Thread Alvaro Herrera
Abhijit Menon-Sen wrote: > At 2014-06-29 20:35:04 +0900, maumau...@gmail.com wrote: > > > > Thanks, I marked it as ready for committer. I hope Fujii san or > > another committer will commit this, refining English expression if > > necessary. > > Since it was just a matter of editing, I went throu

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-29 Thread Pavel Stehule
2014-06-29 15:24 GMT+02:00 Abhijit Menon-Sen : > At 2014-06-29 20:35:04 +0900, maumau...@gmail.com wrote: > > > > Thanks, I marked it as ready for committer. I hope Fujii san or > > another committer will commit this, refining English expression if > > necessary. > > Since it was just a matter of

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-29 Thread Abhijit Menon-Sen
At 2014-06-29 20:35:04 +0900, maumau...@gmail.com wrote: > > Thanks, I marked it as ready for committer. I hope Fujii san or > another committer will commit this, refining English expression if > necessary. Since it was just a matter of editing, I went through the patch and corrected various mino

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-29 Thread Pavel Stehule
2014-06-29 13:35 GMT+02:00 MauMau : > Thanks, I marked it as ready for committer. I hope Fujii san or another > committer will commit this, refining English expression if necessary. > sure Thank you very much Regards Pavel > > Regards > MauMau > >

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-29 Thread MauMau
Thanks, I marked it as ready for committer. I hope Fujii san or another committer will commit this, refining English expression if necessary. Regards MauMau -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/m

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-28 Thread Pavel Stehule
Hi 2014-06-29 0:48 GMT+02:00 MauMau : > From: "Pavel Stehule" > > I modified description of setting system variables in dependency on O.S. >> > > Thank you, it's almost OK. As mentioned in my previous mail, I think > "determines" should be "determine" to follow other messages. I'll mark > th

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-28 Thread MauMau
From: "Pavel Stehule" I modified description of setting system variables in dependency on O.S. Thank you, it's almost OK. As mentioned in my previous mail, I think "determines" should be "determine" to follow other messages. I'll mark this patch as ready for committer when this is fixed.

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-28 Thread Pavel Stehule
Hello I modified description of setting system variables in dependency on O.S. Regards Pavel 2014-06-27 8:54 GMT+02:00 Pavel Stehule : > Hello > > thank you Peter, so now only setting for MS Windows is missing? > > Regards > > Pavel > > > 2014-06-26 21:57 GMT+02:00 Petr Jelinek : > > Hello,

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-26 Thread Pavel Stehule
Hello thank you Peter, so now only setting for MS Windows is missing? Regards Pavel 2014-06-26 21:57 GMT+02:00 Petr Jelinek : > Hello, > > I went through the patch, seems mostly fine, I adjusted some wording, > removed the default .pgpass file info since it's not accurate, and replaced > coup

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-26 Thread Petr Jelinek
Hello, I went through the patch, seems mostly fine, I adjusted some wording, removed the default .pgpass file info since it's not accurate, and replaced couple of phrases with (hopefully) more informative ones. -- Petr Jelinek http://www.2ndQuadrant.com/ PostgreSQL Develop

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-26 Thread Pavel Stehule
2014-06-26 15:26 GMT+02:00 MauMau : > Hello, > > From: "Pavel Stehule" > >> fixed >> > > Thank you. All is fine. > > > > should be ""Environment variables". And this section lacks description >>> for Windows, such as: >>> >>> + printf(_(" NAME=VALUE [NAME=VALUE] psql ...\n or \\setenv NAME >

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-26 Thread MauMau
Hello, From: "Pavel Stehule" fixed Thank you. All is fine. should be ""Environment variables". And this section lacks description for Windows, such as: + printf(_(" NAME=VALUE [NAME=VALUE] psql ...\n or \\setenv NAME [VALUE] in interactive mode\n\n")); + printf(_(" PGPASSFILE

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-25 Thread Pavel Stehule
Hello Here is next update 2014-06-25 17:17 GMT+02:00 MauMau : > OK, let me help you, though I'm only a Japanese who is never confident in > my English. > > (1) > As Fujii-san pointed out, could you add explanation for --help-variables > in doc/src/sgml/ref/psqlref.sgml? > > > (2) > > + printf(_

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-25 Thread MauMau
OK, let me help you, though I'm only a Japanese who is never confident in my English. (1) As Fujii-san pointed out, could you add explanation for --help-variables in doc/src/sgml/ref/psqlref.sgml? (2) + printf(_(" --help-variables list of available configuration variables (options)

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Pavel Stehule
Hello I am sending little bit modified patch by Fujii' comments - but I am not able to fix it more - it is task for someone with better English skill then I have Regards Pavel 2014-06-23 10:02 GMT+02:00 Fujii Masao : > On Mon, Jun 23, 2014 at 12:04 AM, Pavel Stehule > wrote: > > Hello > > >

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Pavel Stehule
2014-06-23 11:53 GMT+02:00 Fujii Masao : > On Mon, Jun 23, 2014 at 6:06 PM, Pavel Stehule > wrote: > > > > > > > > 2014-06-23 10:57 GMT+02:00 Fujii Masao : > > > >> On Mon, Jun 23, 2014 at 5:10 PM, Pavel Stehule > > >> wrote: > >> > Hello > >> > > >> > > >> > 2014-06-23 10:02 GMT+02:00 Fujii Mas

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Fujii Masao
On Mon, Jun 23, 2014 at 6:06 PM, Pavel Stehule wrote: > > > > 2014-06-23 10:57 GMT+02:00 Fujii Masao : > >> On Mon, Jun 23, 2014 at 5:10 PM, Pavel Stehule >> wrote: >> > Hello >> > >> > >> > 2014-06-23 10:02 GMT+02:00 Fujii Masao : >> > >> >> On Mon, Jun 23, 2014 at 12:04 AM, Pavel Stehule >> >>

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Pavel Stehule
2014-06-23 10:57 GMT+02:00 Fujii Masao : > On Mon, Jun 23, 2014 at 5:10 PM, Pavel Stehule > wrote: > > Hello > > > > > > 2014-06-23 10:02 GMT+02:00 Fujii Masao : > > > >> On Mon, Jun 23, 2014 at 12:04 AM, Pavel Stehule < > pavel.steh...@gmail.com> > >> wrote: > >> > Hello > >> > > >> > third vers

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Fujii Masao
On Mon, Jun 23, 2014 at 5:10 PM, Pavel Stehule wrote: > Hello > > > 2014-06-23 10:02 GMT+02:00 Fujii Masao : > >> On Mon, Jun 23, 2014 at 12:04 AM, Pavel Stehule >> wrote: >> > Hello >> > >> > third version with Erik's update >> >> Here are some my comments: >> >> The document of psql needs to be

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Pavel Stehule
Hello 2014-06-23 10:02 GMT+02:00 Fujii Masao : > On Mon, Jun 23, 2014 at 12:04 AM, Pavel Stehule > wrote: > > Hello > > > > third version with Erik's update > > Here are some my comments: > > The document of psql needs to be updated. At least the description of new > option > this patch adds ne

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-23 Thread Fujii Masao
On Mon, Jun 23, 2014 at 12:04 AM, Pavel Stehule wrote: > Hello > > third version with Erik's update Here are some my comments: The document of psql needs to be updated. At least the description of new option this patch adds needs to be added into the document. +printf(_(" --help-variables

Re: [Fwd: Re: [HACKERS] proposal: new long psql parameter --on-error-stop]

2014-06-22 Thread Pavel Stehule
Hello third version with Erik's update Thanks Erik Regards Pavel 2014-06-22 12:01 GMT+02:00 Erik Rijkers : > Hi Pavel, > > It seems you overlooked the patch that I sent? > > There are some typo's in your patch (also in v2) like: > > PROPMPT1, PROPMT2, PROPMPT3 > > SIGLELINE > > I fixed those