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

Fixed AVR build issues #1410

Merged
merged 1 commit into from
Jul 16, 2020
Merged

Conversation

iocapa
Copy link
Contributor

@iocapa iocapa commented Jul 16, 2020

Summary

  • Missing double_t type (on the compiler side)
  • Missing CONFIG_STACK_ALIGNMENT macro
  • Missing linker emulation flags (avr-ld fails at last step)

Impact

Build failed for the aforementioned platform

Testing

@xiaoxiang781216
Copy link
Contributor

@iocapa it's better to squash two patch into one.

@iocapa
Copy link
Contributor Author

iocapa commented Jul 16, 2020

@iocapa it's better to squash two patch into one.

Done.
Sorry, first timer here :)

@acassis
Copy link
Contributor

acassis commented Jul 16, 2020

I will apply, because is passed on all tests, except macOS tests that probably weren't prepared to compile AVR code.

@acassis acassis merged commit 954ee9d into apache:master Jul 16, 2020
@iocapa iocapa deleted the bugfix/avr_build_issues branch July 17, 2020 10:12
@btashton
Copy link
Contributor

@iocapa We now have AVR support in the CI system, thanks for pushing us in that direction. I had to make a few other changes to handle some warnings. Hopefully this will help keep the port in better shape. Please open bugs/pr on anything else you run into.

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