search

BackyardBrains / Spike-Recorder

A neural recording app for the Windows, OS X (Mac) and Linux.
https://backyardbrains.com/products/spikerecorder
31 stars 27 forks source link

Build Fail #64

Open smbaca opened 5 months ago

smbaca commented 5 months ago

On raspberry pi4b and followed instructions. Everything seemed ok until make and then got the following error:

In file included from src/CalibrationWindow.cpp:6: src/engine/RecordingManager.h:19:10: fatal error: BYBBootloaderController.h: No such file or directory 19 | #include "BYBBootloaderController.h" | ^~~~~~~ compilation terminated. make: *** [Makefile:78: src/CalibrationWindow.o] Error 1 smbaca@chinchilla:~/Spike-Recorder $

smbaca commented 5 months ago

Moving the .h file to the correct location worked. Now just struggling with the right bass library for Raspberry pi OS and architecture. Keeps saying it is an incompatible version.

smbaca commented 5 months ago

Ok, per help from Greg, it looks like I have resolved some of the issues by moving some of the cpp and h files to src when compiling. Below is the error I am now getting with the make.

smbaca@chinchilla:~/Spike-Recorder $ make g++ -o src/engine/AudioInputConfig.o -c src/engine/AudioInputConfig.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/CalibrationWindow.o -c src/CalibrationWindow.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/TouchDropDownList.o -c src/widgets/TouchDropDownList.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/HorizontalColorPicker.o -c src/widgets/HorizontalColorPicker.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/widgets/HorizontalColorPicker.cpp: In member function 'virtual void BackyardBrains::HorizontalColorPickerPopup::mousePressEvent(BackyardBrains::Widgets::MouseEvent)': src/widgets/HorizontalColorPicker.cpp:91:68: warning: comparison of integer expressions of different signedness: 'const int' and 'std::vector::size_type' {aka 'long unsigned int'} [-Wsign-compare] 91 | if(event->button() == Widgets::LeftButton && event->pos().y<(_content.size()(SLICE_W +padding)+padding)) | ~~~~^~~~~~~~~~~

