Skip to content

🔥 Remove DISKIO_HOST_DRIVE#1035

Merged
thinkyhead merged 2 commits intoMarlinFirmware:import-2.1.xfrom
thisiskeithb:pr/remove_DISKIO_HOST_DRIVE
May 12, 2024
Merged

🔥 Remove DISKIO_HOST_DRIVE#1035
thinkyhead merged 2 commits intoMarlinFirmware:import-2.1.xfrom
thisiskeithb:pr/remove_DISKIO_HOST_DRIVE

Conversation

@thisiskeithb
Copy link
Copy Markdown
Contributor

Description

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

Benefits

Removes dead config option

Related Issues

@thisiskeithb
Copy link
Copy Markdown
Contributor 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 thisiskeithb force-pushed the pr/remove_DISKIO_HOST_DRIVE branch from b40ca83 to f249c59 Compare April 1, 2024 02:38
@thisiskeithb
Copy link
Copy Markdown
Contributor 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 thisiskeithb force-pushed the pr/remove_DISKIO_HOST_DRIVE branch from f249c59 to 2b9b45b Compare May 2, 2024 00:13
@thisiskeithb
Copy link
Copy Markdown
Contributor Author

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

@thisiskeithb
Copy link
Copy Markdown
Contributor 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
@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