Skip to content

Comments

Fix printing hm2_modbus messages at startup#3824

Open
BsAtHome wants to merge 1 commit intoLinuxCNC:masterfrom
BsAtHome:fix_hm2_modbus-messages
Open

Fix printing hm2_modbus messages at startup#3824
BsAtHome wants to merge 1 commit intoLinuxCNC:masterfrom
BsAtHome:fix_hm2_modbus-messages

Conversation

@BsAtHome
Copy link
Contributor

Fix printing errors/warnings/messages at startup by using rtapi_print() instead of rtapi_print_msg(). The latter does not actually output anything in the rtapi_app_main() function.
This fixes a problem mentioned in #3822 of not finding the PktUART port and hm2_modus not telling the user of the issue.

@smoe smoe added the 2.10-candidate would be nice to have fixed in 2.10 label Feb 23, 2026
@smoe
Copy link
Collaborator

smoe commented Feb 23, 2026

Should this also go into 2.9?

@BsAtHome
Copy link
Contributor Author

Should this also go into 2.9?

Probably. But I can't test-run 2.9 branch because I haven't backported the tcl/tk9 stuff. That said, the same patch should work for both branches.

@andypugh
Copy link
Collaborator

I would guess that if it passes CI as a 2.9 fix it's probably good?

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

Labels

2.10-candidate would be nice to have fixed in 2.10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants