dbdexter-dev / sdrpp_radiosonde

Radiosonde decoder plugin for SDR++
MIT License
80 stars 12 forks source link

compilation error; linux #22

Open gariac opened 1 year ago

gariac commented 1 year ago

Opensuse 15.4 Linux 5.14.21-150400.24.63-default #1 SMP PREEMPT_DYNAMIC Tue May 2 15:49:04 UTC 2023 (fd0cc4f) x86_64 x86_64 x86_64 GNU/Linux

`[ 89%] Building CXX object decoder_modules/sdrpp_radiosonde/CMakeFiles/radiosonde_decoder.dir/src/main.cpp.o /usr/local/src/SDRPlusPlus/decoder_modules/sdrpp_radiosonde/src/main.cpp: In constructor ‘RadiosondeDecoderModule::RadiosondeDecoderModule(std::string)’: /usr/local/src/SDRPlusPlus/decoder_modules/sdrpp_radiosonde/src/main.cpp:57:46: error: no matching function for call to ‘dsp::demod::FM::init(dsp::stream&, float&, float, bool)’ 57 | fmDemod.init(vfo->output, bw, bw/2.0f, false); | ^ In file included from /usr/local/src/SDRPlusPlus/decoder_modules/sdrpp_radiosonde/src/main.hpp:6, from /usr/local/src/SDRPlusPlus/decoder_modules/sdrpp_radiosonde/src/main.cpp:9: /usr/local/src/SDRPlusPlus/core/src/dsp/demod/fm.h:25:14: note: candidate: ‘void dsp::demod::FM::init(dsp::stream, double, double, bool, bool) [with T = float]’ 25 | void init(dsp::stream* in, double samplerate, double bandwidth, bool lowPass, bool highPass) { | ^~~~ /usr/local/src/SDRPlusPlus/core/src/dsp/demod/fm.h:25:14: note: candidate expects 5 arguments, 4 provided make[2]: [decoder_modules/sdrpp_radiosonde/CMakeFiles/radiosonde_decoder.dir/build.make:118: decoder_modules/sdrpp_radiosonde/CMakeFiles/radiosonde_decoder.dir/src/main.cpp.o] Error 1 make[1]: [CMakeFiles/Makefile2:2076: decoder_modules/sdrpp_radiosonde/CMakeFiles/radiosonde_decoder.dir/all] Error 2 make: *** [Makefile:136: all] Error 2

`

rjfrost000 commented 12 months ago

I got the same error last night while trying to build sdr++ nighty master using the instructions on this github site. I was building this on a pi4 but after reading the github again I'm pretty sure the radiosode decoder was made for x86_64 and not arm64

doctorvenables commented 11 months ago

I got this as well and I'm not on arm64 (I'm on Debian Bookworm - may that's the issue?). It seems someone else got this in another open thread.

doctorvenables commented 11 months ago

The pull request fixes this, at least it did for my Debian Bookworm system.

KmanOz commented 3 months ago

The pull request fixes this, at least it did for my Debian Bookworm system.

This certainly fixed it for me. Thanks for that, on Bookworm AMDx64

gariac commented 3 months ago

The pull request fixes this, at least it did for my Debian Bookworm system.

This certainly fixed it for me. Thanks for that, on Bookworm AMDx64

I don't know how to apply a pull but it was easy enough to edit the source code for one line. Also AMD 64.