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
teleport,teleport.client: 4.2.11 -> 5.1.2 #108487
Conversation
pkgs/servers/teleport/default.nix
Outdated
postInstall = '' | ||
mkdir -p $client/bin | ||
cp $out/bin/tsh $client/bin | ||
strip $client/bin/tsh |
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.
strip $client/bin/tsh |
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
1b4c880
to
2760ee9
Compare
I tried to strip everything, and run --help for the teleport app. |
@freezeboy Nifty to see an update finally. I had given up getting it to work due to a bug with submodule cloning of teleports private e submodule a while back. I have a PR for a module I've been meaning to refactor for ages, try and bang some work out on it this week to move it over to using @infinisil's settings work. I'll check that the binary works with my teleport module tomorrow sometime after work. |
Currently rebasing to merge with the update available in master |
2760ee9
to
64d7dad
Compare
Motivation for this change
Upstream update
And split the client
tsh
binary from the rest to reduce the size for "users" maybe servers could also be shrinked.I would prefer someone to double check on their setup as it is quite a sensitive software.
Maybe could be also interesting to add a module and a test for it
I am also unsure if it is better for the default derivation to contain only the client or both the client and the server
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)