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

🔥 Remove DISKIO_HOST_DRIVE #1035

Conversation

thisiskeithb
Copy link
Member

Description

Since MarlinFirmware/Marlin#19939 was closed & never merged, remove DISKIO_HOST_DRIVE option.

Benefits

Removes dead config option

Related Issues

@thisiskeithb
Copy link
Member Author

thisiskeithb commented Mar 28, 2024

CI will fail due to a handful of Geeetech configs that still won't build correctly due to some LCD/library incompatibilities after MarlinFirmware/Marlin@ec7ab5a.

@thinkyhead thinkyhead force-pushed the import-2.1.x branch 2 times, most recently from c5e57b7 to 7eb7090 Compare April 1, 2024 02:16
@thisiskeithb
Copy link
Member Author

thisiskeithb commented Apr 1, 2024

...PR fixed after recent force-pushes to import-2.1.x, but my comment above is still applicable.

@thisiskeithb
Copy link
Member Author

Rebased on current import-2.1.x and I've removed this code from the recently added configs.

@thisiskeithb
Copy link
Member Author

thisiskeithb commented May 8, 2024

CI will fail due to a handful of Geeetech configs that still won't build correctly due to some LCD/library incompatibilities after MarlinFirmware/Marlin@ec7ab5a.

... and apparently some other recent changes:

HEAD is now at f5cf667c95 🎨 Clarify some string parameters (#26949)
Building Configurations in config/examples/CNC/miniRambo ...
Building environment rambo for board MINIRAMBO (1201)...

In file included from Marlin/src/gcode/feature/digipot/../../../inc/MarlinConfigPre.h:37:0,
                 from Marlin/src/gcode/feature/digipot/../../../inc/MarlinConfig.h:28,
                 from Marlin/src/gcode/feature/digipot/M907-M910.cpp:23:
Marlin/src/gcode/feature/digipot/M907-M910.cpp: In static member function 'static void GcodeSuite::M907_report(bool)':
Marlin/src/gcode/feature/digipot/M907-M910.cpp:137:13: error: 'SP_E_STR' was not declared in this scope
           , SP_E_STR,       stepper.motor_current_setting[2]  // E
             ^

Edit: This CNC config was committed directly so it was not checked by CI: e7254f1

@thinkyhead thinkyhead force-pushed the pr/remove_DISKIO_HOST_DRIVE branch from 02c1b49 to 74dd24b Compare May 12, 2024 20:09
@thinkyhead thinkyhead merged commit 870f453 into MarlinFirmware:import-2.1.x May 12, 2024
1 check failed
@thisiskeithb thisiskeithb deleted the pr/remove_DISKIO_HOST_DRIVE branch May 12, 2024 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants