-
Notifications
You must be signed in to change notification settings - Fork 156
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
Poco-Source #54
Comments
Hi! |
Hi! Nice idea, but conan.io is not necessary, because you could use poco as a git sub module. This includes the poco source code on git clone/fetch/pull. |
git submodule is proper solution; however, it requires buildsystem changes because poco libraries are now spread directly under platform directory. OSP should also be moved into separate repository and included as submodule, see #15 I'm thinking probably the cleanest solution would be POCO and OSP in separate submodules, the rest either directly under if anyone wants to tackle this, go for it, see also #53 conan is a separate concern from this |
Before anyone starts doing some work regarding this - please coordinate with me first. |
I agree, changing the build system to CMake (#53) is a necessary requirement. With CMake it is also easy to integrate the build scripts from submodules into the build system of the main project. E. g. this means that the CMake script "platform" can build Poco and OSP as submodules. It is not necessary to carry (possibly outdated) dependencies with you. But where to start? First switch to CMake and then outsource OSP? Or outsource OSP first and then switch to CMake? |
Switch to (add support for) CMake to macchina.io first. This is already planned for July for a customer project. |
I'm wondering that there is a copy of the poco libraries in this project. I think it's better to include poco as a git sub module, because in this case bugfixes from poco are also available for macchina.io. The mirai bot net used insecure IoT devices so it's neccesary to update macchina.io as fast as possible after a bugfix has been submitted to poco.
The text was updated successfully, but these errors were encountered: