From: | Keith Parks <emkxp01(at)mtcc(dot)demon(dot)co(dot)uk> |
---|---|
To: | maillist(at)candle(dot)pha(dot)pa(dot)us |
Cc: | hackers(at)postgresql(dot)org, peter_e(at)gmx(dot)net |
Subject: | Re: [HACKERS] New psql input mode problems |
Date: | 1999-11-07 18:09:42 |
Message-ID: | 199911071809.SAA03018@mtcc.demon.co.uk |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>From: Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us>
>
>> 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,
I hope Peter can confirm that?
One concern is that, currently, we cannot run the regression tests
and are therefore blind to any breakage from patches.
I don't do much for postgresql but I do like to keep an eye out
for breakage on my 2 platforms (SPARC Solaris 7 and S/Linux)
Keith.
From | Date | Subject | |
---|---|---|---|
Next Message | Damond Walker | 1999-11-07 18:45:19 | psql and 6.5.3 |
Previous Message | Bruce Momjian | 1999-11-07 17:55:22 | Re: [HACKERS] New psql input mode problems |