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.

Sign up for GitHub

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

Jump to bottom

Fix NUCLEO_F767ZI build and add tests #19746

Merged
merged 4 commits into from
Oct 16, 2020
Merged

Fix NUCLEO_F767ZI build and add tests #19746

merged 4 commits into from
Oct 16, 2020

Conversation

sjasonsmith
Copy link
Contributor

Description

Recent changes to dependency detection broke the NUCLEO_F767ZI build. Fix it and enable tests for it, since it is the only currently STM32F7 board using the STM32 HAL.

This is the result of my new test prior to fixing the pins file:

[Test NUCLEO_F767ZI] Mixed timer usage...
In file included from buildroot/share/PlatformIO/scripts/../../../../Marlin/src/inc/../pins/pins.h:638,
                 from buildroot/share/PlatformIO/scripts/../../../../Marlin/src/inc/MarlinConfig.h:34,
                 from buildroot/share/PlatformIO/scripts/common-dependencies.h:31:
buildroot/share/PlatformIO/scripts/../../../../Marlin/src/inc/../pins/stm32f7/pins_NUCLEO_F767ZI.h:25:4: error: #error "Oops! Select an STM32F767 environment"

Benefits

Builds.

Configurations

These exactly match the test I added.
Marlin.zip

Related Issues

N/A

@thinkyhead thinkyhead merged commit 7f83231 into MarlinFirmware:bugfix-2.0.x Oct 16, 2020
Zorchz pushed a commit to Zorchz/Marlin-1 that referenced this pull request Oct 17, 2020
@sjasonsmith
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
5b6d1a3
Zorchz pushed a commit to Zorchz/Marlin-1 that referenced this pull request Oct 17, 2020
@sjasonsmith
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
6289c89
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Oct 21, 2020
@sjasonsmith @thinkyhead
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
3696fb6
Speaka pushed a commit to Speaka/Marlin that referenced this pull request Oct 23, 2020
@sjasonsmith @Speaka
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
c288257
Speaka pushed a commit to Speaka/Marlin that referenced this pull request Nov 2, 2020
@sjasonsmith @Speaka
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
495ed9e
@sjasonsmith sjasonsmith deleted the PR/Nucleo_fix branch November 23, 2020 09:32
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
@sjasonsmith @vgadreau
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
b384eab
tharts pushed a commit to tharts/Marlin that referenced this pull request Jan 6, 2021
@sjasonsmith @tharts
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
575b42c
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
@sjasonsmith @kpishere
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
c78412b
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
@sjasonsmith @W4tel-BiDi
Fix NUCLEO_F767ZI build, add tests (MarlinFirmware#19746)
90b7b9f
thinkyhead pushed a commit that referenced this pull request Apr 30, 2021
@sjasonsmith @thinkyhead
Fix NUCLEO_F767ZI build, add tests (#19746)
386e643
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
A: STM32 C: Build / Toolchain
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sjasonsmith @thinkyhead