Change FBs of powerlink to be compatible with latest FORTE version

eclipse-4diac / 4diac-forte

Eclipse Public License 2.0

30 stars 32 forks source link

Change FBs of powerlink to be compatible with latest FORTE version #171

Closed gafert closed 2 months ago

gafert commented 5 months ago

This pull request fixes and adds features in the powerlink module fixing #169:

Removes the old FBs used by powerlink and adds new ones in the new structure
Fixes incorrectly named variables introduced in https://github.com/eclipse-4diac/4diac-forte/commit/3cd52d7c9d66c7923f6f1f2973bbec64b91f3d2a
Adds automatic download and compilation of openPOWERLINK and tinyXML in CMake with FetchContent
Adds powerlink in systemtests

Compile it with -DFORTE_ARCHITECTURE=Posix -DFORTE_COM_ETH=ON -DFORTE_MODULE_POWERLINK=ON

(Problem with OPC UA which was described here resolved in e87b059)

azoitl commented 5 months ago

@gafert in order that we can accept your contribution you need to link your Github account with your eclipse account. Use the same email address for your commits as the one used for your Eclipse account, and sign the Eclipse Contributor Agreement.

gafert commented 4 months ago

I tested it now for a few weeks and it seems to be stable.

@azoitl I saw your comment and I agree. For now I just wanted to make it work again.

azoitl commented 3 months ago

@gafert please ensure that all files have a correct copy right header. Especially the files in the FB directory don't have that. The header generated by 4diac IDE 4diac FORTE export is not suitable for this repo here.

gafert commented 3 months ago

@azoitl I updated to copyright. Does that work for you?

gafert commented 2 months ago

@azoitl I fixed the coding style according to the PDF. I also updated to the new FB style you just introduced. Please tell me if there is anything else or I have missed something :)