Re: [Cvs-dev] Re: [Cvs-test-results] Build CVS (TRUNK) failed.

[Jim Hyslop]

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

Mark D. Baushke wrote:
> Jim Hyslop <address@hidden> writes:
>>It's a start :=)
> 
> 
> Good. Perhaps if you commit what works, we can see what else still
> fails?

I'm just cleaning up a bit, to make sure the checkin doesn't include any
other work-in-progress.

> 
>>I've been working on this, and once those have been resolved, there's
>>still a problem where client.c tries to call strtoumax(). See the thread
>>in bug-cvs: http://lists.gnu.org/archive/html/bug-cvs/2006-06/msg00026.html
> 
> 
> I guess we need to know if there is a uintmax_t type and/or a
> strtoimax() function in your environment or not before anyone else can
> help.

There's an __int64 type, and (as far as I've found) three conversion
functions:

char * _i64toa(__int64, char *, int);
char * _ui64toa(unsigned __int64, char *, int);
__int64 _atoi64(const char *);

Now that I've modified the appropriate HAVE_DECL_ definitions in
config.h.in.in, everything compiles and links if I change
lib/strtoimax.c line 71 from:

  verify (sizeof (INT) == sizeof (long int));

to

  verify (sizeof (INT) >= sizeof (long int));


> 
> 
>>This also requires modifying windows-NT/config.h to define
>>HAVE_DECL_STRTOLL, and I'm having a difficult time trying to understand
>>the interactions between ./configure.in, ./config.h,
>>windows-NT/config.h.in.in, etc.
> 
> 
> Ahhh... maybe I can help.

Thanks for the explanation (and thanks, Derek, for adding that
explanation to mkconfig.pl). That's what I thought, but each time I ran
mkconfig.pl the config.h file it generated had some really weird
results. Basically, it was putting in "#undef " statements for some
values that should have been defined, such as CLIENT_SUPPORT.

I finally tracked it down to this line in mkconfig.pl:

        if (/^#\s*define\s*(\w+)(\s+(.+))?$/)


If I understand the line correctly, then the space and any characters
following 'define' should be optional. My Cygwin version of perl
(5.8.7), however, doesn't match the line unless it has at least a space
after the word 'define'.

I'll commit the changes to get it building correctly for now, and we can
discuss whether the change to lib/strtoimax.c is appropriate, or if
there's a better way to handle it.

- --
Jim Hyslop
Dreampossible: Better software. Simply.     http://www.dreampossible.ca
                 Consulting * Mentoring * Training in
    C/C++ * OOD * SW Development & Practices * Version Management
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (MingW32)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFEjMnfLdDyDwyJw+MRAj0pAKCaVhVCMO4icyxgND0v/36MHJWIJgCeLxZo
hBIiDJdwHgGg4siDsvgVP0E=
=Ysd+
-----END PGP SIGNATURE-----