Bug 51190 - Solaris fails to compile xjdxgen.c
Summary: Solaris fails to compile xjdxgen.c
Status: RESOLVED FIXED
Alias: None
Product: kiten
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Solaris
: NOR normal
Target Milestone: ---
Assignee: Jason Katz-Brown
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-11-25 20:15 UTC by David F. Newman
Modified: 2005-01-28 22:14 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David F. Newman 2002-11-25 20:15:16 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc 2.95.3 
OS:          Solaris

A change commited with version 1.8 of xjdxgen.c causes the build of this file to fail on Solaris.

Here was the change:
-uint32_t dbyte;
-uint32_t  *jindex;
-uint32_t indptr,llone;
+u_int32_t dbyte;
+u_int32_t  *jindex;
+u_int32_t indptr,llone;

Solaris doesn't seem to have u_int32_t, however, it does have uint32_t and when the change is reverted Solaris can compile this file.  

The comment on the commit isn't informative enough to know why the change was made so I can only assume there is a platform out there that requires it.  So the solution would be something along the lines of configure testing for the existance of u_int32_t and uint32_t and setting HAVE_U_INT32_T and HAVE_UINT32_T.
Comment 1 Anne-Marie Mahfouf 2005-01-28 22:14:48 UTC
Tested by Mats Röjestål who successfully compiled the kdeedu module from cvs HEAD on Solaris 10_x86.
I changed "#include <sys/types>"  to "#include <inttypes.h>" which is a more
portable and conforming to standard.
With thanks to Mats.