Re: [Discuss-gnuradio] OpenBTS compilation error

[Martin Dvh]

T. W. wrote:
> First I got the openBTS code from gnuradio site (svn co
> http://gnuradio.org/svn/openbts/trunk/ openbts).
> 
> Then I run the ./bootstrap and ./configure commands succesfully.
> 
> But the following errors occurred when I run make. ("xxxx" was used to
> mask out my username on the machine.)  Can anyone shed some light on
> this?
> 
> I was using a 32-bit Ubuntu 8.10 desktop virtual machine on a 64-bit
> Windows XP machine.
> 
> Thanks!
> 
> ----------------------------
> make[2]: Entering directory `/home/xxxx/openbts/openbts/Transceiver'
> if /bin/bash ../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H -I.
> -I. -I.
> .  -I../CommonLibs -I../Control -I../GSM -I../SIP -I../TRXManager
> -I/usr/local/i
> nclude   -g -O2 -MT radioInterface.lo -MD -MP -MF
> ".deps/radioInterface.Tpo" -c
> -o radioInterface.lo radioInterface.cpp; \
>   then mv -f ".deps/radioInterface.Tpo" ".deps/radioInterface.Plo"; else
> r
> m -f ".deps/radioInterface.Tpo"; exit 1; fi
> libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I. -I.. -I../CommonLibs
> -I../Control
>  -I../GSM -I../SIP -I../TRXManager -I/usr/local/include -g -O2 -MT
> radioInterfac
> e.lo -MD -MP -MF .deps/radioInterface.Tpo -c radioInterface.cpp  -fPIC
> -DPIC -o
> .libs/radioInterface.o
> In file included from USRPDevice.h:25,
>                  from radioInterface.h:25,
>                  from radioInterface.cpp:23:
> /usr/local/include/usrp_bytesex.h:37:2: warning: #warning Using
> non-portable cod
> e (likely wrong other than ILP32).

This is a known bug.
A temporary workaround can be done by using attached patch.

See description below.

Martin DvH wrote:
>  HAVE_BYTESWAP_H is not defined in
> Transceiver/USRPDevice.h
>
> This forces usrp_bytesex.h in using its own byteswap code which does NOT
> work on 64 bit systems.
>
> The disadvantage of putting #define HAVE_BYTESWAP_H in USRPDevice.h is
> that it should be checked first that you actually have byteswap.h.
> This should be done by the configure script and put into config.h.
> USRPDevice.h then should include config.h
>
> I haven't gotten to implementing the configure stuff yet.
>
>
> This patch should be changed to use configure stuff to check for byteswap.h.
> But for people wanting a quick-and-dirty hack it can be used.
>
>
> Greetings,
> Martin
>



> In file included from radioInterface.h:24,
>                  from radioInterface.cpp:23:
> sigProcLib.h:35: warning: ‘typedef’ was ignored in this declaration
> sigProcLib.h:45: warning: ‘typedef’ was ignored in this declaration
> In file included from radioInterface.h:25,
>                  from radioInterface.cpp:23:
> USRPDevice.h:42: error: ‘usrp_standard_rx_sptr’ does not name a type
> USRPDevice.h:43: error: ‘usrp_standard_tx_sptr’ does not name a type
> make[2]: *** [radioInterface.lo] Error 1
> make[2]: Leaving directory `/home/xxxx/openbts/openbts/Transceiver'
> ----------------------

Index: Transceiver/USRPDevice.h
===================================================================
--- Transceiver/USRPDevice.h    (revision 10235)
+++ Transceiver/USRPDevice.h    (working copy)
@@ -19,8 +19,8 @@
 
 */
 
+#define HAVE_BYTESWAP_H
 
-
 #include "usrp_standard.h"
 #include "usrp_bytesex.h"
 #include "usrp_prims.h"