search

TrampolineRTOS / trampoline

Trampoline is a static RTOS for small embedded systems. Its API is aligned with OSEK/VDX OS and AUTOSAR OS 4.2 standards.
GNU General Public License v2.0
599 stars 263 forks source link

ci: Added POSIX examples build. #129

Closed RICCIARDI-Adrien closed 9 months ago

RICCIARDI-Adrien commented 1 year ago

This PR adds all POSIX examples to the CI and also allows to fix some warnings discovered by the CI builds.

RICCIARDI-Adrien commented 10 months ago

Hi @jlbirccyn, this PR is now ready for review (if you have time to).

RICCIARDI-Adrien commented 9 months ago

Hi @jlbirccyn, what do you think about this PR ?

RICCIARDI-Adrien commented 9 months ago

Thanks for the feedback, I will change it. So it's always make.py in all Trampoline, that's correct ?

jlbirccyn commented 9 months ago

So it's always make.py in all Trampoline, that's correct ?

Yes, to build an OSEK/AUTOSAR application, but to build goil, it's build.py

make.py calls build.py

When you run goil for the first time, make.py and build.py are generated. make.py manages dependencies on the oil file. If you modify the oil file, make.py runs goil and goil generates a new build.py file which is then called by make.py to manage the rest of dependencies. This is why there is 2 scripts.

RICCIARDI-Adrien commented 9 months ago

Got it, thank you very much for the explanations ! I will put the proper make.py in the future documentation for new applications. I updated this PR accordingly.

jlbirccyn commented 9 months ago

It is merged !

Oups, except the .yaml files uses build.py instead of make.py. Could you do an update and a new PR to fix that ?

RICCIARDI-Adrien commented 9 months ago

Nice catch ! Here is the PR, I replaced build.py by make.py for all AUTOSAR applications. https://github.com/TrampolineRTOS/trampoline/pull/136