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

Todo squash and better apt and systemd usage in instruction #29

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

StephDC
Copy link

@StephDC StephDC commented Jan 30, 2024

  • L28: minit user would be automatically created by the postinst script of MinKNOW, thus not necessary.
  • L34: the home directory does not seem to matter as long as we are at some home
  • L51: apt-key is deprecated. Use /etc/apt/trusted.gpg.d instead
  • L112: Use sudo -e to let the user edit the file, instead of a simple cat to show content
  • L128: Instead of replacing the whole minknow unit file, use override to only override the user/group settings would be better and survive future minknow upgrades

I just assisted someone to follow this manual to install MinKNOW on Jetson, and it worked fine. Thanks for all your work, and there are a few things I confirmed and modified during the process, with my understanding to those Linux tools.

- minit user would be created automatically during package installation, so no need to create it. Also, it does not matter which directory we are working on as long as we are at home.
- put the key to /etc/apt/trusted.gpg.d as apt-key is deprecated.
- use nano for usb_firmware.toml to let the user know they need to modify the file
- instead of overwriting minknow.service provided by the package, create an override file to only change user and group.
add the missing part to start minknow service
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.

1 participant