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

[WIP] STM32 F1 Cube Support #13

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

BBScholar
Copy link

@BBScholar BBScholar commented Oct 29, 2020

Added stm32f1cube bazel files. Planning to add an example as well.

@nathaniel-brough
Copy link
Collaborator

Thank you for this! Reviewing now

Copy link
Collaborator

@nathaniel-brough nathaniel-brough left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good, if you can just fix up the sha256, that would be fantastic. I'm going to add all of the external repository stuff to the CI at some point as well. For now I am just going to do this manually.

I will wait until you have completed an example before merging.

VERSIONS = {
"1.8.2" : {
"commit" : "441b2cbdc25aa50437a59c4bffe22b88e78942c9",
"sha256" : ""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a complete sha256 just to confirm determinism. If you leave this empty it should print out the sha256 when it fetches the remote reposistory.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it. Will finish up the example this weekend.

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.

2 participants