[Bug-wget] Wget 1.12 v. VMS

[Steven M. Schweda]

>    Today's mostly-VMS-related whining complaints, suggestions, and
> questions, based on the official 1.12 source kit:
> [...]

   One more thing:

   7a.  Lacking Unix-like builders, I stuck some place-holders for
"compilation_string" and "link_string" into "src/vms.c":

IT $ wgx --version
GNU Wget 1.12 built on VMS IA64 V8.3-1H1.

+digest +ipv6 -nls +ntlm +opie +md5/builtin +https -gnutls -openssl
-iri

Wgetrc:
Compile: (Info not available.)
Link: (Info not available.)
[...]

   It might make more sense to make the "Compile:" and "Link:" lines
conditional in some way, and thus omit these lines when they have
nothing useful to offer.  For example:

ALP $ gdiff -u main.c;-1 main.c
--- main.c;-1   2009-09-23 09:30:58 -0500
+++ main.c      2009-09-24 16:21:05 -0500
@@ -826,13 +826,15 @@
                         MAX_CHARS_PER_LINE);
 #endif /* def ENABLE_NLS */

-  format_and_print_line (compile_title,
-                        compilation_string,
-                        MAX_CHARS_PER_LINE);
-
-  format_and_print_line (link_title,
-                        link_string,
-                        MAX_CHARS_PER_LINE);
+  if (compilation_string != NULL)
+    format_and_print_line (compile_title,
+                          compilation_string,
+                          MAX_CHARS_PER_LINE);
+
+  if (link_string != NULL)
+    format_and_print_line (link_title,
+                          link_string,
+                          MAX_CHARS_PER_LINE);

   printf ("\n");
   /* TRANSLATORS: When available, an actual copyright character


ALP $ gdiff -u vms.c;-1 vms.c  !!! You don't have this one yet, but ...
--- vms.c;-1    2009-09-23 07:37:17 -0500
+++ vms.c       2009-09-24 16:21:33 -0500
@@ -1236,8 +1236,8 @@
  * "version.c" information.
  */

-const char *compilation_string = "(Info not available.)";
-const char *link_string = "(Info not available.)";
+const char *compilation_string = NULL;
+const char *link_string = NULL;
 const char *version_string = VERSION;


/*--------------------------------------------------------------------*/



   Typical results:

IT $ wgxl --version
GNU Wget 1.12 built on VMS IA64 V8.3-1H1.

+digest +ipv6 -nls +ntlm +opie +md5/builtin +https -gnutls -openssl
-iri

Wgetrc:

[...]

ALP $ wgxl --version
GNU Wget 1.12 built on VMS Alpha V7.3-2.

+digest +ipv6 -nls +ntlm +opie +md5/builtin +https -gnutls +openssl
-iri

Wgetrc:
    SYS$LOGIN:.wgetrc (user)

[...]


   And, speaking of build-related info, would it be useful on other
system types to include an indicator of large-file support somewhere in
this stuff?  (Or has large-file support become so common that no one
else worries about it now?)  A "sizeof off_t" test would be good enough
for me.

------------------------------------------------------------------------

   Steven M. Schweda               address@hidden
   382 South Warwick Street        (+1) 651-699-9818
   Saint Paul  MN  55105-2547