framework submakefile improvements #114

Merged
gaisser merged 2 commits from KSat/fsfw:mueller_Framework_mk_OS_FSFW into master 2020-07-28 12:37:29 +02:00
Owner

Fixes #115.

OS renamed to OS_FSFW because OS sometimes
is a system variable which can be used to determine the OS (windows, linux).

Fixes #115. OS renamed to OS_FSFW because OS sometimes is a system variable which can be used to determine the OS (windows, linux).
muellerr added the
feature
API Change
labels 2020-06-19 14:38:39 +02:00
gaisser self-assigned this 2020-06-22 21:35:07 +02:00
gaisser reviewed 2020-06-22 21:36:06 +02:00
framework.mk Outdated
@ -34,1 +35,4 @@
else ifeq ($(OS_FSFW),freeRTOS)
CXXSRC += $(wildcard $(FRAMEWORK_PATH)/osal/FreeRTOS/*.cpp)
else ifeq ($(OS_FSFW),host)
CXXSRC += $(wildcard $(FRAMEWORK_PATH)/osal/host/*.cpp)
Owner

This is intended to be used as user specific host?

This is intended to be used as user specific host?
Author
Owner

It's basically an OS implemented as portable as possible for host machines by using C++ features.

It's basically an OS implemented as portable as possible for host machines by using C++ features.
Author
Owner

By detecting the underlying OS, I can for example make thread settings (e.g. sched priority)
by using the native handle.

By detecting the underlying OS, I can for example make thread settings (e.g. sched priority) by using the native handle.
Owner

And that is part of another pull request?

And that is part of another pull request?
Author
Owner

I have not made a pull request yet, although I could do that. there are still some parts (like semaphores) which are not implemented yet because C++20 will feature them..

I have not made a pull request yet, although I could do that. there are still some parts (like semaphores) which are not implemented yet because C++20 will feature them..
gaisser reviewed 2020-07-21 12:06:12 +02:00
framework.mk Outdated
@ -8,11 +8,13 @@ CXXSRC += $(wildcard $(FRAMEWORK_PATH)/controller/*.cpp)
CXXSRC += $(wildcard $(FRAMEWORK_PATH)/coordinates/*.cpp)
CXXSRC += $(wildcard $(FRAMEWORK_PATH)/datalinklayer/*.cpp)
CXXSRC += $(wildcard $(FRAMEWORK_PATH)/datapool/*.cpp)
CXXSRC += $(wildcard $(FRAMEWORK_PATH)/datapoolglob/*.cpp)
Owner

Could you remove the global datapool includes?

Could you remove the global datapool includes?
Author
Owner

This would break a lot of exisiting code which still uses the global datapool

This would break a lot of exisiting code which still uses the global datapool
gaisser closed this pull request 2020-07-28 12:37:29 +02:00
muellerr deleted branch mueller_Framework_mk_OS_FSFW 2020-07-29 19:25:40 +02:00
Sign in to join this conversation.
No description provided.