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.

Sign up for GitHub

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

Jump to bottom

emcrsh: fix bug with extra reply line for 'get spindle_override' #3182

Merged

andypugh merged 1 commit into LinuxCNC:master from Sigma1912:emcrsh_fix-reply-for-get_spindle_override

Jan 5, 2025

Contributor

Sigma1912 commented Nov 16, 2024 •

edited

Loading

currently 'get spindle_override' command in linuxcncrsh will return an extra line containing the reply from the 'get' command before last:

get teleop_enable
TELEOP_ENABLE OFF
get spindle_override
SPINDLE_OVERRIDE 0 49.000000
TELEOP_ENABLE OFF

I'm not sure why that happens but using 'OUT' instead of 'dprintf' seems to fix this issue.


          emcrsh: fix bug with extra reply line for 'get spindle_override'

d85194b

Sigma1912 mentioned this pull request

Linuxcncrsh segmentation faults on get/set spindle commands. #3180

Open

3 tasks

Contributor Author

Sigma1912 commented Nov 29, 2024

Ready, as far as I'm concerned

andypugh merged commit 927bd3c into LinuxCNC:master

9 checks passed

Sigma1912 deleted the emcrsh_fix-reply-for-get_spindle_override branch

January 6, 2025 07:55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet