bitcoin-tools / bitcoin-tools/diyjade
Improve dependency installation
- Dominant language
- Shell
- Stars
- 5
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
This is the feedback from the workshop with 10 diy jade installations from the weekend. All in all it worked pretty well. One participant had problems with installing the dependencies, as the script told him only one dependency to install per run so he had to run the script a lot of times to iterate over all. So improvements would be:
- Add an apt command in the readme in the automated sections to install all dependencies at once
- If the script detects that one dependency is missing, show the missing dependency together will an install command for all others or even better, test for all missing at once and show only these
- Provide an even more noob friendly option for installation: I thought about a python app image which is reproducible build on github ci that contains python and esp-idf, jade, the scripts and all dependencies so that this works independently from the linux distribution / packages installed with just one command execution. With this one large binary file it should be even possible to do offline jade installations
What do you think about it?
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the README's automated installation section and the dependency-checking script mentioned in the issue; compare the requested apt instructions, batched missing-dependency checks, and Python app-image option. Done needs a decided scope with documented installation behavior, including explicit acceptance criteria for reproducible CI builds and one-command or offline installation if the image route is chosen.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python, shell
- Domain
- build-system, devops, documentation
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100