Re: [BUGS] BUG #7817: psql does not relate to footer settings in extended mode

2013-09-10 Thread Bruce Momjian

Applied.

---

On Mon, Sep  9, 2013 at 09:16:05PM -0400, Bruce Momjian wrote:
> On Sun, Jan 20, 2013 at 10:33:37AM +, emes...@redhat.com wrote:
> > The following bug has been logged on the website:
> > 
> > Bug reference:  7817
> > Logged by:  Eli Mesika
> > Email address:  emes...@redhat.com
> > PostgreSQL version: 9.1.7
> > Operating system:   Fedora 16
> > Description:
> > 
> > psql does not relate to footer settings in extended mode
> > Sometimes we need to run a sql command withot generating header and footer.
> > This can be done using the -t flag and --pset=footer=off
> > The problem is that the footer is still diaplyed even if it was set to off
> > if we use the extended mode for the query (-x flag)
> > 
> > Steps to Reproduce:
> > 1) create a table without any data
> > for example
> > create table text(i int);
> > 2) run
> > psql -U  -t --pset=footer=off 
> > 3) No output is generated
> > 4) run
> > psql -U  -t --pset=footer=off -x 
> > 5) Output generated : "(No Rows)"
> > 
> > Actual results:
> > psql does not honour the footer settings when output is defined to be in
> > Extended Mode
> > 
> > Expected results:
> > psql should not generate any output is query has no results and -t and 
> > --pset=footer=off were given
> 
> This has been fixed in PG 9.3 (released today) for the specific options
> you supplied:
> 
>   $ psql -t --pset=footer=off test
>   Default footer is off.
>   psql (9.4devel)
>   Type "help" for help.
>   
>   CREATE TABLE test (x INT);
>   CREATE TABLE
>   SELECT * FROM test;
>   
>   \x
>   Expanded display is on.
>   SELECT * FROM test;
>   
> Unfortunately, this did not fix the more simple case where
> --pset=footer=off is specified, but not -t:
> 
>   $ psql  --pset=footer=off test
>   Default footer is off.
>   psql (9.4devel)
>   Type "help" for help.
>   
>   CREATE TABLE test (x INT);
>   CREATE TABLE
>   SELECT * FROM test;
>x
>   ---
>   
>   \x
>   Expanded display is on.
>   SELECT * FROM test;
> -->   (No rows)
> 
> The attached patch fixes this, and makes it match the rest of the output
> formats, which do honor --pset=footer=off alone for footers.
> 
> -- 
>   Bruce Momjian  http://momjian.us
>   EnterpriseDB http://enterprisedb.com
> 
>   + It's impossible for everything to be true. +

> diff --git a/src/bin/psql/print.c b/src/bin/psql/print.c
> new file mode 100644
> index 5589cea..736225c
> *** a/src/bin/psql/print.c
> --- b/src/bin/psql/print.c
> *** print_aligned_vertical(const printTableC
> *** 1171,1177 
>   if (cont->cells[0] == NULL && cont->opt->start_table &&
>   cont->opt->stop_table)
>   {
> ! if (!opt_tuples_only)
>   fprintf(fout, _("(No rows)\n"));
>   return;
>   }
> --- 1171,1177 
>   if (cont->cells[0] == NULL && cont->opt->start_table &&
>   cont->opt->stop_table)
>   {
> ! if (!opt_tuples_only && cont->opt->default_footer)
>   fprintf(fout, _("(No rows)\n"));
>   return;
>   }

> 
> -- 
> Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-bugs


-- 
  Bruce Momjian  http://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +


-- 
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs


Re: [BUGS] BUG #7817: psql does not relate to footer settings in extended mode

2013-09-09 Thread Bruce Momjian
On Sun, Jan 20, 2013 at 10:33:37AM +, emes...@redhat.com wrote:
> The following bug has been logged on the website:
> 
> Bug reference:  7817
> Logged by:  Eli Mesika
> Email address:  emes...@redhat.com
> PostgreSQL version: 9.1.7
> Operating system:   Fedora 16
> Description:
> 
> psql does not relate to footer settings in extended mode
> Sometimes we need to run a sql command withot generating header and footer.
> This can be done using the -t flag and --pset=footer=off
> The problem is that the footer is still diaplyed even if it was set to off
> if we use the extended mode for the query (-x flag)
> 
> Steps to Reproduce:
> 1) create a table without any data
> for example
> create table text(i int);
> 2) run
> psql -U  -t --pset=footer=off 
> 3) No output is generated
> 4) run
> psql -U  -t --pset=footer=off -x 
> 5) Output generated : "(No Rows)"
> 
> Actual results:
> psql does not honour the footer settings when output is defined to be in
> Extended Mode
> 
> Expected results:
> psql should not generate any output is query has no results and -t and 
> --pset=footer=off were given

This has been fixed in PG 9.3 (released today) for the specific options
you supplied:

$ psql -t --pset=footer=off test
Default footer is off.
psql (9.4devel)
Type "help" for help.

CREATE TABLE test (x INT);
CREATE TABLE
SELECT * FROM test;

\x
Expanded display is on.
SELECT * FROM test;

Unfortunately, this did not fix the more simple case where
--pset=footer=off is specified, but not -t:

$ psql  --pset=footer=off test
Default footer is off.
psql (9.4devel)
Type "help" for help.

CREATE TABLE test (x INT);
CREATE TABLE
SELECT * FROM test;
 x
---

\x
Expanded display is on.
SELECT * FROM test;
--> (No rows)

The attached patch fixes this, and makes it match the rest of the output
formats, which do honor --pset=footer=off alone for footers.

-- 
  Bruce Momjian  http://momjian.us
  EnterpriseDB http://enterprisedb.com

  + It's impossible for everything to be true. +
diff --git a/src/bin/psql/print.c b/src/bin/psql/print.c
new file mode 100644
index 5589cea..736225c
*** a/src/bin/psql/print.c
--- b/src/bin/psql/print.c
*** print_aligned_vertical(const printTableC
*** 1171,1177 
  	if (cont->cells[0] == NULL && cont->opt->start_table &&
  		cont->opt->stop_table)
  	{
! 		if (!opt_tuples_only)
  			fprintf(fout, _("(No rows)\n"));
  		return;
  	}
--- 1171,1177 
  	if (cont->cells[0] == NULL && cont->opt->start_table &&
  		cont->opt->stop_table)
  	{
! 		if (!opt_tuples_only && cont->opt->default_footer)
  			fprintf(fout, _("(No rows)\n"));
  		return;
  	}

-- 
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs


[BUGS] BUG #7817: psql does not relate to footer settings in extended mode

2013-01-21 Thread emesika
The following bug has been logged on the website:

Bug reference:  7817
Logged by:  Eli Mesika
Email address:  emes...@redhat.com
PostgreSQL version: 9.1.7
Operating system:   Fedora 16
Description:

psql does not relate to footer settings in extended mode
Sometimes we need to run a sql command withot generating header and footer.
This can be done using the -t flag and --pset=footer=off
The problem is that the footer is still diaplyed even if it was set to off
if we use the extended mode for the query (-x flag)

Steps to Reproduce:
1) create a table without any data
for example
create table text(i int);
2) run
psql -U  -t --pset=footer=off 
3) No output is generated
4) run
psql -U  -t --pset=footer=off -x 
5) Output generated : "(No Rows)"

Actual results:
psql does not honour the footer settings when output is defined to be in
Extended Mode

Expected results:
psql should not generate any output is query has no results and -t and 
--pset=footer=off were given




-- 
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs