Skip to content

Comments

Fix function arguments to match euslisp update (euslisp/EusLisp@e89411b)#758

Open
nakane11 wants to merge 1 commit intojsk-ros-pkg:masterfrom
nakane11:pointer
Open

Fix function arguments to match euslisp update (euslisp/EusLisp@e89411b)#758
nakane11 wants to merge 1 commit intojsk-ros-pkg:masterfrom
nakane11:pointer

Conversation

@nakane11
Copy link
Member

I encountered a build failure when compiling the roseus package against a version of euslisp built from source following README.

/home/leus/ros/irt/src/jsk-ros-pkg/jsk_roseus/roseus/roseus.cpp: In function ‘void register_roseus()’:
/home/leus/ros/irt/src/jsk-ros-pkg/jsk_roseus/roseus/roseus.cpp:95:44: error: invalid conversion from ‘cell* (*)()’ to ‘cell* (*)(context*, int, cell**)’ [-fpermissive]
   95 |     return add_module_initializer(modname, (pointer (*)())___roseus);}
      |                                            ^~~~~~~~~~~~~~~~~~~~~~~~
      |                                            |
      |                                            cell* (*)()

The cause appears to be related to upstream changes made to lisp/c/eus_proto.h in commit euslisp/EusLisp@e89411b.

This pull request should resolve the issue.

@nakane11 nakane11 requested a review from k-okada June 17, 2025 16:34
@mqcmd196
Copy link
Member

@nakane11 It seems that euslisp/EusLisp@e89411b breaks userspace so I think it's a good idea to send the patch to the upstream to keep the original header definition.

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