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
rust-9p: init #76117
rust-9p: init #76117
Conversation
pkgs/servers/rust-9p/default.nix
Outdated
@@ -0,0 +1,36 @@ | |||
{ | |||
stdenv, fetchFromGitHub, rust | |||
}: |
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.
Should be one line: https://nixos.org/nixpkgs/manual/#sec-syntax
c48763d
to
b839448
Compare
pkgs/servers/rust-9p/default.nix
Outdated
meta = with stdenv.lib; { | ||
description = "9P2000.L server implementation in Rust"; | ||
homepage = "https://github.com/pfpacket/rust-9p"; | ||
license = with licenses; [ bsd3 ]; |
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.
license = with licenses; [ bsd3 ]; | |
license = licenses.bsd3; |
I think we usually only use a list for multiple licenses.
pkgs/servers/rust-9p/default.nix
Outdated
maintainers = with maintainers; [ raskin ]; | ||
|
||
# macOS build fails: https://github.com/pfpacket/rust-9p/issues/7 | ||
platforms = with platforms; linux; |
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’s nothing that makes macOS intrinsically unsupported, so I think this should be
platforms = with platforms; linux; | |
platforms = platforms.unix; |
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.
No version of upstream code has ever worked on any version of macOS, as far as I understand. We need either broken or platform limitation, and I think platform limitation is fine in this case.
pkgs/servers/rust-9p/default.nix
Outdated
|
||
cargoSha256 = "1d33nwj3i333a6ji3r3037mgg553lc3wsawm0pz13kbvhjf336i8"; | ||
|
||
RUSTC_BOOTSTRAP=1; |
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.
RUSTC_BOOTSTRAP=1; | |
RUSTC_BOOTSTRAP = 1; |
b839448
to
147d414
Compare
Should the package be called unpfs? rust-9p seems to be more of the
name of the library.
|
147d414
to
853953a
Compare
Should the package be called unpfs? rust-9p seems to be more of the
name of the library.
Indeed; when I started the packaging I thought that packaing rust-9p in
a way that allows its use as a libary, too, would be straightforward.
|
(cherry-picked from NixOS#76117)
Motivation for this change
A Rust-based implementation of 9p2000.L, seems more trustable than C-based diod
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify
cc @qyliss