Re: Mehdi Dogguy 2015-02-22 <20150222174611.gb14...@dogguy.org>
> On Sun, Feb 22, 2015 at 03:32:26PM +0100, Christoph Berg <m...@debian.org> 
> wrote:
> > + 
> > +-\set ECHO all;
> > ++\set ECHO none
> > +
> 
> Is it missing a semi-colon?

No, to the contrary: The parser didn't understand "all;" and
substituted it by "none" internally. The current test failure is
because the previously silent substitution is now accompanied by a
warning:

*** /tmp/adt-run.NG5yKZ/build.gjP/orafce-3.0.7/expected/dbms_pipe_session_A.out 
2014-07-27 20:10:01.000000000 +0000
--- /tmp/adt-run.NG5yKZ/build.gjP/orafce-3.0.7/results/dbms_pipe_session_A.out  
2015-02-21 06:53:26.265125313 +0000
***************
*** 9,14 ****
--- 9,15 ----
              0
  (1 row)
  
+ unrecognized value "all;" for "ECHO"; assuming "none"
   createimplicitpipe 
  --------------------
   


The minimal fix is to just replace "all;" by what the parser was doing
internally anyway. (The proper fix that was applied upstream is to use
"all", but then the rest of the expected output file needs dozens of
changes because everything is echoed.)

Christoph
-- 
e...@df7cb.de | http://www.df7cb.de/

Attachment: signature.asc
Description: Digital signature

Reply via email to