1.12 h5repack should only print if verbose (#2274)

* h5repack should only print if verbose

Also chage dump test to use long option

* Add note

* Update yaml workflows

2 files changed, 37 insertions, 27 deletions

diff --git a/tools/src/h5repack/h5repack_copy.c b/tools/src/h5repack/h5repack_copy.c
index d8a9e15..f3ab22e 100644
--- a/tools/src/h5repack/h5repack_copy.c
+++ b/tools/src/h5repack/h5repack_copy.c
@@ -666,15 +666,17 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti
      *-------------------------------------------------------------------------
      */
 
-    if (options->verbose == 2) {
-        HDprintf("-----------------------------------------------------------------\n");
-        HDprintf(" Type     Filter (Compression)        Timing read/write    Name\n");
-        HDprintf("-----------------------------------------------------------------\n");
-    }
-    else {
-        HDprintf("-----------------------------------------\n");
-        HDprintf(" Type     Filter (Compression)     Name\n");
-        HDprintf("-----------------------------------------\n");
+    if (options->verbose > 0) {
+        if (options->verbose == 2) {
+            HDprintf("-----------------------------------------------------------------\n");
+            HDprintf(" Type     Filter (Compression)        Timing read/write    Name\n");
+            HDprintf("-----------------------------------------------------------------\n");
+        }
+        else {
+            HDprintf("-----------------------------------------\n");
+            HDprintf(" Type     Filter (Compression)     Name\n");
+            HDprintf("-----------------------------------------\n");
+        }
     }
 
     if (travt->objs) {
@@ -692,10 +694,12 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti
                      *-------------------------------------------------------------------------
                      */
                 case H5TRAV_TYPE_GROUP:
-                    if (options->verbose == 2)
-                        HDprintf(FORMAT_OBJ_NOTIME, "group", travt->objs[i].name);
-                    else
-                        HDprintf(FORMAT_OBJ, "group", travt->objs[i].name);
+                    if (options->verbose > 0) {
+                        if (options->verbose == 2)
+                            HDprintf(FORMAT_OBJ_NOTIME, "group", travt->objs[i].name);
+                        else
+                            HDprintf(FORMAT_OBJ, "group", travt->objs[i].name);
+                    }
 
                     /* open input group */
                     if ((grp_in = H5Gopen2(fidin, travt->objs[i].name, H5P_DEFAULT)) < 0)
@@ -1199,7 +1203,7 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti
                                 if (options->verbose > 0) {
                                     double ratio = 0;
 
-                                    /* only print the compression ration if there was a filter request */
+                                    /* only print the compression ratio if there was a filter request */
                                     if (apply_s && apply_f && req_filter) {
                                         /* get the storage size of the output dataset */
                                         dsize_out = H5Dget_storage_size(dset_out);
@@ -1305,10 +1309,12 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti
                         if (H5Dclose(dset_out) < 0)
                             H5TOOLS_GOTO_ERROR((-1), "H5Dclose failed");
 
-                        if (options->verbose == 2)
-                            HDprintf(FORMAT_OBJ_TIME, "dset", 0.0, write_time, travt->objs[i].name);
-                        else
-                            HDprintf(FORMAT_OBJ, "dset", travt->objs[i].name);
+                        if (options->verbose > 0) {
+                            if (options->verbose == 2)
+                                HDprintf(FORMAT_OBJ_TIME, "dset", 0.0, write_time, travt->objs[i].name);
+                            else
+                                HDprintf(FORMAT_OBJ, "dset", travt->objs[i].name);
+                        }
 
                     } /* end whether we have request for filter/chunking */
 
@@ -1320,10 +1326,12 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti
                  *-------------------------------------------------------------------------
                  */
                 case H5TRAV_TYPE_NAMED_DATATYPE:
-                    if (options->verbose == 2)
-                        HDprintf(FORMAT_OBJ_NOTIME, "type", travt->objs[i].name);
-                    else
-                        HDprintf(FORMAT_OBJ, "type", travt->objs[i].name);
+                    if (options->verbose > 0) {
+                        if (options->verbose == 2)
+                            HDprintf(FORMAT_OBJ_NOTIME, "type", travt->objs[i].name);
+                        else
+                            HDprintf(FORMAT_OBJ, "type", travt->objs[i].name);
+                    }
 
                     if ((type_in = H5Topen2(fidin, travt->objs[i].name, H5P_DEFAULT)) < 0)
                         H5TOOLS_GOTO_ERROR((-1), "H5Topen2 failed");
@@ -1362,10 +1370,12 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti
                  */
                 case H5TRAV_TYPE_LINK:
                 case H5TRAV_TYPE_UDLINK:
-                    if (options->verbose == 2)
-                        HDprintf(FORMAT_OBJ_NOTIME, "link", travt->objs[i].name);
-                    else
-                        HDprintf(FORMAT_OBJ, "link", travt->objs[i].name);
+                    if (options->verbose > 0) {
+                        if (options->verbose == 2)
+                            HDprintf(FORMAT_OBJ_NOTIME, "link", travt->objs[i].name);
+                        else
+                            HDprintf(FORMAT_OBJ, "link", travt->objs[i].name);
+                    }
 
                     /* Check -X option. */
                     if (options->merge) {
diff --git a/tools/test/h5dump/CMakeTests.cmake b/tools/test/h5dump/CMakeTests.cmake
index 933aba9..2505e84 100644
--- a/tools/test/h5dump/CMakeTests.cmake
+++ b/tools/test/h5dump/CMakeTests.cmake
@@ -1159,7 +1159,7 @@
   ADD_H5ERR_MASK_TEST (torderlinks2 0 "unable to open external file, external link file name = 'fname'" --enable-error-stack --sort_by=name --sort_order=descending tfcontents1.h5)
 
   # tests for floating point user defined printf format
-  ADD_H5_TEST (tfpformat 0 --enable-error-stack -m %.7f tfpformat.h5)
+  ADD_H5_TEST (tfpformat 0 --enable-error-stack --format=%.7f tfpformat.h5)
 
   # tests for traversal of external links
   ADD_H5ERR_MASK_TEST (textlinksrc 0 "Too many soft links in path" --enable-error-stack textlinksrc.h5)