thiagoralves / OpenPLC_v3

OpenPLC Runtime version 3
GNU General Public License v3.0
1.1k stars 443 forks source link

Develop refactor #57

Closed pedromorgan closed 5 years ago

pedromorgan commented 5 years ago

restructure project issue #56

pedromorgan commented 5 years ago

Ok so travis fails https://travis-ci.org/pedromorgan/OpenPLC_v3/builds/524792608?utm_medium=notification&utm_source=email

Compiling main program...
++ pkg-config --cflags --libs libmodbus
+ g++ -std=gnu++11 dnp3.cpp dnp3_dummy.cpp dnp3_publisher.cpp dnp3_receiver.cpp enip.cpp glueVars.cpp hardware_layer.cpp interactive_server.cpp main.cpp modbus.cpp modbus_master.cpp server.cpp Config0.o Res0.o -o /home/plc/OpenPLC_v3/build/bin/openplc -I /home/plc/OpenPLC_v3/runtime/core/lib -I /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1 -pthread -fpermissive -I/usr/local/include/modbus -L/usr/local/lib -lmodbus -lasiodnp3 -lasiopal -lopendnp3 -lopenpal -w
enip.cpp: In function ‘int processEnipMessage(unsigned char*, int)’:
enip.cpp:138:9: error: ‘log’ was not declared in this scope
         log(log_msg);
         ^~~
enip.cpp:138:9: note: suggested alternative: ‘long’
         log(log_msg);
         ^~~
         long
interactive_server.cpp: In function ‘void processCommand(unsigned char*, int)’:
interactive_server.cpp:296:17: error: ‘log_msg’ was not declared in this scope
         sprintf(log_msg, "Issued start_enip() command to start on port: %d\n", readCommandArgument(buffer));
                 ^~~~~~~
interactive_server.cpp:296:17: note: suggested alternative:
In file included from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/formatter.h:9:0,
                 from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/logger.h:22,
                 from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/details/registry.h:15,
                 from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/spdlog.h:13,
                 from interactive_server.cpp:39:
/home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/details/log_msg.h:16:8: note:   ‘spdlog::details::log_msg’
 struct log_msg
        ^~~~~~~
interactive_server.cpp:317:17: error: ‘log_msg’ was not declared in this scope
         sprintf(log_msg, "Issued stop_enip() command\n");
                 ^~~~~~~
interactive_server.cpp:317:17: note: suggested alternative:
In file included from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/formatter.h:9:0,
                 from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/logger.h:22,
                 from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/details/registry.h:15,
                 from /home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/spdlog.h:13,
                 from interactive_server.cpp:39:
/home/plc/OpenPLC_v3/runtime/vendor/spdlog-1.3.1/spdlog/details/log_msg.h:16:8: note:   ‘spdlog::details::log_msg’
 struct log_msg
        ^~~~~~~
+ '[' 1 -ne 0 ']'
+ echo 'Error compiling C openplc files'
Error compiling C openplc files
+ echo 'Compilation finished with errors!'
Compilation finished with errors!
+ exit 1
pedromorgan commented 5 years ago

@garretfick do I need to include fakie/ and catch22/

pedromorgan commented 5 years ago

Supperceeded