[KimDaBa] MD5 sums completed. (Compile errors and what to do on kde/qt 3.1.1)

[Matt]

On Wed, 21 Jan 2004 10:51, Reimar Imhof wrote:
> > | > KimDaBa mailing list
> > | > KimDaBa at klaralvdalens-datakonsult.se
> > | > http://sulaco.hrhansen.dk/mailman/listinfo/kimdaba
> > |
> > | I get the following error when compiling this latest snapshot:
> > |
> > | make[3]: Entering directory
> > | `/home/matt/appdownloads/kimdaba-2004-01-19/kimdaba'
> > | if g++ -DHAVE_CONFIG_H -I. -I. -I.. -I/usr/lib/qt3//include
> > | -I/usr/X11R6/include  -I./compat  -DQT_THREAD_SUPPORT 
> > | -D_REENTRANT -Wnon-virtual-dtor -Wno-long-long -Wundef -Wall -W
> > | -Wpointer-arith -Wwrite-strings -ansi -D_XOPEN_SOURCE=500
> > | -D_BSD_SOURCE -Wcast-align -Wconversion -Wchar-subscripts -O2
> > | -Wformat-security
> > | -Wmissing-format-attribute -fno-exceptions -fno-check-new
> > | -fno-common -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL
> > | -DQT_NO_COMPAT -DQT_NO_TRANSLATION   -MT imagedb.o -MD -MP -MF
> > | ".deps/imagedb.Tpo" \ -c -o imagedb.o `test -f 'imagedb.cpp' ||
> > | echo './'`imagedb.cpp; \ then mv -f ".deps/imagedb.Tpo"
> > | ".deps/imagedb.Po"; \
> > | else rm -f ".deps/imagedb.Tpo"; exit 1; \
> > | fi
> > | imagedb.cpp: In member function `void ImageDB::loadExtraFiles()':
> > | imagedb.cpp:211: error: `wasCanceled' undeclared (first use this
> > | function)
> > | imagedb.cpp:211: error: (Each undeclared identifier is reported
> > | only once for each function it appears in.)
> > | imagedb.cpp: In member function `void
> > | ImageDB::calculateMissingMD5sums(QValueList<ImageInfo*>&)':
> > | imagedb.cpp:387: error: `wasCanceled' undeclared (first use this
> > | function)
> > | make[3]: *** [imagedb.o] Error 1
> > | make[3]: Leaving directory
> > | `/home/matt/appdownloads/kimdaba-2004-01-19/kimdaba'
> > | make[2]: *** [all-recursive] Error 1
> > | make[2]: Leaving directory
> > | `/home/matt/appdownloads/kimdaba-2004-01-19/kimdaba'
> > | make[1]: *** [all-recursive] Error 1
> > | make[1]: Leaving directory
> > | `/home/matt/appdownloads/kimdaba-2004-01-19' make: *** [all]
> > | Error 2
> > |
> > | Any suggestions?
> >
> > Strange indeed, I just tried installing the snapshot myself erhh
> > actually as my girlfriend (her account using a KDE 3.1.4), and I
> > had no problems.
> >
> > Does anyone else have this problem?
> > Has anyone else installed successfully?
> > Which version of Qt and KDE are you using?
> >
> > Cheers
> > Jesper.
> > _______________________________________________
> > KimDaBa mailing list
> > KimDaBa at klaralvdalens-datakonsult.se
> > http://sulaco.hrhansen.dk/mailman/listinfo/kimdaba
>
> Hello Jesper, hello people on the list,
>
> I'm running kde 3.1.1 and I think it's qt 3.1.1, at least
> qprogressdialog.h got an comment saying 3.1.1
>
> There are two points:
>
> When I compile the snapshot kimdaba-2004-01-19 I get the same error
> message like mentioned above.
> Here's what I've done to make it work:
> In my header qprogressdialog.h there's only a function called
> "wasCancelled()" but imagedb.cpp needs "wasCanceled()".
> So I changed the lines 211 and 387 to use "wasCancelled()".
>
> The second point is the problem in viewer.cpp, lines 191, 195, 199,
> 207, 211 and 215 like mentioned by Matt some days ago:
> Here it's a different implementation of a QAction constructor.
> I've done the following changes:
> For line 191 I changed
> "action = new QAction( QIconSet(), i18n("Centered"), 0,
> wallpaperPopup );" to
> "action = new QAction( i18n("Centered"), QIconSet(),
> i18n("Centered"), QKeySequence(), wallpaperPopup );"
> and so on for the following menu items.
>
> And now I got md5sums in my kindaba-database!
>
> Could you have a look at these changes and apply them to the next
> snapshot, so the next snapshots could be compiled on a kde/qt 3.1.1 -
> system?
>
> Thanks a lot
> Reimar
>
> PS.: I've just made a mistake: I showed my wife that your girlfriend
> is using kde. Well she is using... (don't want to talk about it!!)  
> ;-)
>
> _______________________________________________
> KimDaBa mailing list
> KimDaBa at klaralvdalens-datakonsult.se
> http://sulaco.hrhansen.dk/mailman/listinfo/kimdaba

Thanks for posting your fix.  I had already fixed the syntax problems in 
viewer.cpp, but Jesper mentioned that these would come out of the new 
releases, so I didn't bother posting back to the list.  Also, I've got 
no idea about c++, I was just looking at the differences between the 
lines in the function above, and the lines I was having trouble with, 
and matched them up to see if that works, so I wasn't sure whether what 
I was doing was a legitimate fix or not.

Like you, I hope that these get fixed in the next release, so that they 
compile without manual editing.  Not that that is a major.  It's worth 
it to be able to use this fantastic program.

-- 
Regards

Matt