docs(install): add package manager commands and cargo install#1280
docs(install): add package manager commands and cargo install#128077xt wants to merge 5 commits into
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e2659cfebf
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4d0f846cef
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
forced pushed to fix a typo in commit message |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7871fe95a0
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9880252a5d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 759c545900
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| paru -S cabin | ||
|
|
||
| # Nix/NixOS | ||
| nix-shell -p nixpkgs.cabin # temporary |
There was a problem hiding this comment.
The nix-shell -p nixpkgs.cabin example is not a valid -p package selector for typical Nix setups: nix-shell --packages/-p expects attribute names (or full expressions), so this form attempts to resolve an attribute path that is usually undefined and fails before launching the shell. Users following the Linux install instructions on Nix/NixOS will hit an immediate command error instead of getting a working Cabin environment.
Useful? React with 👍 / 👎.
|
|
||
| # Nix/NixOS | ||
| nix-shell -p nixpkgs.cabin # temporary | ||
| nix profile install nixpkgs#cabin # per-use |
There was a problem hiding this comment.
The nix profile install nixpkgs#cabin command targets the wrong nixpkgs package name: in nixpkgs, cabin is the font package, while the Cabin CLI is packaged as cabinpkg (with mainProgram = "cabin"). As written, this instruction installs a font package and leaves users without the cabin executable.
Useful? React with 👍 / 👎.
ken-matsui
left a comment
ken-matsui
left a comment
There was a problem hiding this comment.
INSTALL.md is for installing from source. Please update docs/installation.md instead.
|
Closing due to inactivity. |
2 participants
I'd recommend merge this after the rust binary releases are up on github