Skip to content

[BUG] GD32 MFL cannot build without U8glib update #27775

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

Open
1 task done
bmourit opened this issue Mar 31, 2025 · 3 comments
Open
1 task done

[BUG] GD32 MFL cannot build without U8glib update #27775

bmourit opened this issue Mar 31, 2025 · 3 comments
Labels
Bug: Confirmed ! T: HAL & APIs Topic related to the HAL and internal APIs.
Milestone

Comments

@bmourit
Copy link
Contributor

bmourit commented Mar 31, 2025

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

Yes, and the problem still exists.

Bug Description

The initial support for GD32 that was merged had the pull request in the U8glib repo linked in, so I'm not sure if it was intentionally merged anyway without it - perhaps pending LCD changes? If so, this can be closed. If it was just accidentally missed, then that repo has the feature request and patch already requested. I'm only creating the bug report here since Marlin wont build without the changes.

Bug Timeline

No response

Expected behavior

Successful build.

Actual behavior

Errors with build due to the ARDUINO define and the library not excluding ARDUINO_ARCH_MFL in various files.

Steps to Reproduce

No response

Version of Marlin Firmware

bugfix-2.1.x

Printer model

Ender-3

Electronics

GD32

LCD/Controller

Stock

Other add-ons

No response

Bed Leveling

None

Your Slicer

None

Host Software

None

Don't forget to include

  • A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

The U8glib patch here: MarlinFirmware/U8glib-HAL#35

@thisiskeithb thisiskeithb added this to the Version 2.1.3 milestone Apr 1, 2025
@bmourit
Copy link
Contributor Author

bmourit commented Apr 2, 2025

Progress! I see this was merged, so now we'll just wait for the new package to be released. Awesome! And thank you!

EDIT:
The only thing preventing building is in features.ini
HAS_MARLINUI_U8GLIB = marlinfirmware/U8glib-HAL@0.5.4

While the code was merge, this still is using the last release to build. Just wanted to mention that in case it got missed. If this is known and intentional for now, just ignore this. 🙂 Otherwise bumping the release, for example to U8glib-HAL@0.5.5, will likely be required. This isn't something I can do or patch though.

@ellensp
Copy link
Contributor

ellensp commented Apr 12, 2025

If you change that to the following it uses latest code vs latest release
HAS_MARLINUI_U8GLIB = https://github.com/MarlinFirmware/U8glib-HAL/archive/refs/heads/master.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Confirmed ! T: HAL & APIs Topic related to the HAL and internal APIs.
Projects
None yet
Development

No branches or pull requests

3 participants