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

vxi11/osiRpc.h: Only RTEMS with RTEMS_LEGACY_STACK needs rpcTaskInit #153

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

Conversation

tboegi
Copy link
Contributor

@tboegi tboegi commented Apr 26, 2022

Reported by Michael Davidsaver:
With RTEMS 5 for pc686 with the libbsd (aka. "new") network stack I get:
In file included from ../../asyn/vxi11/drvVxi11.c:41:0:
../../asyn/vxi11/drvVxi11.c: In function 'vxiConnectPort':
../../asyn/vxi11/osiRpc.h:25:21: error: implicit declaration of function 'rtems_rpc_task_init'; did you mean 'rtems_task_exit'? [-Werror=implicit-function-declaration]
#define rpcTaskInit rtems_rpc_task_init
^
../../asyn/vxi11/drvVxi11.c:908:12: note: in expansion of macro 'rpcTaskInit'
if(rpcTaskInit() == -1) {

================
Solution:
Use RTEMS_LEGACY_STACK to switch between the "old" and "new"
networkstack for RTEMS

@AppVeyorBot
Copy link

Build asyn 1.0.112 failed (commit 72f1d6a125 by @tboegi)

Reported by Michael Davidsaver:
With RTEMS 5 for pc686 with the libbsd (aka. "new") network stack I get:
In file included from ../../asyn/vxi11/drvVxi11.c:41:0:
../../asyn/vxi11/drvVxi11.c: In function 'vxiConnectPort':
../../asyn/vxi11/osiRpc.h:25:21: error: implicit declaration of function 'rtems_rpc_task_init'; did you mean 'rtems_task_exit'? [-Werror=implicit-function-declaration]
 #define rpcTaskInit rtems_rpc_task_init
                     ^
../../asyn/vxi11/drvVxi11.c:908:12: note: in expansion of macro 'rpcTaskInit'
         if(rpcTaskInit() == -1) {

================
Solution:
Use RTEMS_LEGACY_STACK to switch between the "old" and "new"
networkstack for RTEMS
@tboegi tboegi force-pushed the rtems-rpcTaskInit branch from 123ce86 to a95ae9a Compare April 27, 2022 06:24
@AppVeyorBot
Copy link

Build asyn 1.0.114 failed (commit ce19bd7379 by @tboegi)

@AppVeyorBot
Copy link

Build asyn 1.0.234 completed (commit 47dbae3ad2 by @tboegi)

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