Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build Fail #64

Open
smbaca opened this issue Jun 5, 2024 · 10 comments
Open

Build Fail #64

smbaca opened this issue Jun 5, 2024 · 10 comments

Comments

@smbaca
Copy link

smbaca commented Jun 5, 2024

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
Copy link
Author

smbaca commented Jun 6, 2024

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
Copy link
Author

smbaca commented Jun 8, 2024

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::vectorBackyardBrains::Widgets::Color::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::vectorBackyardBrains::SpikeTrain::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::vectorBackyardBrains::SpikeTrain::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::vectorBackyardBrains::SpikeTrain::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
Copy link
Author

smbaca commented Jun 8, 2024

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
Copy link

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
Copy link
Contributor

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

@stanislavmircic
Copy link
Contributor

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 <sigslot.h>
| ^~~~~~~~~~~
compilation terminated.
make: *** [Makefile.win:195: src/MainView.o] Error 1

I will try to fix it and merge it.

@smbaca
Copy link
Author

smbaca commented Nov 18, 2024 via email

@amendolajamie
Copy link

amendolajamie commented Nov 18, 2024

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
Copy link

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

@smbaca
Copy link
Author

smbaca commented Nov 19, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants