rtai: fix includes for build to succeed#3894
Conversation
|
Thank you very much for this! |
|
@BsAtHome Can you also fix an issue of 4e080ac this PR? After having some confusion why master builds with LXRT real time and 2.9 fails, I discovered that the above commit just disables LXRT accidentally: NTULINUX/RTAI#20 Line 203 should be changed to: The reason behind: ^In that case, RTAI_CONFIG should be autodetected for uspace build. RTAI kernel RT: or This works fine. |
|
I'm sorry, but line 203 does not contain what you say it should. Do you means line 192, as in: diff --git a/src/configure.ac b/src/configure.ac
index 917c1f69d5..ed1ad16cd5 100644
--- a/src/configure.ac
+++ b/src/configure.ac
@@ -189,7 +189,7 @@ AC_ARG_ENABLE(toolnml,
#at this point if RTS is empty, we need to find RT ourselves
AC_MSG_CHECKING([for rtai-config])
-if test -z "$RTS"; then
+if test -z "$RTS" || test "$RTS" = uspace; then
DIRS="/usr/realtime-`uname -r` /usr/realtime /usr/realtime* /usr /usr/src/rtai*"
else
DIRS="$RTS" |
|
Sorry, these lines always move, i referred your old commit, in master it's 192. ^^looks fine, thanks! Full patch, including indentation fix, feel free to apply it on your MR: diff --git a/src/configure.ac b/src/configure.ac
index 917c1f69d5..ba5c0877b6 100644
--- a/src/configure.ac
+++ b/src/configure.ac
@@ -189,14 +189,15 @@ AC_ARG_ENABLE(toolnml,
#at this point if RTS is empty, we need to find RT ourselves
AC_MSG_CHECKING([for rtai-config])
-if test -z "$RTS"; then
+if test -z "$RTS" || test "$RTS" = uspace; then
DIRS="/usr/realtime-`uname -r` /usr/realtime /usr/realtime* /usr /usr/src/rtai*"
else
DIRS="$RTS"
fi
- RTAI_CONFIG=none
- for location in $DIRS; do
- for subdir in . scripts bin; do
+
+RTAI_CONFIG=none
+for location in $DIRS; do
+ for subdir in . scripts bin; do
for signature in rtai; do
candidate=$location/$subdir/$signature-config
if test "$RTAI_CONFIG" = "none" -a -x $candidate; then |
1a68182 to
3a919d1
Compare
|
Patch added and force pushed. |
|
Good to go everyone? |
|
Good to go but this still needs: |
Yes, but they are two separate issues. |
|
Good from my side. First merge that one so #3884 can be finally tested and also merged. |
This PR addresses some of the build problems discovered in #3873 to make RTAI compile again.