Re: printf failure on HP/UX 10.20 with 6.9.92

[Peter Fales]

On Thu, Feb 07, 2008 at 02:11:24AM +0100, Bruno Haible wrote:
> I committed this. It should fix it. Please test it, since I don't have access
> to a HP-UX 10 machine. To test it, create a test directory through the command

Thanks for the mail.  The request to test it may have been directed
at me, but I'm not quite sure how to do that.   The patches in the mail
won't apply to the latest coreutils package, and there doesn't appear
to be a gnulib package.

For the record, here's the temporary patch I used.   It seems simpler, and
I haven't found any problems yet.  Though this change is specifically 
for the buggy HP-HUX 10.20, it has (AFAIK) no impact on  the behavior
of any other platforms:

diff -ur coreutils-6.10/lib/vasnprintf.c coreutils-6.10.new/lib/vasnprintf.c
--- coreutils-6.10/lib/vasnprintf.c     2007-11-26 15:00:20.000000000 -0600
+++ coreutils-6.10.new/lib/vasnprintf.c 2008-02-04 16:46:47.000000000 -0600
@@ -1606,6 +1606,7 @@
                                followed by a positive field width."  */
                            flags |= FLAG_LEFT;
                            width = (unsigned int) (-arg);
+                           a.arg[dp->width_arg_index].a.a_int = width; 
                          }
                        else
                          width = arg;
@@ -2085,6 +2086,7 @@
                                followed by a positive field width."  */
                            flags |= FLAG_LEFT;
                            width = (unsigned int) (-arg);
+                           a.arg[dp->width_arg_index].a.a_int = width; 
                          }
                        else
                          width = arg;
@@ -2590,6 +2592,7 @@
                                followed by a positive field width."  */
                            flags |= FLAG_LEFT;
                            width = (unsigned int) (-arg);
+                           a.arg[dp->width_arg_index].a.a_int = width; 
                          }
                        else
                          width = arg;
@@ -3613,6 +3616,7 @@
                                followed by a positive field width."  */
                            flags |= FLAG_LEFT;
                            width = (unsigned int) (-arg);
+                           a.arg[dp->width_arg_index].a.a_int = width; 
                          }
                        else
                          width = arg;


-- 
Peter Fales
Alcatel-Lucent
Member of Technical Staff
2000 Lucent Lane
Room: 1C-436
Naperville, IL 60566-7033
Email: address@hidden
Phone: 630 979 8031