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 quickstart description #16

Merged
merged 5 commits into from
Feb 11, 2024
Merged

Add quickstart description #16

merged 5 commits into from
Feb 11, 2024

Conversation

ZoomTen
Copy link
Collaborator

@ZoomTen ZoomTen commented Jan 1, 2024

It's the one for Debian, for now.

Should address #14 even though it's not "quite" a one-liner, since it also covers installing the dependencies.

Some notes while writing the quickstart:

  • Should we be able to select stable (latest version) and devel (master) versions?
  • Making the curl URL tidier (could be by e.g. making a URL in gbdev.io that redirects to the script)
  • Could use a command to print the rgbenv dirs
  • Or maybe take inspiration from version managers that have an eval $(version_manager init) command?
  • The script can be placed locally, of course, but placing it in /usr/local/bin is just a convenience thing (no need to track additional $PATH dirs—do we need an additional install.sh script?

Should address #14 even though it's not "quite" a one-liner, since it also covers installing the dependencies.
Copy link
Member

@ISSOtm ISSOtm left a comment

Choose a reason for hiding this comment

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

The placement and general idea look good to me; there are things I'd question about some details, though.

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
ZoomTen and others added 4 commits January 3, 2024 23:16
Co-authored-by: Eldred Habert <eldredhabert0@gmail.com>
Co-authored-by: Eldred Habert <eldredhabert0@gmail.com>
Co-authored-by: Eldred Habert <eldredhabert0@gmail.com>
Copy link
Member

@ISSOtm ISSOtm 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 to me!

@ZoomTen ZoomTen merged commit 210ac8f into master Feb 11, 2024
4 checks passed
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.

None yet

2 participants