Re: [HACKERS] New psql input mode problems - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [HACKERS] New psql input mode problems
Date
Msg-id 199911071755.MAA00844@candle.pha.pa.us
Whole thread Raw
In response to New psql input mode problems  (Keith Parks <emkxp01@mtcc.demon.co.uk>)
Responses Re: [HACKERS] New psql input mode problems  (Peter Eisentraut <e99re41@DoCS.UU.SE>)
List pgsql-hackers
> Hi all,
> 
> I was wondering why all the regression tests failed for me so i ran one
> in the interactive mode.
> 
> 
> mtcc:[/usr/local/pgsql/src/test/regress](73)% /usr/local/pgsql/bin/psql 
> regression
> Welcome to psql, the PostgreSQL interactive terminal.
> 
> Type:  \copyright for distribution terms
>        \h for help with SQL commands
>        \? for help on internal slash commands
>        \g or terminate with semicolon to execute query
>        \q to quit
> 
> regression=> \i sql/boolean.sql 
> 
> regression=>
> 
> I got nothing onscreen and no work was done.
> 
> After some digging I found that in non interactive mode psql
> stops processing a file as soon as it gets to a blank line.
> 
> This seems to be where it goes wrong. (mainloop.c)
> 
> /* No more input.  Time to quit, or \i done */
> if (line == NULL || (!pset->cur_cmd_interactive && *line == '\0'))
> 
> When a blank line is encountered in the input 
> 
>     line = gets_fromFile(source);
>     
> returns an empty string ('\0') and terminates the processing.
> 
> with the if clause reduced to checking for line == NULL psql
> does the work but fails badly due to the differences between
> results and expected. (comments, QUERY:, echo processing)

> 
> Is the intention to modify expected to agree with the new
> results output, or fix psql to output in the expected format?

Good question.  We need to know if people like the current output
format, or the old one better?

Looks like your change in testing just for NULL is correct, and I will
apply a patch.

--  Bruce Momjian                        |  http://www.op.net/~candle maillist@candle.pha.pa.us            |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] psql and 6.5.3
Next
From: Keith Parks
Date:
Subject: Re: [HACKERS] New psql input mode problems