Certainly not a typo in binutils-2.21.51/bfd/configure.com for VMS/Alpha targets !

[Philippe Vouters <philippe dot vouters at laposte dot net>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dear binutils maintainers,

Surely an outdated information in binutils-2.21.51/bfd/configure.com
for VMS/Alpha targets !

$ set def PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51]
$ @setup all
Configuring BFD for alpha target
Generate `bfdver.h' from 'version.h' and `configure.in'.
Generate `targmatch.h'
Generate `config.h'
Copy sysdep.h
Generate build.com
Generate libiberty build.com
Generate opcodes/build.com
Configuring binutils for alpha target
Created `config.h'
Generate binutils build.com
Generate targ-cpu.[ch]
Generate targ-env.h
Generate obj-format.[ch]
Generate atof-targ.c
Generate gas/config.h
Generate gas/build.com
Compiling cpu-alpha.c
Compiling vms.c
%CC-F-OPENIN, error opening
PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms.c
; as input
- -RMS-E-FNF, file not found
$ dire PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms*

Directory PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]

vms-alpha.c;1       vms-lib.c;1         vms-misc.c;1       
vms.h;1           

Total of 4 files.

Also lacking vms-hdr.c, vms-gsd.c, vms-tir.c source files !

vms-alpha.c lacking :
#ifdef __DECC
#include <inttypes.h>
#endif
This is necessary to correctly define intptr_t used in the code.

In the very same source code:
Compiling vms-alpha.c

  if (PRIV (recrd.rec_size) <= 0)
......^
%CC-I-QUESTCOMPARE, In this statement, the unsigned expression
"(((struct vms_pr
ivate_data_struct ...)(abfd)->tdata.any)->recrd.rec_size)" is being
compared with a relational operator to a constant whose value is not
greater than zero.  This might not be
what you intended.
at line number 806 in file
PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms-alpha.c;2

          if (section->reloc_count <= 0)
..............^
%CC-I-QUESTCOMPARE, In this statement, the unsigned expression
"section->reloc_count" is being compared with a relational operator to
a constant whose value is not greater than zero.  This might not be
what you intended.
at line number 3757 in file
PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms-alpha.c;2

Proposed correction:
if ((int)PRIV (recrd.rec_size) <= 0) at line 806
if ((int)section->reloc_count <= 0) at line 3757

Compiling targets.c

        &DEFAULT_VECTOR,
.........^
%CC-E-UNDECLARED, In the initializer for bfd_default_vector[0],
"bfd_alpha_vec"
is not declared.
at line number 1316 in file
PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]targe
ts.c;1

The above error is caused by my previous changes to bfd/configure.com.

Proposed changes to bfd/configure.com in the Alpha section:
$ DEFS="""HAVE_vms_alpha_vec"","+-
    """DEFAULT_VECTOR=vms_alpha_vec"","+-
    """SELECT_VECS=&vms_alpha_vec"","+-
    """SELECT_ARCHITECTURES=&bfd_alpha_arch"""
$ FILES="cpu-alpha,vms-alpha,vms-lib,vms-misc,"

With all these Alpha/VMS specific changes after my changes for IA64/VMS, a
                    $ sho sys/noproc
                    OpenVMS V8.3  on node LION  30-MAR-2011
08:00:00.34  Uptime  86 01:30:18
                    $ sho cpu

                    System: LION, AlphaServer ES40

                    CPU ownership sets:
                   Active               0-3
                   Configure            0-3

                    CPU state sets:
                   Potential            0-3
                   Autostart            0-3
                   Powered Down         None
                   Not Present          None
                   Hard Excluded        None
                   Failover             None
                    $ show default
                      PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51]
                    $ @setup all
cleanly builds everything with absolutely no
informational/warning/error message.

Quick check:

$ nm :== $PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.binutils]nm.EXE
$ pipe nm sys$share:DECC$SHR.EXE | search sys$pipe: gmtime
0023e650 A DECC$GMTIME
0023e640 A DECC$GMTIME_R
00260550 A DECC$__UTCTZ_GMTIME
002604b8 A DECC$__UTCTZ_GMTIME_R
0023a410 A DECC$__UTC_GMTIME
0023a378 A DECC$__UTC_GMTIME_R
0023a410 A decc$__utc_gmtime
0023a378 A decc$__utc_gmtime_r
00260550 A decc$__utctz_gmtime
002604b8 A decc$__utctz_gmtime_r
0023e650 A decc$gmtime
0023e640 A decc$gmtime_r

Yours truly,
Philippe

- -- 
Philippe Vouters (Fontainebleau/France)
URL: http://vouters.dyndns.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAk2TH00ACgkQsUYBDUbOpdpqiQCfbRnEuJLZzr8v4pS+jjokNumY
yJYAnjfm8Ir7zBoYs0VGjMpNk7+Ab3iU
=9yyA
-----END PGP SIGNATURE-----