Marlin: [BUG] (Unable to build SKR_Pro with current Marlin download)

Created on 26 Oct 2019  Â·  37Comments  Â·  Source: MarlinFirmware/Marlin

Description


build fails in current version posted for SKR_Pro_v1_1

Steps to Reproduce

  1. Download and unzip Marlin bugfix -2.0.x
  2. Open in PlatformIO
  3. Change in configuration.h

    define SERIAL_PORT -1

define MOTHERBOARD BOARD_BIGTREE_SKR_PRO_V1_1

  1. Change in platforio.ini
    \default_envs = BIGTREE_SKR_PRO
  2. Save and compile.

Expected behavior: [What you expect to happen]
Builds without issue

Actual behavior: [What actually happens]
Builds and fails in multiple places.

Additional Information

Previous downloads of Marlin with framework 5.4.3 builds without issue

  • Include a ZIP file containing your Configuration.h and Configuration_adv.h files.
  • Provide pictures or links to videos that clearly demonstrate the issue.
  • See How Can I Contribute for additional guidelines.
Confirmed !

All 37 comments

Not sure I understand what you mean by using BTT repository , this originally worked a couple of weeks ago. Does BTT link there repository to Marlins?

I think it relates to using the BigTreeTech repo, who forked Marlin - but it is not the official Marlin repo anymore.
I would love to use the official Marlin codebase, especially since every new development and change won't be available necessarily in their repo. And according to ebraiman's youtube videos this worked not too long ago!

Use the BTT repository.

Personally i think this is not called for, we dont support forks so we dont ask people to go elsewhere

Previous downloads of Marlin with framework 5.4.3 builds without issue

  • Include a ZIP file containing your Configuration.h and Configuration_adv.h files.
  • Provide pictures or links to videos that clearly demonstrate the issue.
  • See How Can I Contribute for additional guidelines.

what is missing in the first post?

The download of Marlin firmware was from www.marlinfw.org. Not sure I'm understanding your answer as I've been using for months. A new download yesterday stopped working. I'm thinking new SW check-in changed compiling.

  • Include a ZIP file containing your Configuration.h and Configuration_adv.h files.

hint hint ^^^

the config is wrong it for mega+ramps board and not SKR Pro

define MOTHERBOARD BOARD_RAMPS_14_EFB

Here are the files

platformio.zip
Configuration.zip

i removed the first zip file as we dont need the complete marlin, just the config files

so the changes i see in config are simple

configuration.h:

define SERIAL_PORT 0

changes to

define SERIAL_PORT -1

define MOTHERBOARD BOARD_RAMPS_14_EFB

changes to

define MOTHERBOARD BOARD_BIGTREE_SKR_PRO_V1_1

platform.io is that also from marlin? there should only be changes to default_envs

This might make it easier to follow

https://youtu.be/VsKHwYtqLz0

Just follow video with www.marlinfw.org download from today

Does anyone know if bug fixes have some type of svn like number for tracking in downloads?

was the changes i discovered correct?

Yes. except platformio.ini which was SKR_Pro specified as normal. But the video will fill in the blanks there.

I didn't want to post the video, but seeing how I've done about one a week and do not know when the change occurred in marlinfw at least you have a start and end date.

if the platform.io comes from outside source we cant use it

but error is the same even if we use platform.io from marlin

compile error: https://pastebin.com/Y01k60KD

Configuration&Platformio.zip

Thank you.

Is it okay to design a build script to confirm broken builds for each board type in python?

yes if you can,

i think the problem is that for each of the example config there really should be an platform.io ini file
@shitcreek @thinkyhead

Well depends on my time and a public server to run it on.

there is an automatic build system testing things called travis
i think only @shitcreek and @thinkyhead can add boards and configs to that
but it would be a good way to test configs

I have the same problem after updating the STSTM32 platform, the bug is discussed on the STSTM32 repo: https://github.com/platformio/platform-ststm32/issues/305.

I was able to compile by modifying the PlatformIO.ini file to force the use of the previous version of STSTM32:

platform = [email protected]

platform = [email protected]

This is actually already part of the current download of the Marlin 2.0 commit available on github. Still there are many errors mostly related to software-serial and compilation fails.

So the current build of Marlin doesn't work with the SKR Pro?

Marlin 2.0 works with SKR Pro, it can be compiled (I did) using the modification above, by @BastR :
"I was able to compile by modifying the PlatformIO.ini file to force the use of the previous version of STSTM32:
platform = [email protected]"

Related issues: #15655, here
https://github.com/MarlinFirmware/Marlin/pull/15655

platform = [email protected]

This is actually already part of the current download of the Marlin 2.0 commit available on github. Still there are many errors mostly related to software-serial and compilation fails.

Yes because @thinkyhead committed yesterday : https://github.com/MarlinFirmware/Marlin/commit/e0f68cda1e0ffe4a969c34efb1d4dff893ce67b2

I tried to compile with today's build and I also have issues but with the TMCStepper library:

In file included from Marlin/src/module/stepper/trinamic.h:29:0,
from Marlin/src/module/stepper/indirection.h:44,
from Marlin/src/module/stepper.h:47,
from Marlin/src/Marlin.cpp:37:
/Users/bastien/.platformio/lib/TMCStepper_ID5513/src/TMCStepper.h:26:11: fatal error: SoftwareSerial.h: No such file or directory

#include
^~~~~~
compilation terminated.
* [.pio/build/BIGTREE_SKR_PRO/src/src/Marlin.cpp.o] Error 1

I'm not sure what to do with SoftwareSerial.h, but by forcing the version of the TMCstepper library to the previous version too, I can compile.

Change TMCStepper@>=0.5.0,<1.0.0 by [email protected], @teemuatlut

I havent tried this yet, but Bigtree says to use their version of TMCStepper: https://github.com/bigtreetech/BIGTREETECH-SKR-PRO-V1.1/tree/master/firmware

Here is my most up to date for an FT-6. BL touch enabled, EEPROM enabled in configuration and flash emulation turned on in pins. Pulling TMCStepper from Bigtree in platformio.

Errors out with Conditionals_LCD.h

Configs.zip

From this morning try with https://github.com/MarlinFirmware/Marlin/pull/15655 it builds okay on Travis with default config. you will need to define a different timer on platformio.ini or it will hang on boot. we didn't define which one however.

@ebraiman @DroneMang

tried what @LinoBarreca suggested?

@boelle, at this point it probably doesn't make sense for people to use the changes in the PR @LinoBarreca is working on, unless they are actively involved in debugging issues with the STM32 5.7 upgrade.

I am about to post a PR which should alleviate a lot of these SKR Pro / SoftwareSerial issues temporarily, until the move to 5.7+ is made. It unfortunately has to point to my own fork of a SoftwareSerial library, but that will hopefully be short-lived.

This appears to be fixed with a download from Marlinfw.org with change(s) to MB for SKR Pro v1.1 and platformio.ini with serial port -1.

Fixed

I still have similar problem. unable to build latest bugfix version of Marlin.

@yavuzongul —
This Issue Queue is for Marlin bug reports and development-related issues, and we prefer not to handle user-support questions here. (As noted on this page.) For best results getting help with configuration and troubleshooting, please use the following resources:

After seeking help from the community, if the consensus points to a bug in Marlin, then you should post a bug report.

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.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

otisczech picture otisczech  Â·  3Comments

heming3501 picture heming3501  Â·  4Comments

W8KDB picture W8KDB  Â·  4Comments

ahsnuet09 picture ahsnuet09  Â·  3Comments

Kaibob2 picture Kaibob2  Â·  4Comments