-
Notifications
You must be signed in to change notification settings - Fork 9
feat(main): add "INSTALL.md" file #726
base: main
Are you sure you want to change the base?
Changes from all commits
ab20b64
d40e120
00186ec
17bdc07
2e25063
6a3fc66
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -11,6 +11,10 @@ | |
| { | ||
| "type": "generic", | ||
| "path": "pkg/config/config.go" | ||
| }, | ||
| { | ||
| "type": "generic", | ||
| "path": "INSTALL.md" | ||
| } | ||
| ] | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,6 +10,10 @@ | |
| { | ||
| "type": "generic", | ||
| "path": "pkg/config/config.go" | ||
| }, | ||
| { | ||
| "type": "generic", | ||
| "path": "INSTALL.md" | ||
| } | ||
| ] | ||
| } | ||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,254 @@ | ||||||||||||||||||||||||||||||||||||||||||||
| # Installing otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| This guide will help you install **otdfctl** on your computer. Follow the steps for your operating system. | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| ## What is otdfctl? | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| `otdfctl` is a command-line tool for working with OpenTDF. Once installed, you'll be able to run it from any folder on your computer. | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| ## 📥 Installation Instructions | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| <details> | ||||||||||||||||||||||||||||||||||||||||||||
| <summary><h3>For macOS Users</h3></summary> | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| #### Step 1: Download the Right Version | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| First, you need to know which type of Mac you have: | ||||||||||||||||||||||||||||||||||||||||||||
| - **Apple Silicon (M1, M2, M3, etc.)**: Download the `arm64` version | ||||||||||||||||||||||||||||||||||||||||||||
| - **Intel Mac**: Download the `amd64` version | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **Don't know which one you have?** Open Terminal and type: | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| uname -m | ||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
| - If it says `arm64`, you have Apple Silicon | ||||||||||||||||||||||||||||||||||||||||||||
| - If it says `x86_64`, you have an Intel Mac | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| #### Step 2: Download and Install | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **For Apple Silicon Macs (M1/M2/M3):** | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| # Download the latest version | ||||||||||||||||||||||||||||||||||||||||||||
| curl -LO https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-darwin-arm64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The version To improve maintainability, consider using a variable for the version number at the beginning of each script block. This would allow for a single point of update per script and reduce the chance of errors. Example: VERSION="0.28.0"
curl -LO https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/otdfctl-${VERSION}-darwin-arm64.tar.gz
# ... and use ${VERSION} in subsequent commands
|
||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Extract the file | ||||||||||||||||||||||||||||||||||||||||||||
| tar -xzf otdfctl-0.28.0-darwin-arm64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Create the directory if it doesn't exist | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mkdir -p /usr/local/bin | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Move it to a folder in your PATH | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mv target/otdfctl-0.28.0-darwin-arm64 /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The path This same issue exists on lines 63, 103, 124, and 145.
Suggested change
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The path
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Make it executable | ||||||||||||||||||||||||||||||||||||||||||||
| sudo chmod +x /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Clean up the downloaded file | ||||||||||||||||||||||||||||||||||||||||||||
| rm otdfctl-0.28.0-darwin-arm64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+33
to
+49
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This installation script has two issues:
The suggested change below fixes both issues by removing # Set the version you want to install
VERSION="0.28.0"
TARBALL="otdfctl-${VERSION}-darwin-arm64.tar.gz"
BINARY="otdfctl-${VERSION}-darwin-arm64"
# Download the release archive
curl -LO "https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/${TARBALL}"
# Extract the file
tar -xzf "${TARBALL}"
# Create the directory if it doesn't exist
sudo mkdir -p /usr/local/bin
# Move it to a folder in your PATH
sudo mv "${BINARY}" /usr/local/bin/otdfctl
# Make it executable
sudo chmod +x /usr/local/bin/otdfctl
# Clean up the downloaded file
rm "${TARBALL}"
Comment on lines
+33
to
+49
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The installation script has two issues:
I suggest refactoring the script to use variables. This makes it more readable, easier to update, and fixes the incorrect path. # Download the latest version
VERSION="0.28.0"
OS="darwin"
ARCH="arm64"
FILENAME="otdfctl-${VERSION}-${OS}-${ARCH}"
curl -LO "https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/${FILENAME}.tar.gz"
# Extract the file
tar -xzf "${FILENAME}.tar.gz"
# Create the directory if it doesn't exist
sudo mkdir -p /usr/local/bin
# Move it to a folder in your PATH
sudo mv otdfctl /usr/local/bin/otdfctl
# Make it executable
sudo chmod +x /usr/local/bin/otdfctl
# Clean up the downloaded file
rm "${FILENAME}.tar.gz" |
||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **For Intel Macs:** | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| # Download the latest version | ||||||||||||||||||||||||||||||||||||||||||||
| curl -LO https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-darwin-amd64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Extract the file | ||||||||||||||||||||||||||||||||||||||||||||
| tar -xzf otdfctl-0.28.0-darwin-amd64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Create the directory if it doesn't exist | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mkdir -p /usr/local/bin | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Move it to a folder in your PATH | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mv target/otdfctl-0.28.0-darwin-amd64 /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As with the arm64 instruction, the
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Make it executable | ||||||||||||||||||||||||||||||||||||||||||||
| sudo chmod +x /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Clean up the downloaded file | ||||||||||||||||||||||||||||||||||||||||||||
| rm otdfctl-0.28.0-darwin-amd64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+54
to
+70
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similar to the Apple Silicon script, this script has an incorrect # Download the latest version
VERSION="0.28.0"
OS="darwin"
ARCH="amd64"
FILENAME="otdfctl-${VERSION}-${OS}-${ARCH}"
curl -LO "https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/${FILENAME}.tar.gz"
# Extract the file
tar -xzf "${FILENAME}.tar.gz"
# Create the directory if it doesn't exist
sudo mkdir -p /usr/local/bin
# Move it to a folder in your PATH
sudo mv otdfctl /usr/local/bin/otdfctl
# Make it executable
sudo chmod +x /usr/local/bin/otdfctl
# Clean up the downloaded file
rm "${FILENAME}.tar.gz" |
||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| > **Note:** You'll be asked for your password when using `sudo`. This is normal and required to install the tool. | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| </details> | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| <details> | ||||||||||||||||||||||||||||||||||||||||||||
| <summary><h3>For Linux Users</h3></summary> | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| #### Step 1: Download the Right Version | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| Choose based on your system: | ||||||||||||||||||||||||||||||||||||||||||||
| - **Most modern PCs**: `amd64` | ||||||||||||||||||||||||||||||||||||||||||||
| - **Raspberry Pi or ARM devices**: `arm64` or `arm` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **Not sure?** Run this command: | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| uname -m | ||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| #### Step 2: Download and Install | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **For amd64 (most common):** | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| # Download the latest version | ||||||||||||||||||||||||||||||||||||||||||||
| curl -LO https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-linux-amd64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Extract the file | ||||||||||||||||||||||||||||||||||||||||||||
| tar -xzf otdfctl-0.28.0-linux-amd64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Create the directory if it doesn't exist | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mkdir -p /usr/local/bin | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Move it to a folder in your PATH | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mv target/otdfctl-0.28.0-linux-amd64 /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Make it executable | ||||||||||||||||||||||||||||||||||||||||||||
| sudo chmod +x /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Clean up the downloaded file | ||||||||||||||||||||||||||||||||||||||||||||
| rm otdfctl-0.28.0-linux-amd64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+97
to
+113
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This script also contains an incorrect # Download the latest version
VERSION="0.28.0"
OS="linux"
ARCH="amd64"
FILENAME="otdfctl-${VERSION}-${OS}-${ARCH}"
curl -LO "https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/${FILENAME}.tar.gz"
# Extract the file
tar -xzf "${FILENAME}.tar.gz"
# Create the directory if it doesn't exist
sudo mkdir -p /usr/local/bin
# Move it to a folder in your PATH
sudo mv otdfctl /usr/local/bin/otdfctl
# Make it executable
sudo chmod +x /usr/local/bin/otdfctl
# Clean up the downloaded file
rm "${FILENAME}.tar.gz" |
||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **For ARM64:** | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| # Download the latest version | ||||||||||||||||||||||||||||||||||||||||||||
| curl -LO https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-linux-arm64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Extract the file | ||||||||||||||||||||||||||||||||||||||||||||
| tar -xzf otdfctl-0.28.0-linux-arm64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Create the directory if it doesn't exist | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mkdir -p /usr/local/bin | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Move it to a folder in your PATH | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mv target/otdfctl-0.28.0-linux-arm64 /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As with the other installation commands, the
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Make it executable | ||||||||||||||||||||||||||||||||||||||||||||
| sudo chmod +x /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Clean up the downloaded file | ||||||||||||||||||||||||||||||||||||||||||||
| rm otdfctl-0.28.0-linux-arm64.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+118
to
+134
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The # Download the latest version
VERSION="0.28.0"
OS="linux"
ARCH="arm64"
FILENAME="otdfctl-${VERSION}-${OS}-${ARCH}"
curl -LO "https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/${FILENAME}.tar.gz"
# Extract the file
tar -xzf "${FILENAME}.tar.gz"
# Create the directory if it doesn't exist
sudo mkdir -p /usr/local/bin
# Move it to a folder in your PATH
sudo mv otdfctl /usr/local/bin/otdfctl
# Make it executable
sudo chmod +x /usr/local/bin/otdfctl
# Clean up the downloaded file
rm "${FILENAME}.tar.gz" |
||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| **For ARM (32-bit):** | ||||||||||||||||||||||||||||||||||||||||||||
| ```bash | ||||||||||||||||||||||||||||||||||||||||||||
| # Download the latest version | ||||||||||||||||||||||||||||||||||||||||||||
| curl -LO https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-linux-arm.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Extract the file | ||||||||||||||||||||||||||||||||||||||||||||
| tar -xzf otdfctl-0.28.0-linux-arm.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Create the directory if it doesn't exist | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mkdir -p /usr/local/bin | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Move it to a folder in your PATH | ||||||||||||||||||||||||||||||||||||||||||||
| sudo mv target/otdfctl-0.28.0-linux-arm /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As with the other installation commands, the
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Make it executable | ||||||||||||||||||||||||||||||||||||||||||||
| sudo chmod +x /usr/local/bin/otdfctl | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| # Clean up the downloaded file | ||||||||||||||||||||||||||||||||||||||||||||
| rm otdfctl-0.28.0-linux-arm.tar.gz | ||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+139
to
+155
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The # Download the latest version
VERSION="0.28.0"
OS="linux"
ARCH="arm"
FILENAME="otdfctl-${VERSION}-${OS}-${ARCH}"
curl -LO "https://github.com/opentdf/otdfctl/releases/download/v${VERSION}/${FILENAME}.tar.gz"
# Extract the file
tar -xzf "${FILENAME}.tar.gz"
# Create the directory if it doesn't exist
sudo mkdir -p /usr/local/bin
# Move it to a folder in your PATH
sudo mv otdfctl /usr/local/bin/otdfctl
# Make it executable
sudo chmod +x /usr/local/bin/otdfctl
# Clean up the downloaded file
rm "${FILENAME}.tar.gz" |
||||||||||||||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| </details> | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| --- | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| <details> | ||||||||||||||||||||||||||||||||||||||||||||
| <summary><h3>For Windows Users</h3></summary> | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| #### Step 1: Download the Right Version | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| Most Windows computers use `amd64`. ARM versions are for Surface Pro X or other ARM-based Windows devices. | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| #### Step 2: Download and Install | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| 1. **Download the file** for your system: | ||||||||||||||||||||||||||||||||||||||||||||
| - For most PCs: [otdfctl-0.28.0-windows-amd64.zip](https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-windows-amd64.zip) | ||||||||||||||||||||||||||||||||||||||||||||
| - For ARM devices: [otdfctl-0.28.0-windows-arm64.zip](https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-windows-arm64.zip) | ||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||
| 2. **Extract the ZIP file:** | ||||||||||||||||||||||||||||||||||||||||||||
| - Right-click the downloaded file | ||||||||||||||||||||||||||||||||||||||||||||
| - Select "Extract All..." | ||||||||||||||||||||||||||||||||||||||||||||
| - Choose a location and create a folder: `%USERPROFILE%\otdfctl` | ||||||||||||||||||||||||||||||||||||||||||||
| - The executable will be in this folder (you may need to navigate into a subfolder) | ||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+178
to
+179
|
||||||||||||||||||||||||||||||||||||||||||||
| - Choose a location and create a folder: `%USERPROFILE%\otdfctl` | |
| - The executable will be in this folder (you may need to navigate into a subfolder) | |
| - Choose any temporary location (for example, your `Downloads` folder) and complete the extraction | |
| - After extraction, create the folder `%USERPROFILE%\otdfctl` if it doesn't already exist | |
| - Move `otdfctl.exe` from the extracted folder into `%USERPROFILE%\otdfctl` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a couple of areas for improvement in the Windows installation instructions:
- Hardcoded Links: The download links on lines 165-166 point to a specific version (
v0.28.0). These will become outdated with the next release. It's better to direct users to the "Latest Release" page where they can find the correct assets. - Incorrect Extraction Path: The note on line 172 mentions that the user might need to navigate into a subfolder. However, the
.ziparchives for Windows contain theotdfctl.exeexecutable at the root, so this note is misleading and should be removed.
| 1. **Download the file** for your system: | |
| - For most PCs: [otdfctl-0.28.0-windows-amd64.zip](https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-windows-amd64.zip) | |
| - For ARM devices: [otdfctl-0.28.0-windows-arm64.zip](https://github.com/opentdf/otdfctl/releases/download/v0.28.0/otdfctl-0.28.0-windows-arm64.zip) | |
| 2. **Extract the ZIP file:** | |
| - Right-click the downloaded file | |
| - Select "Extract All..." | |
| - Choose a location and create a folder: `%USERPROFILE%\otdfctl` | |
| - The executable will be in this folder (you may need to navigate into a subfolder) | |
| 1. **Download the file** for your system from the [latest release page](https://github.com/opentdf/otdfctl/releases/latest). Look for the `.zip` file that matches your system architecture (e.g., `otdfctl-x.y.z-windows-amd64.zip`). | |
| 2. **Extract the ZIP file:** | |
| - Right-click the downloaded file | |
| - Select "Extract All..." | |
| - Choose a location and create a folder: `%USERPROFILE%\otdfctl` | |
| - The executable will be in this folder. |
Copilot
AI
Dec 19, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The example path %USERPROFILE%\otdfctl does not match the suggested extraction location from step 2 (C:\Program Files\otdfctl). This inconsistency may confuse users. Either use the same path in both places or clarify that users should use whichever path they chose during extraction.
| - Click "New" and add the folder path where you extracted `otdfctl.exe` (e.g., `%USERPROFILE%\otdfctl`) | |
| - Click "New" and add the folder path where you extracted `otdfctl.exe` (e.g., `C:\Program Files\otdfctl`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The manual installation steps for Windows are a good start. As you asked for feedback on this, here are a couple of suggestions to improve the user experience:
-
Clarify Extraction Step: The instruction on line 179, "The executable will be in this folder (you may need to navigate into a subfolder)", is a bit vague. It would be more helpful to be specific about the structure of the ZIP file. For example: "After extraction, you will find
otdfctl.exeinside a folder likeotdfctl-0.28.0-windows-amd64. Moveotdfctl.exeto the%USERPROFILE%\otdfctlfolder you created." -
Automate with a Script: For a much smoother experience, you could provide a PowerShell script to automate the download, extraction, and PATH update. Users could run it with a single command. Here's an example:
# Set variables $version = "0.28.0" # This would be updated with releases $arch = "amd64" # or "arm64" $url = "https://github.com/opentdf/otdfctl/releases/download/v$version/otdfctl-$version-windows-$arch.zip" $installDir = "$env:USERPROFILE\otdfctl" $zipFile = "$env:TEMP\otdfctl.zip" # Create installation directory if it doesn't exist if (-not (Test-Path -Path $installDir)) { New-Item -ItemType Directory -Path $installDir | Out-Null } # Download and extract Write-Host "Downloading otdfctl v$version..." Invoke-WebRequest -Uri $url -OutFile $zipFile Write-Host "Extracting to $installDir..." Expand-Archive -Path $zipFile -DestinationPath $installDir -Force # Add to user's PATH if not already present $currentUserPath = [System.Environment]::GetEnvironmentVariable('Path', 'User') if ($currentUserPath -notlike "*$installDir*") { $newPath = "$currentUserPath;$installDir" [System.Environment]::SetEnvironmentVariable('Path', $newPath, 'User') Write-Host "Added '$installDir' to your PATH. Please restart your terminal for the changes to take effect." } else { Write-Host "'$installDir' is already in your PATH." } # Clean up Remove-Item $zipFile Write-Host "Installation complete."
-
Package Managers: Long-term, submitting
otdfctlto a package manager like Winget or Scoop would be ideal for simplifying installation and updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar to other parts of the file, the version number here is hardcoded. This might confuse users if they install a different version than the one mentioned. It would be better to show a generic output to avoid confusion and make the documentation more version-agnostic.
| otdfctl version 0.28.0 | |
| otdfctl version <VERSION> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The version number 0.28.0 is hardcoded throughout the installation instructions (appearing in lines 33, 36, 42, 48, 54, 57, 63, 69, 94, 97, 103, 109, 115, 118, 124, 130, 136, 139, 145, 151, 165, 166, 194). This creates a maintenance burden when updating versions. Consider adding a note at the beginning of the document instructing users to replace 0.28.0 with the latest version number, or create a variable placeholder that can be easily updated in one place.