Skip to content
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

Implement the FreeRTOS git repo as a submodule #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Implement the FreeRTOS git repo as a submodule #26

wants to merge 1 commit into from

Conversation

neilbalch
Copy link

I just realized that FreeRTOS now publishes their source to a GitHub repo... maybe we could make use of that as a submodule? 😄

This builds successfully on my local WSL instance.

@RasmusB
Copy link

RasmusB commented Sep 8, 2020

Even better, use the FreeRTOS-Kernel repo instead. That way you don't have do add all the demo folder and other cruft that you don't need for your own projects.

@neilbalch
Copy link
Author

@RasmusB That has a different folder structure from the downloadable zip file and the repo I've linked to... Am I missing something?

@RasmusB
Copy link

RasmusB commented Sep 9, 2020

@neilbalch

The repo you linked to contains absolutely everything related to FreeRTOS; for example demo projects for all supported platforms. If you just want to submodule FreeRTOS to use in your own project, you might as well just submodule FreeRTOS-Kernel, which contains everyting that is actually needed to run FreeRTOS and nothing else.

If you take a look at the repo you have included, it actually submodules FreeRTOS-Kernel in https://github.com/FreeRTOS/FreeRTOS/tree/master/FreeRTOS in this way.

@neilbalch
Copy link
Author

Ah, I'll look into that today.

@neilbalch
Copy link
Author

Huh, strangely, the ./rtos/FreeRTOS/Source folder is empty... wtf? Everything still compiles...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants