pietern / goestools

Tools to work with signals and files from GOES satellites
https://pietern.github.io/goestools/
BSD 2-Clause "Simplified" License
374 stars 83 forks source link

Some issues building with Bullseye #113

Open RyLeeRyno opened 3 years ago

RyLeeRyno commented 3 years ago

Hello Goesers,

I'm new to Github so forgive me if I'm not providing build recommendations in the correct context.

To Summarize: Debian Bullseye introduced OpenCV 4 and Project 7 which are incompatible with the existing build. I've had to make a few modifications as follows:

CMakeLists.txt in the main folder.

src/goesproc/CMakeLists.txt

src/lib/packet_reader.h

src/lib/packet_writer.h

src/goesproc/config.cc

src/goesproc/image.cc

And lastly see https://github.com/pietern/goestools/pull/83

I hope this is helpful for anyone struggling with compiling goestools with Bullseye. Thank you.

hjf commented 3 years ago

I added those changes but I'm getting this error on the latest Bullseye:

In file included from /home/pi/goestools/src/lib/packet_reader.cc:1:
/home/pi/goestools/src/lib/packet_reader.h:11:38: error: ‘uint8_t’ was not declared in this scope
   11 |   virtual bool nextPacket(std::array<uint8_t, 892>& out) = 0;
      |                                      ^~~~~~~
/home/pi/goestools/src/lib/packet_reader.h:11:50: error: template argument 1 is invalid
   11 |   virtual bool nextPacket(std::array<uint8_t, 892>& out) = 0;
      |                                                  ^
make[2]: *** [src/lib/CMakeFiles/packet_reader.dir/build.make:82: src/lib/CMakeFiles/packet_reader.dir/packet_reader.cc.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1681: src/lib/CMakeFiles/packet_reader.dir/all] Error 2

I think in your comment you mentioned src/lib/packet_writer.h twice. One should be src/lib/packet_reader.h

Higgy710 commented 2 years ago

CMakeLists.txt in the main folder.

Line 84 replace with: include_directories(BEFORE SYSTEM /usr/include/opencv4)

Replacing this messed up some dependencies. Adding this line after 84 worked though.