Re: [Pan-devel] [SOLVED} pan 0.90 and compile errors on FC5 x86_64

[Steven Adeff]

On 4/3/06, Jeff Vian <address@hidden> wrote:
> On Mon, 2006-04-03 at 12:19 -0400, Steven Adeff wrote:
> > On 4/3/06, Jeff Vian <address@hidden> wrote:
> > > I just tried compiling after first installing a couple of devel packages
> > > I was missing.
> > >
> > > Now in the make stage I hit an error.
> > > --------------------------
> > > make[3]: Entering directory
> > > `/home/jeff/download/pan/pan-0.90/pan/usenet-utils'
> > > if g++ -DHAVE_CONFIG_H -I. -I. -I../..  -I../.. -I/usr/include/gmime-2.0
> > > -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include
> > > -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include     -g -O2 -MT
> > > utf8-utils.o -MD -MP -MF ".deps/utf8-utils.Tpo" -c -o utf8-utils.o
> > > utf8-utils.cc; \
> > > then mv -f ".deps/utf8-utils.Tpo" ".deps/utf8-utils.Po"; else rm -f
> > > ".deps/utf8-utils.Tpo"; exit 1; fi
> > > utf8-utils.cc: In function 'char* pan::mime_part_to_utf8(GMimePart*,
> > > const char*)':
> > > utf8-utils.cc:253: error: cannot convert 'guint*' to 'size_t*' for
> > > argument '2' to 'const char* g_mime_part_get_content(const GMimePart*,
> > > size_t*)'
> > > make[3]: *** [utf8-utils.o] Error 1
> > > make[3]: Leaving directory
> > > `/home/jeff/download/pan/pan-0.90/pan/usenet-utils'
> > > -----------------------------------
> > >
> > > Is this error related to the differences between i386 and x86_64?
> > > Or is it related to the change from gcc 4.0.2 to 4.1.0?
> > >
> > > On my FC4 i386 system pan 0.90 compiles and works well.
> > > On the FC5 x86_64 the above error is seen very quickly.
> >
> > answers to compiling issues for x64 are in here in this thread:
> > http://lists.gnu.org/archive/html/pan-devel/2006-04/msg00000.html
> >
> Thanks.  It seems I had missed seeing the fix in my perusal of earlier
> messages here.
>
> I made the change in utf8-utils and recompiled.
> It then failed with the same error in body-pane.cc
> Changed "guint len(0)" to "size_t len(0)" on line 506
>
> I had to change line 252 in utf8-utils.cc and line 506 in body-pane.cc.
>
> It then compiled and works well.
> Now for some dedicated testing. :-)
>
> Would it not be best for the code to check the architecture and use the
> appropriate type since the guint is not a valid type for use on x86-64
> architecture.

I'd assume, but remember this is beta1 after a HUGE rewrite, theres
bound to be things like this pop up. I believe theres also a patch
floating on the user list for fixing loading large groups. I'll assume
in a week or two we may see a second beta with the fixes for these
issues, some new features, etc.

--
Steve