g++ -o src/widgets/HorizontalNumberPicker.o -c src/widgets/HorizontalNumberPicker.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/libraries/tinyxml2.o -c src/libraries/tinyxml2.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/RecordingManager.o -c src/engine/RecordingManager.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/engine/RecordingManager.cpp:116: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 116 #pragma mark - HID device
src/engine/RecordingManager.cpp:543: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 543 #pragma mark - Serial device
src/engine/RecordingManager.cpp:835: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 835 #pragma mark - File device
src/engine/RecordingManager.cpp:1354: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 1354 #pragma mark - Advance one step
src/engine/RecordingManager.cpp:2272: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2272 #pragma mark - Filters
src/engine/RecordingManager.cpp:2491: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2491 #pragma mark - Bind/Unbind device
src/engine/RecordingManager.cpp:2803: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2803 #pragma mark - Bootloader
src/engine/RecordingManager.cpp:2849: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2849 #pragma mark - Joystick related
src/engine/RecordingManager.cpp:2900: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2900 #pragma mark - Input Config related
src/engine/RecordingManager.cpp: In member function 'bool BackyardBrains::RecordingManager::loadFile(const char*)': src/engine/RecordingManager.cpp:870:35: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 870 for(unsigned int i = 0; i < nchan; i++) { ^~~~~ src/engine/RecordingManager.cpp:882:35: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 882 for(unsigned int i = 0; i < nchan; i++) { ^~~~~ src/engine/RecordingManager.cpp: In member function 'void BackyardBrains::RecordingManager::advanceSerialMode(uint32_t)': src/engine/RecordingManager.cpp:1702:9: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 1702 if(triggerd) ^~ src/engine/RecordingManager.cpp:1705:13: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 1705 delete[] channels; ^~ src/engine/RecordingManager.cpp:1725:36: warning: comparison of integer expressions of different signedness: 'int' and 'uint32_t' {aka 'unsigned int'} [-Wsign-compare] 1725 for( int index= 0;index<samples;index++) ~^~~~ src/engine/RecordingManager.cpp: In member function 'void BackyardBrains::RecordingManager::advance(uint32_t)': src/engine/RecordingManager.cpp:2096:31: warning: comparison of integer expressions of different signedness: 'int' and 'DWORD' {aka 'unsigned int'} [-Wsign-compare] 2096 for (int i = 0; i < samplesRead/channum; i++) { ^~~~~~~ src/engine/RecordingManager.cpp:2136:31: warning: comparison of integer expressions of different signedness: 'int' and 'DWORD' {aka 'unsigned int'} [-Wsign-compare] 2136 for (int i = 0; i < samplesRead/channum; i++) { ^~~~~~~ src/engine/RecordingManager.cpp:2184:32: warning: comparison of integer expressions of different signedness: 'int' and 'DWORD' {aka 'unsigned int'} [-Wsign-compare] 2184 for (int i = 0; i < samplesRead/channum; i++) { ^~~~~~~ src/engine/RecordingManager.cpp: In member function 'void BackyardBrains::RecordingManager::saveInputConfigSettings()': src/engine/RecordingManager.cpp:3145:22: warning: variable 'foundPort' set but not used [-Wunused-but-set-variable] 3145 bool foundPort = false; ^~~~~ In file included from src/engine/FileRecorder.h:6, from src/engine/RecordingManager.cpp:4: src/engine/WavTxtRecorder.h: At global scope: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 static uint8 s_outbuf[BUF_SIZE]; ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 static uint8 s_inbuf[BUF_SIZE]; ^~~ g++ -o src/engine/AnalysisManager.o -c src/engine/AnalysisManager.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/FileRecorder.o -c src/engine/FileRecorder.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags In file included from src/engine/FileRecorder.h:6, from src/engine/FileRecorder.cpp:1: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 static uint8 s_outbuf[BUF_SIZE]; ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 static uint8 s_inbuf[BUF_SIZE]; ^~~ g++ -o src/engine/Player.o -c src/engine/Player.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/ArduinoSerial.o -c src/engine/ArduinoSerial.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/engine/ArduinoSerial.cpp:166: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 166 #pragma mark - Port scanning and opening
src/engine/ArduinoSerial.cpp:1711: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 1711 #pragma mark - Reading and processing
src/engine/ArduinoSerial.cpp:2866: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2866 #pragma mark - Getters/parameters
src/engine/ArduinoSerial.cpp:2907: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 2907 #pragma mark - Messages - Write
src/engine/ArduinoSerial.cpp: In member function 'int BackyardBrains::ArduinoSerial::processDataIntoSamples(char, int, int16_t)': src/engine/ArduinoSerial.cpp:1903:13: warning: unused variable 'g' [-Wunused-variable] 1903 int g=0; ^ src/engine/ArduinoSerial.cpp:1904:14: warning: unused variable 'enablePrint' [-Wunused-variable] 1904 bool enablePrint = false; ^~~ src/engine/ArduinoSerial.cpp: In member function 'void BackyardBrains::ArduinoSerial::executeOneMessage(std::string, std::string, int)': src/engine/ArduinoSerial.cpp:2511:25: warning: unused variable 'channel' [-Wunused-variable] 2511 int channel = std::stoi(channelStr); ^~~ src/engine/ArduinoSerial.cpp:2556:25: warning: unused variable 'channel' [-Wunused-variable] 2556 int channel = std::stoi(channelStr); ^~~ g++ -o src/engine/SpikeSorter.o -c src/engine/SpikeSorter.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/engine/SpikeSorter.cpp: In member function 'void BackyardBrains::SpikeSorter::findAllSpikes(const std::string&, int)': src/engine/SpikeSorter.cpp:99:18: warning: comparison of integer expressions of different signedness: 'int' and 'DWORD' {aka 'unsigned int'} [-Wsign-compare] 99 for(int i=0;i<info.chans;i++) ~^~~ src/engine/SpikeSorter.cpp:103:18: warning: comparison of integer expressions of different signedness: 'int' and 'DWORD' {aka 'unsigned int'} [-Wsign-compare] 103 for(int i=0;i<info.chans;i++) ~^~~ g++ -o src/engine/SpikeAnalysis.o -c src/engine/SpikeAnalysis.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/BASSErrors.o -c src/engine/BASSErrors.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/FileReadUtil.o -c src/engine/FileReadUtil.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/FFTBackend.o -c src/engine/FFTBackend.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/HIDUsbManager.o -c src/engine/HIDUsbManager.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/engine/HIDUsbManager.cpp:816: warning: ignoring '#pragma mark ' [-Wunknown-pragmas] 816 #pragma mark - Read/parse

g++ -o src/engine/EkgBackend.o -c src/engine/EkgBackend.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/FilterBase.o -c src/engine/FilterBase.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/engine/FilterBase.cpp: In member function 'void BackyardBrains::FilterBase::filterContiguousData(double, uint32_t, bool)': src/engine/FilterBase.cpp:84:14: warning: comparison of integer expressions of different signedness: 'int' and 'uint32_t' {aka 'unsigned int'} [-Wsign-compare] 84 | for(n=2;n<numFrames+2;n++) | ~^~~~ g++ -o src/engine/HighPassFilter.o -c src/engine/HighPassFilter.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/LowPassFilter.o -c src/engine/LowPassFilter.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/NotchFilter.o -c src/engine/NotchFilter.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/LayoutItem.o -c src/widgets/LayoutItem.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/BoxLayout.o -c src/widgets/BoxLayout.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/Widget.o -c src/widgets/Widget.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/Painter.o -c src/widgets/Painter.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/PushButton.o -c src/widgets/PushButton.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/DropDownList.o -c src/widgets/DropDownList.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/ScrollBar.o -c src/widgets/ScrollBar.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/Application.o -c src/widgets/Application.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/BitmapFontGL.o -c src/widgets/BitmapFontGL.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/TextureGL.o -c src/widgets/TextureGL.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/LoadTexture.o -c src/widgets/LoadTexture.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/Label.o -c src/widgets/Label.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/ErrorBox.o -c src/widgets/ErrorBox.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/TextInput.o -c src/widgets/TextInput.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/widgets/TextInput.cpp: In member function 'virtual void BackyardBrains::Widgets::TextInput::keyDownEvent(const SDL_Event&)': src/widgets/TextInput.cpp:246:47: warning: comparison of integer expressions of different signedness: 'std::__cxx11::basic_string::size_type' {aka 'long unsigned int'} and 'int' [-Wsign-compare] 246 | if((_text.size()-_invisibleOffset)>_cursorPosition) | ~~~~~~~^~~~ src/widgets/TextInput.cpp: In member function 'int BackyardBrains::Widgets::TextInput::getInt()': src/widgets/TextInput.cpp:296:30: warning: unused variable 'sz' [-Wunused-variable] 296 | std::string::size_type sz; // alias of size_t | ^~ src/widgets/TextInput.cpp: In member function 'float BackyardBrains::Widgets::TextInput::getFloat()': src/widgets/TextInput.cpp:322:30: warning: unused variable 'sz' [-Wunused-variable] 322 | std::string::size_type sz; // alias of size_t | ^~ g++ -o src/widgets/RangeSelector.o -c src/widgets/RangeSelector.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/SwitchLayout.o -c src/widgets/SwitchLayout.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/ToolTip.o -c src/widgets/ToolTip.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/Plot.o -c src/widgets/Plot.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/widgets/TabBar.o -c src/widgets/TabBar.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/DropDownList.o -c src/DropDownList.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/Log.o -c src/Log.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/Log.cpp: In static member function 'static void BackyardBrains::Log::msg(const char, ...)': src/Log.cpp:44:14: warning: unused variable 'ms' [-Wunused-variable] 44 | long ms = tp.tv_sec 1000 + tp.tv_usec / 1000; | ^~ src/Log.cpp:55:9: warning: second parameter of 'va_start' not last named argument [-Wvarargs] 55 | va_start(args,format); | ^~~~ g++ -o src/Game.o -c src/Game.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags In file included from src/engine/FileRecorder.h:6, from src/Game.h:7, from src/Game.cpp:1: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/main.o -c src/main.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags In file included from src/engine/FileRecorder.h:6, from src/Game.h:7, from src/main.cpp:2: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/MainView.o -c src/MainView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags In file included from src/engine/FileRecorder.h:6, from src/MainView.cpp:14: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/AudioView.o -c src/AudioView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/AudioView.cpp: In member function 'virtual void BackyardBrains::AudioView::mouseMotionEvent(BackyardBrains::Widgets::MouseEvent)': src/AudioView.cpp:1306:23: warning: unused variable 'newGain' [-Wunused-variable] 1306 | float newGain = _prevGainstd::fabs((height()_channels[_clickedGain].pos-event->pos().y)/(float)_clickedPixelOffset); | ^~~ In file included from src/engine/FileRecorder.h:6, from src/AudioView.cpp:9: src/engine/WavTxtRecorder.h: At global scope: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/ConfigView.o -c src/ConfigView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/AnalysisView.o -c src/AnalysisView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/AnalysisView.cpp: In member function 'void BackyardBrains::AnalysisView::trainDeleted(int)': src/AnalysisView.cpp:265:44: warning: comparison of integer expressions of different signedness: 'int' and 'std::vector::size_type' {aka 'long unsigned int'} [-Wsign-compare] 265 | for(int i=_trainList->selectedTrain();i<_spikeTrains.size()-1;i++) | ~^~~~~~ In file included from src/engine/FileRecorder.h:6, from src/AnalysisView.cpp:10: src/engine/WavTxtRecorder.h: At global scope: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/AnalysisAudioView.o -c src/AnalysisAudioView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/AnalysisTrainList.o -c src/AnalysisTrainList.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/AnalysisTrainList.cpp: In member function 'void BackyardBrains::AnalysisTrainList::setInitialSelection()': src/AnalysisTrainList.cpp:30:22: warning: comparison of integer expressions of different signedness: 'int' and 'std::vector::size_type' {aka 'long unsigned int'} [-Wsign-compare] 30 | for(int st = 0;st<_spikeTrains.size()-1;st++) | ^~~~~~~~ src/AnalysisTrainList.cpp: In member function 'virtual void BackyardBrains::AnalysisTrainList::paintEvent()': src/AnalysisTrainList.cpp:93:21: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 93 | if(i==_selectedTrain && _spikeTrains[i].channelIndex == channelIndex) | ~^~~~ src/AnalysisTrainList.cpp:131:21: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 131 | if(i==_selectedTrain && _selectedTrain != (int)_spikeTrains.size()-1 && _spikeTrains[i].channelIndex == channelIndex) | ~^~~~ src/AnalysisTrainList.cpp: In member function 'virtual void BackyardBrains::AnalysisTrainList::mousePressEvent(BackyardBrains::Widgets::MouseEvent)': src/AnalysisTrainList.cpp:184:52: warning: comparison of integer expressions of different signedness: 'int' and 'std::vector::size_type' {aka 'long unsigned int'} [-Wsign-compare] 184 | for(int tempSTIndex = 0;tempSTIndex<_spikeTrains.size();tempSTIndex++) | ~~~^~~~~~~~ g++ -o src/AnalysisPlots.o -c src/AnalysisPlots.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/RecordingBar.o -c src/RecordingBar.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags In file included from src/engine/FileRecorder.h:6, from src/RecordingBar.cpp:6: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/ColorDropDownList.o -c src/ColorDropDownList.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/FFTView.o -c src/FFTView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/ThresholdPanel.o -c src/ThresholdPanel.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/engine/WavTxtRecorder.o -c src/engine/WavTxtRecorder.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/engine/WavTxtRecorder.cpp: In member function 'bool BackyardBrains::WavTxtRecorder::ensure_file_exists_and_is_readable(const char)': src/engine/WavTxtRecorder.cpp:118:3: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 118 | if (!p) | ^~ src/engine/WavTxtRecorder.cpp:121:5: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 121 | fseek(p, 0, SEEK_END); | ^~~~~ In file included from src/engine/WavTxtRecorder.cpp:9: src/engine/WavTxtRecorder.h: At global scope: src/engine/WavTxtRecorder.h:21:14: warning: 's_outbuf' defined but not used [-Wunused-variable] 21 | static uint8 s_outbuf[BUF_SIZE]; | ^~~~ src/engine/WavTxtRecorder.h:20:14: warning: 's_inbuf' defined but not used [-Wunused-variable] 20 | static uint8 s_inbuf[BUF_SIZE]; | ^~~ g++ -o src/FirmwareUpdateView.o -c src/FirmwareUpdateView.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags g++ -o src/BYBBootloaderController.o -c src/BYBBootloaderController.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/BYBBootloaderController.cpp: In member function 'void BackyardBrains::BYBBootloaderController::startUpdateProcess()': src/BYBBootloaderController.cpp:124:16: warning: unused variable 'len' [-Wunused-variable] 124 | size_t len = 0; | ^~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:139:33: error: expected constructor, destructor, or type conversion before ';' token 139 | initTransferOfFirmware(); | ^ src/BYBBootloaderController.cpp:140:9: error: 'stage' does not name a type 140 | stage = BOOTLOADER_STAGE_OFF; | ^~~~~ src/BYBBootloaderController.cpp:143:10: error: 'BYBBootloaderController' has not been declared 143 | void BYBBootloaderController::initTransferOfFirmware() | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'void initTransferOfFirmware()': src/BYBBootloaderController.cpp:148:29: error: 'readDataFromSerialPort' was not declared in this scope 148 | while((readLength = readDataFromSerialPort(tempBuffer))<=0) | ^~~~~~ src/BYBBootloaderController.cpp:163:13: error: 'writeDataToSerialPort' was not declared in this scope 163 | writeDataToSerialPort(tempBuffer, 1); | ^~~~~ src/BYBBootloaderController.cpp:164:33: error: 'readDataFromSerialPort' was not declared in this scope 164 | while((readLength = readDataFromSerialPort(tempBuffer))<=0) | ^~~~~~ src/BYBBootloaderController.cpp:176:42: error: 'dataFromFile' was not declared in this scope 176 | uint32_t programSize = (uint32_t)dataFromFile.size(); | ^~~~ src/BYBBootloaderController.cpp:181:9: error: 'writeDataToSerialPort' was not declared in this scope 181 | writeDataToSerialPort(tempBuffer, 4); | ^~~~~ src/BYBBootloaderController.cpp:185:19: error: 'HexRecord' was not declared in this scope; did you mean 'BackyardBrains::HexRecord'? 185 | std::list::iterator it; | ^~~~~ | BackyardBrains::HexRecord In file included from src/BYBBootloaderController.cpp:22: src/BYBBootloaderController.h:32:8: note: 'BackyardBrains::HexRecord' declared here 32 | struct HexRecord | ^~~~~ src/BYBBootloaderController.cpp:185:28: error: template argument 1 is invalid 185 | std::list::iterator it; | ^ src/BYBBootloaderController.cpp:185:28: error: template argument 2 is invalid src/BYBBootloaderController.cpp:185:40: error: expected initializer before 'it' 185 | std::list::iterator it; | ^~ src/BYBBootloaderController.cpp:186:9: error: 'it' was not declared in this scope; did you mean 'int'? 186 | it = dataFromFile.begin(); | ^~ | int src/BYBBootloaderController.cpp:189:33: error: 'readDataFromSerialPort' was not declared in this scope 189 | while((readLength = readDataFromSerialPort(tempBuffer))<=0) | ^~~~~~ src/BYBBootloaderController.cpp:194:17: error: 'progress' was not declared in this scope 194 | progress = 100; | ^~~~ src/BYBBootloaderController.cpp:206:17: error: 'progress' was not declared in this scope; did you mean 'tempProgress'? 206 | progress = tempProgress; | ^~~~ | tempProgress src/BYBBootloaderController.cpp:214:9: error: 'progress' was not declared in this scope 214 | progress = 100; | ^~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:218:9: error: 'BYBBootloaderController' has not been declared 218 | int BYBBootloaderController::readDataFromSerialPort(char buffer) | ^~~~~~~ In file included from /usr/include/aarch64-linux-gnu/sys/types.h:179, from /usr/include/stdlib.h:395, from /usr/include/c++/12/cstdlib:75, from /usr/include/c++/12/ext/string_conversions.h:41, from /usr/include/c++/12/bits/basic_string.h:3960, from /usr/include/c++/12/string:53, from /usr/include/c++/12/bits/locale_classes.h:40, from /usr/include/c++/12/bits/ios_base.h:41, from /usr/include/c++/12/ios:42, from /usr/include/c++/12/istream:38, from /usr/include/c++/12/fstream:38, from src/BYBBootloaderController.h:16: src/BYBBootloaderController.cpp: In function 'int readDataFromSerialPort(char)': src/BYBBootloaderController.cpp:227:20: error: 'portHandle' was not declared in this scope 227 | FD_SET(portHandle, &read_fds); | ^~~~~~ src/BYBBootloaderController.cpp:250:36: error: 'fd' was not declared in this scope 250 | if (size == 0 && ioctl(fd, TIOCMGET, &bits) < 0) | ^~ src/BYBBootloaderController.cpp:250:40: error: 'TIOCMGET' was not declared in this scope 250 | if (size == 0 && ioctl(fd, TIOCMGET, &bits) < 0) | ^~~~ src/BYBBootloaderController.cpp:250:30: error: 'ioctl' was not declared in this scope 250 | if (size == 0 && ioctl(fd, TIOCMGET, &bits) < 0) | ^~~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:289:9: error: 'BYBBootloaderController' has not been declared 289 | int BYBBootloaderController::writeDataToSerialPort(const void ptr, int len) | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'int writeDataToSerialPort(const void, int)': src/BYBBootloaderController.cpp:297:31: error: 'portHandle' was not declared in this scope 297 | n = write(portHandle, (const char )ptr + written, len - written); | ^~~~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:353:10: error: 'BYBBootloaderController' has not been declared 353 | void BYBBootloaderController::parseLineOfHex(char lineFromFile) | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'void parseLineOfHex(char)': src/BYBBootloaderController.cpp:356:9: error: 'checkChecksum' was not declared in this scope 356 | checkChecksum(lineFromFile); | ^~~~~ src/BYBBootloaderController.cpp:363:12: error: 'currentAddress' was not declared in this scope; did you mean 'tempCurrentAddress'? 363 | if(currentAddress!=0 && tempCurrentAddress!=currentAddress && tempCurrentAddress>currentAddress) | ^~~~~~ | tempCurrentAddress src/BYBBootloaderController.cpp:371:24: error: 'currentIndexInsidePage' was not declared in this scope 371 | if(currentIndexInsidePage==0) | ^~~~~~ src/BYBBootloaderController.cpp:373:25: error: 'newHexRecord' was not declared in this scope 373 | newHexRecord = new HexRecord(); | ^~~~ src/BYBBootloaderController.cpp:373:44: error: expected type-specifier before 'HexRecord' 373 | newHexRecord = new HexRecord(); | ^~~~~ src/BYBBootloaderController.cpp:382:21: error: 'newHexRecord' was not declared in this scope 382 | newHexRecord->dataPage[currentIndexInsidePage]=255; | ^~~~ src/BYBBootloaderController.cpp:382:44: error: 'currentIndexInsidePage' was not declared in this scope 382 | newHexRecord->dataPage[currentIndexInsidePage]=255; | ^~~~~~ src/BYBBootloaderController.cpp:389:25: error: 'dataFromFile' was not declared in this scope; did you mean 'lineFromFile'? 389 | dataFromFile.push_back(newHexRecord); | ^~~~ | lineFromFile src/BYBBootloaderController.cpp:405:17: error: 'getDataRecord' was not declared in this scope 405 | getDataRecord(lineFromFile, lengthOfData); | ^~~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:430:10: error: 'BYBBootloaderController' has not been declared 430 | void BYBBootloaderController::getDataRecord(char lineFromFile, int lengthOfData) | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'void getDataRecord(char, int)': src/BYBBootloaderController.cpp:435:16: error: 'currentIndexInsidePage' was not declared in this scope 435 | if(currentIndexInsidePage==0) | ^~~~~~ src/BYBBootloaderController.cpp:437:17: error: 'newHexRecord' was not declared in this scope 437 | newHexRecord = new HexRecord(); | ^~~~ src/BYBBootloaderController.cpp:437:36: error: expected type-specifier before 'HexRecord' 437 | newHexRecord = new HexRecord(); | ^~~~~ src/BYBBootloaderController.cpp:444:40: error: 'currentAddress' was not declared in this scope 444 | newHexRecord->address =currentAddress; | ^~~~~~ src/BYBBootloaderController.cpp:450:13: error: 'newHexRecord' was not declared in this scope 450 | newHexRecord->dataPage[currentIndexInsidePage]=tempByte; | ^~~~ src/BYBBootloaderController.cpp:450:36: error: 'currentIndexInsidePage' was not declared in this scope 450 | newHexRecord->dataPage[currentIndexInsidePage]=tempByte; | ^~~~~~ src/BYBBootloaderController.cpp:451:13: error: 'currentAddress' was not declared in this scope 451 | currentAddress++; | ^~~~~~ src/BYBBootloaderController.cpp:458:17: error: 'dataFromFile' was not declared in this scope; did you mean 'lineFromFile'? 458 | dataFromFile.push_back(newHexRecord); | ^~~~ | lineFromFile src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:464:9: error: 'BYBBootloaderController' has not been declared 464 | int BYBBootloaderController::checkChecksum(char lineFromFile) | ^~~~~~~ src/BYBBootloaderController.cpp:490:1: error: expected declaration before '}' token 490 | } | ^ make: *** [Makefile:83: src/BYBBootloaderController.o] Error 1

smbaca commented 5 months ago

All of the errors now concern the BYBBootloaderController.cpp

g++ -o src/BYBBootloaderController.o -c src/BYBBootloaderController.cpp -g -O2 -Isrc -Isupport -Isrc/libraries -I. -Lsrc/support -Wall -DSIGSLOT_PURE_ISO --std=c++11 sdl2-config --cflags src/BYBBootloaderController.cpp: In member function 'void BackyardBrains::BYBBootloaderController::startUpdateProcess()': src/BYBBootloaderController.cpp:124:16: warning: unused variable 'len' [-Wunused-variable] 124 | size_t len = 0; | ^~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:139:33: error: expected constructor, destructor, or type conversion before ';' token 139 | initTransferOfFirmware(); | ^ src/BYBBootloaderController.cpp:140:9: error: 'stage' does not name a type 140 | stage = BOOTLOADER_STAGE_OFF; | ^~~~~ src/BYBBootloaderController.cpp:143:10: error: 'BYBBootloaderController' has not been declared 143 | void BYBBootloaderController::initTransferOfFirmware() | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'void initTransferOfFirmware()': src/BYBBootloaderController.cpp:148:29: error: 'readDataFromSerialPort' was not declared in this scope 148 | while((readLength = readDataFromSerialPort(tempBuffer))<=0) | ^~~~~~ src/BYBBootloaderController.cpp:163:13: error: 'writeDataToSerialPort' was not declared in this scope 163 | writeDataToSerialPort(tempBuffer, 1); | ^~~~~ src/BYBBootloaderController.cpp:164:33: error: 'readDataFromSerialPort' was not declared in this scope 164 | while((readLength = readDataFromSerialPort(tempBuffer))<=0) | ^~~~~~ src/BYBBootloaderController.cpp:176:42: error: 'dataFromFile' was not declared in this scope 176 | uint32_t programSize = (uint32_t)dataFromFile.size(); | ^~~~ src/BYBBootloaderController.cpp:181:9: error: 'writeDataToSerialPort' was not declared in this scope 181 | writeDataToSerialPort(tempBuffer, 4); | ^~~~~ src/BYBBootloaderController.cpp:185:19: error: 'HexRecord' was not declared in this scope; did you mean 'BackyardBrains::HexRecord'? 185 | std::list::iterator it; | ^~~~~ | BackyardBrains::HexRecord In file included from src/BYBBootloaderController.cpp:22: src/BYBBootloaderController.h:32:8: note: 'BackyardBrains::HexRecord' declared here 32 | struct HexRecord | ^~~~~ src/BYBBootloaderController.cpp:185:28: error: template argument 1 is invalid 185 | std::list::iterator it; | ^ src/BYBBootloaderController.cpp:185:28: error: template argument 2 is invalid src/BYBBootloaderController.cpp:185:40: error: expected initializer before 'it' 185 | std::list::iterator it; | ^~ src/BYBBootloaderController.cpp:186:9: error: 'it' was not declared in this scope; did you mean 'int'? 186 | it = dataFromFile.begin(); | ^ | int src/BYBBootloaderController.cpp:189:33: error: 'readDataFromSerialPort' was not declared in this scope 189 | while((readLength = readDataFromSerialPort(tempBuffer))<=0) | ^~~~~~ src/BYBBootloaderController.cpp:194:17: error: 'progress' was not declared in this scope 194 | progress = 100; | ^~~~ src/BYBBootloaderController.cpp:206:17: error: 'progress' was not declared in this scope; did you mean 'tempProgress'? 206 | progress = tempProgress; | ^~~~ | tempProgress src/BYBBootloaderController.cpp:214:9: error: 'progress' was not declared in this scope 214 | progress = 100; | ^~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:218:9: error: 'BYBBootloaderController' has not been declared 218 | int BYBBootloaderController::readDataFromSerialPort(char buffer) | ^~~~~~~ In file included from /usr/include/aarch64-linux-gnu/sys/types.h:179, from /usr/include/stdlib.h:395, from /usr/include/c++/12/cstdlib:75, from /usr/include/c++/12/ext/string_conversions.h:41, from /usr/include/c++/12/bits/basic_string.h:3960, from /usr/include/c++/12/string:53, from /usr/include/c++/12/bits/locale_classes.h:40, from /usr/include/c++/12/bits/ios_base.h:41, from /usr/include/c++/12/ios:42, from /usr/include/c++/12/istream:38, from /usr/include/c++/12/fstream:38, from src/BYBBootloaderController.h:16: src/BYBBootloaderController.cpp: In function 'int readDataFromSerialPort(char)': src/BYBBootloaderController.cpp:227:20: error: 'portHandle' was not declared in this scope 227 | FD_SET(portHandle, &read_fds); | ^~~~~~ src/BYBBootloaderController.cpp:250:36: error: 'fd' was not declared in this scope 250 | if (size == 0 && ioctl(fd, TIOCMGET, &bits) < 0) | ^~ src/BYBBootloaderController.cpp:250:40: error: 'TIOCMGET' was not declared in this scope 250 | if (size == 0 && ioctl(fd, TIOCMGET, &bits) < 0) | ^~~~ src/BYBBootloaderController.cpp:250:30: error: 'ioctl' was not declared in this scope 250 | if (size == 0 && ioctl(fd, TIOCMGET, &bits) < 0) | ^~~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:289:9: error: 'BYBBootloaderController' has not been declared 289 | int BYBBootloaderController::writeDataToSerialPort(const void ptr, int len) | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'int writeDataToSerialPort(const void, int)': src/BYBBootloaderController.cpp:297:31: error: 'portHandle' was not declared in this scope 297 | n = write(portHandle, (const char )ptr + written, len - written); | ^~~~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:353:10: error: 'BYBBootloaderController' has not been declared 353 | void BYBBootloaderController::parseLineOfHex(char lineFromFile) | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'void parseLineOfHex(char)': src/BYBBootloaderController.cpp:356:9: error: 'checkChecksum' was not declared in this scope 356 | checkChecksum(lineFromFile); | ^~~~~ src/BYBBootloaderController.cpp:363:12: error: 'currentAddress' was not declared in this scope; did you mean 'tempCurrentAddress'? 363 | if(currentAddress!=0 && tempCurrentAddress!=currentAddress && tempCurrentAddress>currentAddress) | ^~~~~~ | tempCurrentAddress src/BYBBootloaderController.cpp:371:24: error: 'currentIndexInsidePage' was not declared in this scope 371 | if(currentIndexInsidePage==0) | ^~~~~~ src/BYBBootloaderController.cpp:373:25: error: 'newHexRecord' was not declared in this scope 373 | newHexRecord = new HexRecord(); | ^~~~ src/BYBBootloaderController.cpp:373:44: error: expected type-specifier before 'HexRecord' 373 | newHexRecord = new HexRecord(); | ^~~~~ src/BYBBootloaderController.cpp:382:21: error: 'newHexRecord' was not declared in this scope 382 | newHexRecord->dataPage[currentIndexInsidePage]=255; | ^~~~ src/BYBBootloaderController.cpp:382:44: error: 'currentIndexInsidePage' was not declared in this scope 382 | newHexRecord->dataPage[currentIndexInsidePage]=255; | ^~~~~~ src/BYBBootloaderController.cpp:389:25: error: 'dataFromFile' was not declared in this scope; did you mean 'lineFromFile'? 389 | dataFromFile.push_back(newHexRecord); | ^~~~ | lineFromFile src/BYBBootloaderController.cpp:405:17: error: 'getDataRecord' was not declared in this scope 405 | getDataRecord(lineFromFile, lengthOfData); | ^~~~~ src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:430:10: error: 'BYBBootloaderController' has not been declared 430 | void BYBBootloaderController::getDataRecord(char lineFromFile, int lengthOfData) | ^~~~~~~ src/BYBBootloaderController.cpp: In function 'void getDataRecord(char, int)': src/BYBBootloaderController.cpp:435:16: error: 'currentIndexInsidePage' was not declared in this scope 435 | if(currentIndexInsidePage==0) | ^~~~~~ src/BYBBootloaderController.cpp:437:17: error: 'newHexRecord' was not declared in this scope 437 | newHexRecord = new HexRecord(); | ^~~~ src/BYBBootloaderController.cpp:437:36: error: expected type-specifier before 'HexRecord' 437 | newHexRecord = new HexRecord(); | ^~~~~ src/BYBBootloaderController.cpp:444:40: error: 'currentAddress' was not declared in this scope 444 | newHexRecord->address =currentAddress; | ^~~~~~ src/BYBBootloaderController.cpp:450:13: error: 'newHexRecord' was not declared in this scope 450 | newHexRecord->dataPage[currentIndexInsidePage]=tempByte; | ^~~~ src/BYBBootloaderController.cpp:450:36: error: 'currentIndexInsidePage' was not declared in this scope 450 | newHexRecord->dataPage[currentIndexInsidePage]=tempByte; | ^~~~~~ src/BYBBootloaderController.cpp:451:13: error: 'currentAddress' was not declared in this scope 451 | currentAddress++; | ^~~~~~ src/BYBBootloaderController.cpp:458:17: error: 'dataFromFile' was not declared in this scope; did you mean 'lineFromFile'? 458 | dataFromFile.push_back(newHexRecord); | ^~~~ | lineFromFile src/BYBBootloaderController.cpp: At global scope: src/BYBBootloaderController.cpp:464:9: error: 'BYBBootloaderController' has not been declared 464 | int BYBBootloaderController::checkChecksum(char lineFromFile) | ^~~~~~~ src/BYBBootloaderController.cpp:490:1: error: expected declaration before '}' token 490 | } | ^ make: *** [Makefile:83: src/BYBBootloaderController.o] Error 1

amendolajamie commented 1 week ago

I submitted a change that fixes it for linux on x86. I expect that it will also build on a raspberry pi now. The change is not merged at this time, hopefully someone will review it and merge it.

stanislavmircic commented 1 week ago

Thank you for your help. We really appreciate it. We will review the code and test everything before merging.

stanislavmircic commented 5 days ago

I tried to compile with Makefile.win on Windows but it fails:

Executing task: make -f Makefile.win VERBOSE=1

g++ -c src/MainView.cpp -o src/MainView.o In file included from src/MainView.h:4, from src/MainView.cpp:1: src/widgets/Widget.h:13:10: fatal error: sigslot.h: No such file or directory 13 | #include | ^~~ compilation terminated. make: *** [Makefile.win:195: src/MainView.o] Error 1

I will try to fix it and merge it.

smbaca commented 5 days ago

When I was working on this the BYB folks mentioned that some of the .h files were in different locations and that helped me resolve a few errors. Might be the same situation here.

Hope to tryout the build on the raspberry pi system soon.

On Nov 18, 2024, at 10:37 AM, stanislavmircic @.***> wrote:

I tried to compile with Makefile.win on Windows but it fails:

Executing task: make -f Makefile.win VERBOSE=1

g++ -c src/MainView.cpp -o src/MainView.o In file included from src/MainView.h:4, from src/MainView.cpp:1: src/widgets/Widget.h:13:10: fatal error: sigslot.h: No such file or directory 13 | #include | ^~~ compilation terminated. make: *** [Makefile.win:195: src/MainView.o] Error 1

I will try to fix it and merge it.

— Reply to this email directly, view it on GitHub https://github.com/BackyardBrains/Spike-Recorder/issues/64#issuecomment-2483402426, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVXMEVZIRH24UAREIQIUIL2BICVNAVCNFSM6AAAAABIZYD42GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBTGQYDENBSGY. You are receiving this because you authored the thread.

amendolajamie commented 5 days ago

I also found some paths were broken for the linux build. I modified the #include macros to match the directory structure. Maybe there are more broken links in the windows build that I did not see.

Edit: On further inspection, the linux build also uses sigslot.h and the #include paths are working fine, so maybe it's just something with the windows makefile.

Edit2: The linux Makefile has "-Isupport" which is why sigslot.h is found in the path. The windows version does not have that -- this seems to be the issue with the windows build.

amendolajamie commented 4 days ago

Just now I was able to build and run it on my raspberry pi 4 running ubuntu. I haven't tested it yet .

smbaca commented 4 days ago

That’s fantastic. I will try to build and link to the BYB spiker box and see how it goes and report back.

-smb

On Nov 19, 2024, at 1:40 PM, James Amendolagine @.***> wrote:

Just now I was able to build and run it on my raspberry pi 4 running ubuntu. I haven't tested it yet .

— Reply to this email directly, view it on GitHub https://github.com/BackyardBrains/Spike-Recorder/issues/64#issuecomment-2486472776, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVXMEWYT74VDV7T3J7LLN32BOA3VAVCNFSM6AAAAABIZYD42GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBWGQ3TENZXGY. You are receiving this because you authored the thread.