[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#424844: 64bit building of ircservices



On Thu, 2007-11-08 at 09:15 -0800, Dan Sully wrote:
> I ran into this issue the other day. The simple fix is to add:
> 
> #include <inttypes.h>
> 
> at the top of config.h (which is generated from ./configure)
> 
> What is the historical reason for not accepting that?

Andrew Church's response to my patch was:

"Thanks for the note, but I'm going to reject most of this as
unnecessary; Services doesn't use the int*_t types for historical
reasons (see section 11-1 of the technical manual)"

Here is the relevant section:

"Services defines its own types for integers of specific sizes: int8,
uint32, and the like. The standard header file stdint.h declares similar
types, using names like int8_t and uint32_t; however, these are not used
because this header file did not exist (or was not standardized, at
least) during the early stages of Services' development, when the need
for sized integer types arose. Changing the code to use the standard
types would remove the need to define those types in the configure
script."

http://www.ircservices.za.net/docs/tech/11.html#s1

This suggests that the maintainer is leaving this change for future
work, but might possibly agree to change all occurrences of those types
throughout the entire project. In practise, it's a very large change to
be making this close to a 5.1 release, and the maintainer has said that
IRC Services is close to "end of life".

I suggest you ask the ircservices coding mailing list about 64-bit
support. I get the impression that the maintainer prefers to keep hold
of the copyright of the project, so doesn't accept large patches.

-- 
Tim Retout <tim@retout.co.uk>

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: