search

PLCnext / CppExamples

Collection of various C++ sample code for PLCnext Technology controllers.
MIT License
31 stars 20 forks source link

Example templates outdated #3

Closed OWarneke closed 3 years ago

OWarneke commented 5 years ago

The components inheritance template changed. -> Please update to latest best-practice.

This is no breaking change but a convenience feature.

Adito5393 commented 3 years ago

Could you provide a link to these "latest best-practice" templates?

martinboers commented 3 years ago

The background to this issue is that the plcncli includes some project templates. The first version of plcncli included one template called "project", then the template "acfproject" was added later, and now the template "consumablelibrary" has also been included with the latest version.

Most of the project examples in this repository were created with earlier versions of the "project" or "acfproject" templates. These templates have evolved over time. Projects that were created with the "old" templates still build OK, but they do not have the same structure as projects that are created with later versions of these templates.

So, to answer your question: The latest version of the plcncli always includes the latest best-practice project templates.

OWarneke commented 3 years ago

Projects to update:

https://github.com/PLCnext/CppExamples/tree/update-TemplateTolatest

martinboers commented 3 years ago

I have now pushed an update to that development branch, which updates the code in the ThreadExample folder (which I think checks the third box in @OWarneke's list). I notice that NotificationExample also needs to be updated - I will work on that next week.