fsphil / hacktv

Analogue TV transmitter for the HackRF
GNU General Public License v3.0
669 stars 80 forks source link

fails to build against soapysdr 0.8 #81

Closed alexmyczko closed 2 years ago

alexmyczko commented 2 years ago

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=993698

test from a master checkout of today:

In file included from soapysdr.c:20:
/usr/include/SoapySDR/Device.h:309:19: note: expected ‘const size_t *’ {aka ‘const long unsigned int *’} but argument is of type ‘char *’
  309 |     const size_t *channels,
      |     ~~~~~~~~~~~~~~^~~~~~~~
soapysdr.c:135:77: warning: passing argument 5 of ‘SoapySDRDevice_setupStream’ makes integer from pointer without a cast [-Wint-conversion]
  135 |  if(SoapySDRDevice_setupStream(rf->d, &rf->s, SOAPY_SDR_TX, SOAPY_SDR_CS16, NULL, 0, NULL) != 0)
      |                                                                             ^~~~
      |                                                                             |
      |                                                                             void *
In file included from soapysdr.c:20:
/usr/include/SoapySDR/Device.h:310:18: note: expected ‘size_t’ {aka ‘const long unsigned int’} but argument is of type ‘void *’
  310 |     const size_t numChans,
      |     ~~~~~~~~~~~~~^~~~~~~~
soapysdr.c:135:5: error: too many arguments to function ‘SoapySDRDevice_setupStream’
  135 |  if(SoapySDRDevice_setupStream(rf->d, &rf->s, SOAPY_SDR_TX, SOAPY_SDR_CS16, NULL, 0, NULL) != 0)
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from soapysdr.c:20:
/usr/include/SoapySDR/Device.h:306:31: note: declared here
  306 | SOAPY_SDR_API SoapySDRStream *SoapySDRDevice_setupStream(SoapySDRDevice *device,
      |                               ^~~~~~~~~~~~~~~~~~~~~~~~~~
make[2]: *** [Makefile:32: soapysdr.o] Error 1
make[2]: *** Waiting for unfinished jobs....
fsphil commented 2 years ago

I've believe this is fixed now, however I'm not able to properly test at the moment.

alexmyczko commented 2 years ago

Many thanks, I confirm it's fine now.