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

Added a way towards easy builds #157

Merged
merged 2 commits into from
Jan 6, 2021
Merged

Added a way towards easy builds #157

merged 2 commits into from
Jan 6, 2021

Conversation

rvben
Copy link
Contributor

@rvben rvben commented Jan 5, 2021

Hello!

I believe that it's very important, for the success of a project (and forgetting contributions), that compilation is as easy as possible,
to make it as painless as possible. With Docker, there's no worry about having the right SDK's or having the code folder in the right location, it just works.

I've added the Dockerfile and a simple Makefile to show how easy it can be. If you want, I can make a separate repository for this 'builder', but I think it's easier to maintain if the compilation environment is kept close to the code.

Let me know what you think :)

Have a great day!

Ruben

@rvben
Copy link
Contributor Author

rvben commented Jan 5, 2021

Noticed that there's already a Docker image that is used for CI, so adjusted the Makefile to use that one instead of my own Dockerfile. :) Added a PR for that Docker image in the respective repository with some improvements.

@atc1441
Copy link
Owner

atc1441 commented Jan 6, 2021

Hey thank you for the PR
Unfortunately i did not work with docker till now since i am mostly using windows for my projects.

Does this PR changes some other files or only adds the Dockerfile and Makefile in root on the repo ?

@benleb
Copy link

benleb commented Jan 6, 2021

This just adds another make target (build) to the Makefile which starts a docker container to build the firmware image 👍

Copy link

@benleb benleb left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@atc1441 atc1441 merged commit 6a4b5b5 into atc1441:master Jan 6, 2021
@atc1441
Copy link
Owner

atc1441 commented Jan 6, 2021

Perfect :)

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.

3 participants