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

Better build system #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

guglie
Copy link

@guglie guglie commented May 25, 2020

  • Build fixed on newer versions of platformio
  • Version numbers are set in platformio.ini
  • Firmware file is renamed mavesp-{board}-{version} e.g. mavesp-esp12e-1.2.2.bin

rmackay9 and others added 4 commits May 25, 2020 10:34
'F' ESP8266 macro (defined in cores/esp8266/WString.h) is undefined in  mavesp8266.h to prevent conflict with lib/mavlink/mavlink_sha256.h.
It is needed in ESP8266WebServer-impl.h that gets included from ESP8266WebServer.h.
Reordering those fixes the build, the proper fix would be to rename 'F' in MAVLink subproject.
@guglie guglie force-pushed the pr-better-build-sys branch from 702b10e to c558c76 Compare May 25, 2020 10:58
…e with version and platform

+ ignore another platformio dir
@guglie guglie force-pushed the pr-better-build-sys branch from c558c76 to f62afa6 Compare May 25, 2020 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants