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

[BUG] Creality Ender 5 Plus configs won't compile on Bugfix #21663

Closed
D337z opened this issue Apr 19, 2021 · 2 comments
Closed

[BUG] Creality Ender 5 Plus configs won't compile on Bugfix #21663

D337z opened this issue Apr 19, 2021 · 2 comments

Comments

@D337z
Copy link

D337z commented Apr 19, 2021

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

Could not get code to compile using modified configs, so I attempted to use the example config specifically for my printer. Come to find out, it wasn't the config that seemed to be the issue. This is using the latest bugfix Marlin and config files in PlatformIO without any changes to the example config files. From what I'm told, it is supposed to download any required libs automatically. If not, perhaps adding instruction to the error would be helpful.

Compiling .pio\build\mega2560\src\src\libs\numtostr.cpp.o
Marlin\src\lcd\extui\lib\dgus\origin\DGUSDisplayDef.cpp:310:1: error: no matches converting function 'GetVariable' to type 'void (*)(struct DGUS_VP_Variable&, void*)'
 };
 ^
In file included from Marlin\src\lcd\extui\lib\dgus\origin\DGUSDisplayDef.cpp:32:0:
Marlin\src\lcd\extui\lib\dgus\origin\../DGUSDisplay.h:76:15: note: candidate is: template<class T, void (* Setter)(float, T), T selector> static void DGUSDisplay::GetVariable(DGUS_VP_Variable&, void*)
   static void GetVariable(DGUS_VP_Variable &var, void *val_ptr) {
               ^~~~~~~~~~~
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp: In static member function 'static void DGUSScreenHandler::DGUSLCD_SD_FileSelected(DGUS_VP_Variable&, void*)':
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:47:22: error: 'filelist' was not declared in this scope
     if (touched_nr > filelist.count()) return;
                      ^~~~~~~~
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:47:22: note: suggested alternative: 'filepos_t'
     if (touched_nr > filelist.count()) return;
                      ^~~~~~~~
                      filepos_t
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:48:10: error: 'filelist' was not declared in this scope
     if (!filelist.seek(touched_nr)) return;
          ^~~~~~~~
*** [.pio\build\mega2560\src\src\lcd\extui\lib\dgus\origin\DGUSDisplayDef.cpp.o] Error 1
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:48:10: note: suggested alternative: 'filepos_t'
     if (!filelist.seek(touched_nr)) return;
          ^~~~~~~~
          filepos_t
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:50:9: error: 'filelist' was not declared in this scope
     if (filelist.isDir()) {
         ^~~~~~~~
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:50:9: note: suggested alternative: 'filepos_t'
     if (filelist.isDir()) {
         ^~~~~~~~
         filepos_t
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:65:87: error: 'filelist' was not declared in this scope
     HandleUserConfirmationPopUp(VP_SD_FileSelectConfirm, nullptr, PSTR("Print file"), filelist.filename(), PSTR("from SD Card?"), true, true, false, true);
                                                                                       ^~~~~~~~
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:65:87: note: suggested alternative: 'filepos_t'
     HandleUserConfirmationPopUp(VP_SD_FileSelectConfirm, nullptr, PSTR("Print file"), filelist.filename(), PSTR("from SD Card?"), true, true, false, true);
                                                                                       ^~~~~~~~
                                                                                       filepos_t
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp: In static member function 'static void DGUSScreenHandler::DGUSLCD_SD_StartPrint(DGUS_VP_Variable&, void*)':
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:69:10: error: 'filelist' was not declared in this scope
     if (!filelist.seek(file_to_print)) return;
          ^~~~~~~~
Marlin\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp:69:10: note: suggested alternative: 'filepos_t'
     if (!filelist.seek(file_to_print)) return;
          ^~~~~~~~
          filepos_t
compilation terminated due to -fmax-errors=5.
*** [.pio\build\mega2560\src\src\lcd\extui\lib\dgus\origin\DGUSScreenHandler.cpp.o] Error 1

Bug Timeline

No response

Expected behavior

Expected a working default compilation for an Ender 5 Plus.

Actual behavior

Compilation failure due to errors in DGUS.

Steps to Reproduce

Download Marlin nightly build and accompanying configuration files.
Add configuration file for Creality Ender 5 Plus to the Marlin directory.
Load up PlatformIO
Open Marlin nightly build folder
Attempt compilation
Observe failure

Version of Marlin Firmware

1.70.2 BL

Printer model

Creality Ender 5 Plus

Electronics

Stock board, Microswiss direct drive extruder, soon to be installed 12864 LCD display board

Add-ons

Nuthin

Your Slicer

Cura

Host Software

OctoPrint

@ellensp
Copy link
Contributor

ellensp commented Apr 19, 2021

Duplicate of #21576

@ellensp ellensp marked this as a duplicate of #21576 Apr 19, 2021
@ellensp ellensp closed this as completed Apr 19, 2021
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Jun 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants