docs: improve Linux manual installation guide#497
Open
sanki92 wants to merge 3 commits intotofuutils:mainfrom
Open
docs: improve Linux manual installation guide#497sanki92 wants to merge 3 commits intotofuutils:mainfrom
sanki92 wants to merge 3 commits intotofuutils:mainfrom
Conversation
- Added comprehensive Linux manual installation instructions - Included both system-wide and user-local installation options - Added prerequisites section for Linux manual installation - Improved commands based on community suggestions in issue tofuutils#263 - Used wget instead of curl for better compatibility - Added clear examples for different installation scenarios Fixes tofuutils#263 Signed-off-by: shanky <sankalpt92@gmail.com>
88dc174 to
85c0c26
Compare
Collaborator
|
Thanks! I'll check it tonight |
kvendingoldo
requested changes
Oct 6, 2025
|
|
||
| </details> | ||
|
|
||
| <details markdown="1"><summary><b>Linux: Manual Installation Prerequisites</b></summary><br> |
Collaborator
There was a problem hiding this comment.
I suggest you rename sections:
Linux: RPM -> Linux: CentOS/Fedora
Linux: dkpg -> Linux: Debian/Ubuntu
And add new information there. I'm not sure that we need wget, and jq. Curl is enough to install tenv and cosign.
|
|
||
| ```sh | ||
| # Download and install tenv system-wide | ||
| wget -q -O - "https://api.github.com/repos/tofuutils/tenv/releases/latest" | \ |
|
|
||
| ```sh | ||
| # Download to /usr/bin (requires sudo) | ||
| wget -q -O - "https://api.github.com/repos/tofuutils/tenv/releases/latest" | \ |
| ```sh | ||
| # Install to /usr/local/tenv and create symlinks | ||
| sudo mkdir -p /usr/local/tenv | ||
| wget -q -O - "https://api.github.com/repos/tofuutils/tenv/releases/latest" | \ |
| mkdir -p ${BINDIR} | ||
|
|
||
| # Download and install tenv locally | ||
| wget -q -O - "https://api.github.com/repos/tofuutils/tenv/releases/latest" | \ |
- Renamed 'Linux: RPM' to 'Linux: CentOS/Fedora' - Renamed 'Linux: dkpg' to 'Linux: Debian/Ubuntu' - Updated prerequisites to use curl instead of wget and jq - Replaced all wget commands with curl equivalents - Updated installation notes to reflect curl usage Addresses feedback from kvendingoldo Signed-off-by: shanky <sankalpt92@gmail.com>
- Removed jq from prerequisites completely - Updated all installation commands to use direct download URLs Signed-off-by: shanky <sankalpt92@gmail.com>
Author
|
Hi @kvendingoldo! Thanks for the review feedback. I've addressed all your suggestions. The PR description has also been updated to reflect these changes. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improve Linux manual installation guide
Description
The current Linux manual installation documentation was pretty bare-bones - just a one-liner pointing to releases. This PR addresses issue #263 by adding proper installation instructions that actually help users get tenv running on their systems.
What's changed
Added a proper Linux manual installation section that covers:
Tasks completed
Testing
Fixes #263