Skip to content

add additional obsolete files to preflight test. - #27990

Merged
thinkyhead merged 1 commit into
MarlinFirmware:bugfix-2.1.xfrom
ellensp:update-old-files-preflight-check
Aug 8, 2025
Merged

add additional obsolete files to preflight test. #27990
thinkyhead merged 1 commit into
MarlinFirmware:bugfix-2.1.xfrom
ellensp:update-old-files-preflight-check

Conversation

@ellensp

@ellensp ellensp commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

Description

A user decided to extract bugfix 2.1.x over Marlin 2.1.2.5 creating many code conflicts.

These are part of bugfix 2.1.x (note the u8g subdirectory)

\src\lcd\dogm\u8g\u8g_dev_ssd1306_sh1106_128x64_I2C.cpp
\src\lcd\dogm\u8g\u8g_dev_ssd1306_sh1106_128x64_SWSPI.cpp
\src\lcd\dogm\u8g\u8g_dev_ssd1309_12864.cpp
\src\lcd\dogm\u8g\u8g_dev_st7565_64128n_HAL.cpp
\src\lcd\dogm\u8g\u8g_dev_st7920_128x64_HAL.cpp
\src\lcd\dogm\u8g\u8g_dev_tft_upscale_from_128x64.cpp
\src\lcd\dogm\u8g\u8g_dev_uc1701_mini12864_HAL.cpp
\src\lcd\dogm\u8g\ultralcd_st7920_u8glib_rrd_AVR.cpp

These are part of 2.1.2.5
\src\lcd\dogm\u8g_dev_ssd1306_sh1106_128x64_I2C.cpp
\src\lcd\dogm\u8g_dev_ssd1309_12864.cpp
\src\lcd\dogm\u8g_dev_st7565_64128n_HAL.cpp
\src\lcd\dogm\u8g_dev_st7920_128x64_HAL.cpp
\src\lcd\dogm\u8g_dev_tft_upscale_from_128x64.cpp
\src\lcd\dogm\u8g_dev_uc1701_mini12864_HAL.cpp
\src\lcd\dogm\ultralcd_st7920_u8glib_rrd_AVR.cpp

the 2.1.2.5 files should not be part of bugfix

Added the 2.1.2.5 files to the existing preflight-checks.py to make sure the user deletes them from newer versions of Marlin.

Benefits

Code has a chance of building

Related Issues

  • [BUG] 2.1.2.5 With Bugfixes, Bugfix 2.1.x, Stock cr10 Display wont compile #27989
  • @thinkyhead
    thinkyhead merged commit 1244a21 into MarlinFirmware:bugfix-2.1.x Aug 8, 2025
    130 of 131 checks passed
    @ellensp
    ellensp deleted the update-old-files-preflight-check branch August 9, 2025 03:41
    bsdnomad pushed a commit to bsdnomad/Marlin that referenced this pull request Sep 25, 2025
    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