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

Add instruction for building traditionally on UNIX system #627

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

timkenhan
Copy link

No description provided.

@akira25
Copy link

akira25 commented Nov 22, 2024

I like this one. Should speed up development-cycles a lot by shorter build times.

@timkenhan
Copy link
Author

@akira25 Glad you are liking it!

Personally, I'm quite bothered the primary build instruction prompts us to use the Github codespace. Due to the large overlap between the tinkerer community and the open-source community, I think it would be more appropriate to feature a more primitive instruction like the one I provided. Meanwhile, the less tech-savvy crowd would be using the web-based flasher provided from the release page, just like they always have been.

I also think that Docker might be overkill for the job, but I understand that not everyone uses the same exact system configurations and Docker would help making it all more uniform. Nevertheless, it puzzles me that the README attributes testing the build on Ubuntu ("Compiler" section) while the Docker uses Arch Linux.

@egzumer I hope you kindly consider my reasoning above. I am open for feedback, especially for getting the build instruction on par with the Docker build output.

bitbam added a commit to bitbam/uv-k5-firmware-custom that referenced this pull request Dec 28, 2024
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