Thread: Verbosity of genbki.pl
(This was discussed in the "way forward on bootstrap data" thread,
but we didn't do anything about it.)
Traditionally genbki.pl has printed "Writing foo" for every file
it writes out. As of 372728b0d that is a heck of a lot more files
than it once was, so you get this:
$ make -j8 -s
Writing fmgroids.h
Writing fmgrprotos.h
Writing fmgrtab.c
Writing pg_proc_d.h
Writing pg_type_d.h
Writing pg_attribute_d.h
Writing pg_class_d.h
Writing pg_attrdef_d.h
Writing pg_constraint_d.h
Writing pg_inherits_d.h
Writing pg_index_d.h
Writing pg_operator_d.h
Writing pg_opfamily_d.h
Writing pg_opclass_d.h
Writing pg_am_d.h
Writing pg_amop_d.h
Writing pg_amproc_d.h
Writing pg_language_d.h
Writing pg_largeobject_metadata_d.h
Writing pg_largeobject_d.h
Writing pg_aggregate_d.h
Writing pg_statistic_ext_d.h
Writing pg_statistic_d.h
Writing pg_rewrite_d.h
Writing pg_trigger_d.h
Writing pg_event_trigger_d.h
Writing pg_description_d.h
Writing pg_cast_d.h
Writing pg_enum_d.h
Writing pg_namespace_d.h
Writing pg_conversion_d.h
Writing pg_depend_d.h
Writing pg_database_d.h
Writing pg_db_role_setting_d.h
Writing pg_tablespace_d.h
Writing pg_pltemplate_d.h
Writing pg_authid_d.h
Writing pg_auth_members_d.h
Writing pg_shdepend_d.h
Writing pg_shdescription_d.h
Writing pg_ts_config_d.h
Writing pg_ts_config_map_d.h
Writing pg_ts_dict_d.h
Writing pg_ts_parser_d.h
Writing pg_ts_template_d.h
Writing pg_extension_d.h
Writing pg_foreign_data_wrapper_d.h
Writing pg_foreign_server_d.h
Writing pg_user_mapping_d.h
Writing pg_foreign_table_d.h
Writing pg_policy_d.h
Writing pg_replication_origin_d.h
Writing pg_default_acl_d.h
Writing pg_init_privs_d.h
Writing pg_seclabel_d.h
Writing pg_shseclabel_d.h
Writing pg_collation_d.h
Writing pg_partitioned_table_d.h
Writing pg_range_d.h
Writing pg_transform_d.h
Writing pg_sequence_d.h
Writing pg_publication_d.h
Writing pg_publication_rel_d.h
Writing pg_subscription_d.h
Writing pg_subscription_rel_d.h
Writing postgres.bki
Writing schemapg.h
Writing postgres.description
Writing postgres.shdescription
All of PostgreSQL successfully made. Ready to install.
which seems a tad excessive, especially in a -s run.
I think we should drop the per-file notices. But there are a couple
of alternatives as to what to do exactly:
1. Print nothing at all. That's more in keeping with our modern
build practices, but maybe it's too big a change?
2. Print just one message like "Generating postgres.bki and related
files", and I guess a second one for fmgroids.h and related files.
I don't have a strong preference. Opinions?
regards, tom lane
> 1. Print nothing at all. That's more in keeping with our modern
> build practices, but maybe it's too big a change?
>
> 2. Print just one message like "Generating postgres.bki and related
> files", and I guess a second one for fmgroids.h and related files.
>
> I don't have a strong preference. Opinions?
Second point, pls. I'd like to see some stage done
--
Teodor Sigaev E-mail: teodor@sigaev.ru
WWW: http://www.sigaev.ru/
On 4/9/18, Teodor Sigaev <teodor@sigaev.ru> wrote: >> 1. Print nothing at all. That's more in keeping with our modern >> build practices, but maybe it's too big a change? >> >> 2. Print just one message like "Generating postgres.bki and related >> files", and I guess a second one for fmgroids.h and related files. >> >> I don't have a strong preference. Opinions? > > Second point, pls. I'd like to see some stage done The attached patch does #2. -John Naylor
Attachment
Hi, On 2018-04-08 13:33:42 -0400, Tom Lane wrote: > Traditionally genbki.pl has printed "Writing foo" for every file > it writes out. > 2. Print just one message like "Generating postgres.bki and related > files", and I guess a second one for fmgroids.h and related files. +0.5. - Andres
Andres Freund <andres@anarazel.de> writes:
> On 2018-04-08 13:33:42 -0400, Tom Lane wrote:
>> Traditionally genbki.pl has printed "Writing foo" for every file
>> it writes out.
>> 2. Print just one message like "Generating postgres.bki and related
>> files", and I guess a second one for fmgroids.h and related files.
> +0.5.
Hearing no votes against, done that way.
regards, tom lane
On 4/9/18 15:02, Tom Lane wrote: > Andres Freund <andres@anarazel.de> writes: >> On 2018-04-08 13:33:42 -0400, Tom Lane wrote: >>> Traditionally genbki.pl has printed "Writing foo" for every file >>> it writes out. > >>> 2. Print just one message like "Generating postgres.bki and related >>> files", and I guess a second one for fmgroids.h and related files. > >> +0.5. > > Hearing no votes against, done that way. Maybe we can take another look at this. Right now, make -s world produces *only* the following output: $ make -s world Generating BKI files and symbol definition headers... Generating fmgrtab.c, fmgroids.h, and fmgrprotos.h... PostgreSQL, contrib, and documentation successfully made. Ready to install. That seems kind of inconsistent and silly. I suggest we remove the output from these scripts. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> Maybe we can take another look at this. Right now, make -s world
> produces *only* the following output:
> $ make -s world
> Generating BKI files and symbol definition headers...
> Generating fmgrtab.c, fmgroids.h, and fmgrprotos.h...
> PostgreSQL, contrib, and documentation successfully made. Ready to install.
> That seems kind of inconsistent and silly. I suggest we remove the
> output from these scripts.
FWIW, that'd be fine with me. Teodor spoke against removing that output
entirely, but perhaps he's the only one who thinks that way. For myself,
I've gotten mighty used to running "make -s" and expecting no output
if all is well. If I wanted to see what was getting rebuilt, I wouldn't
use -s.
regards, tom lane
Greetings, * Tom Lane (tgl@sss.pgh.pa.us) wrote: > Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes: > > Maybe we can take another look at this. Right now, make -s world > > produces *only* the following output: > > > $ make -s world > > Generating BKI files and symbol definition headers... > > Generating fmgrtab.c, fmgroids.h, and fmgrprotos.h... > > PostgreSQL, contrib, and documentation successfully made. Ready to install. > > > That seems kind of inconsistent and silly. I suggest we remove the > > output from these scripts. > > FWIW, that'd be fine with me. Teodor spoke against removing that output > entirely, but perhaps he's the only one who thinks that way. For myself, > I've gotten mighty used to running "make -s" and expecting no output > if all is well. If I wanted to see what was getting rebuilt, I wouldn't > use -s. +1 for making them not output anything if all is well. Thanks! Stephen
Attachment
On Sat, Apr 28, 2018 at 01:02:10PM -0400, Stephen Frost wrote: > +1 for making them not output anything if all is well. +1. -- Michael
Attachment
On 4/29/18 06:50, Michael Paquier wrote: > On Sat, Apr 28, 2018 at 01:02:10PM -0400, Stephen Frost wrote: >> +1 for making them not output anything if all is well. > > +1. done -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services