Debian Build issues with twinkle

[Michel de Boer]

Hi Mark,

Thanks for your feed back.

1. Are you sure that $QTDIR is set correctly? As far as I know qmake
   is always in $QTDIR/bin.

   Anyway I add some additional checking in configure to see if qmake
   is in $PATH

2. I will update the spec.in file in the next release

3. I will fix the 64bit troubles. It seems I missed the invalid cast
   in events.cpp
   And I will completely change the stunRand() function in stun.cxx
   The platform dependencies give me headaches. If I knew this mess
   was inside the stun code I wouldn't have taken this open source
   stun project into my code :-)

4. It's a long while ago I include the gsm codec. I don't think I
   changed the code itself. I only remember I had to work on the
   makefiles as it did not build initially. At the time I used it
   libgsm was not a library that was commonly available at systems
   and as it was small I decided to link it statically into the
   code.

Regards,
Michel


Mark Purcell wrote:

>Michel,
>
>Had a few issues with the Debian builds.
>
>1. Your configure{.in} presumes that qmake is available under 
>$(QTDIR)/bin/qmake, which isn't true for Debian.
>
>--- configure.in.orig   2005-10-29 15:15:22.000000000 +0100
>+++ configure.in        2005-10-29 15:04:32.000000000 +0100
>@@ -92,7 +92,7 @@
>        echo "LIBS += -lasound" >> $QT_INCL_PRO
>        ])
>
>-$QTDIR/bin/qmake -o src/gui/Makefile src/gui/twinkle.pro
>+/usr/bin/qmake -o src/gui/Makefile src/gui/twinkle.pro
> echo "distdir:" >> src/gui/Makefile
>
> # Strip the -O2 flag from CXXFLAGS for building the SIP parser.
>
>Perhaps some sort of test for qmake would be better, rather than hardcoding as 
>I have done above.
>
>2. I noticed for 3.2 that the build-depends have grown from what is detailed 
>in the documentation.  For the Debian package I am using:
>
>Build-Depends: debhelper (>= 4.0.0), autotools-dev, libcommoncpp2-dev 
>(>=1.3.10-4), libccrtp-dev (>=1.3.4), help2man, libqt-mt-dev (>= 3:3.3.4-4), 
>qt3-apps-dev, automake1.9, libxml2-dev, cdbs, kdelibs-bin, kdelibs4-dev, 
>libasound2-dev, libxml2-dev
>
>In particular I had to add the kde, asound and xml2 depends.
>
>3. As you might be aware we currently build Debian packages across 11 
>architectures so it is very good for checking conformant code.
>
>The Buildd logs for all debian builds for twinkle are available here:
>
>http://buildd.debian.org/build.php?pkg=twinkle
>
>Of note, now that powerpc and others are building, is that ia64, sparc and 
>alpha are still failing.
>
>ia64/ alpha:
>  
>
>>if g++ -DHAVE_CONFIG_H -I. -I. -I. -I /usr/include 
>>    
>>
>-DDATADIR=\"/usr/share/twinkle\" -DQT_THREAD_SUPPORT   -g -Wall -O2 
>-I/usr/include/cc++2 -D_GNU_SOURCE -MT events.o -MD -MP -MF 
>".deps/events.Tpo" -c -o events.o events.cpp; \
>  
>
>>then mv -f ".deps/events.Tpo" ".deps/events.Po"; else rm -f 
>>    
>>
>".deps/events.Tpo"; exit 1; fi
>  
>
>>events.cpp: In destructor 't_event_queue::~t_event_queue()':
>>events.cpp:328: error: cast from 't_event*' to 'int' loses precision
>>make[4]: *** [events.o] Error 1
>>    
>>
>
>sparc:
>  
>
>>if g++ -DHAVE_CONFIG_H -I. -I. -I../../src -I ../../src -I /usr/include  
>>    
>>
>-DQT_THREAD_SUPPORT   -g -Wall -O2 -I/usr/include/cc++2 -D_GNU_SOURCE -MT 
>stun.o -MD -MP -MF ".deps/stun.Tpo" -c -o stun.o stun.cxx; \
>  
>
>>then mv -f ".deps/stun.Tpo" ".deps/stun.Po"; else rm -f ".deps/stun.Tpo"; 
>>    
>>
>exit 1; fi
>  
>
>>stun.cxx: In function 'int stunRand()':
>>stun.cxx:680: error: 'gethrtime' was not declared in this scope
>>    
>>
>
>4. I'm also looking in the longer term to be able to use system supplied 
>libgsm, rather than those included and statically linked within twinkle.  
>Have you modified the gsmlib or can system defaults be used safely?
>
>Mark
>
>PS. Some other Debian twinkle links you might be interested in are:
>
>http://packages.debian.org/twinkle 
>http://packages.qa.debian.org/twinkle
>http://bugs.debian.org/twinkle
>
>
>  
>

-- 
Michel de Boer
www.twinklephone.com