Skip to content

Grbl over tcp fluidnc#69

Merged
Schildkroet merged 2 commits intoSchildkroet:masterfrom
PSIAlt:grbl_over_tcp_fluidnc
Mar 26, 2026
Merged

Grbl over tcp fluidnc#69
Schildkroet merged 2 commits intoSchildkroet:masterfrom
PSIAlt:grbl_over_tcp_fluidnc

Conversation

@PSIAlt
Copy link
Copy Markdown
Contributor

@PSIAlt PSIAlt commented Dec 21, 2025

I was struggling to connect FluidNC over WIFI because GrIP is different protocol than a plain GRBL sent over TCP (which is used by FluidNC).
So there's the fix, now we support both, and make clarifications in buttons in protocol select box.

@Schildkroet
Copy link
Copy Markdown
Owner

Hello,

There is an error with heightmap:

../src/frmmain_heightmap.cpp: At global scope:
../src/frmmain_heightmap.cpp:104:6: error: redefinition of ‘bool frmMain::hasHeightmapData()’
  104 | bool frmMain::hasHeightmapData()
      |      ^~~~~~~
../src/frmmain_heightmap.cpp:60:6: note: ‘bool frmMain::hasHeightmapData()’ previously defined here
   60 | bool frmMain::hasHeightmapData()
      |      ^~~~~~~
../src/frmmain_heightmap.cpp:115:35: error: redefinition of ‘frmMain::HeightmapPreflightResult frmMain::maybePromptEnableHeightmapBeforeRun()’
  115 | frmMain::HeightmapPreflightResult frmMain::maybePromptEnableHeightmapBeforeRun()
      |                                   ^~~~~~~
../src/frmmain_heightmap.cpp:71:35: note: ‘frmMain::HeightmapPreflightResult frmMain::maybePromptEnableHeightmapBeforeRun()’ previously defined here
   71 | frmMain::HeightmapPreflightResult frmMain::maybePromptEnableHeightmapBeforeRun()
      |                                   ^~~~~~~
../src/frmmain_heightmap.cpp:115:10: error: ‘enum frmMain::HeightmapPreflightResult’ is private within this context
  115 | frmMain::HeightmapPreflightResult frmMain::maybePromptEnableHeightmapBeforeRun()
      |          ^~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../src/frmmain_heightmap.cpp:21:
../src/frmmain.h:323:10: note: declared private here
  323 |     enum HeightmapPreflightResult { HmProceedRun, HmEnableButDoNotRun, HmCancel };

PSIAlt added 2 commits March 16, 2026 18:15
Not the same as GrIP (which had misleading name in protocol select box)
For FluidNC tcp connection use GRBL+Ethernet selection
@PSIAlt PSIAlt force-pushed the grbl_over_tcp_fluidnc branch from d9b9100 to 1e3be1e Compare March 16, 2026 18:15
@Schildkroet Schildkroet merged commit ffc009a into Schildkroet:master Mar 26, 2026
2 of 3 checks passed
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