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
mmtc: init at 0.2.6 #105702
mmtc: init at 0.2.6 #105702
Conversation
Result of 1 package built:
|
Ofborg didn't seem to like it.
What does it mean? What can I do to fix it? |
"Minimal mpd terminal client that aims to be simple yet highly configurable"; | ||
homepage = "https://github.com/figsoda/mmtc"; | ||
license = licenses.mpl20; | ||
maintainers = [ ]; |
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.
Missing maintainers and platforms.
I'm not sure what platforms exactly is it going to work on, so it's set to platforms.all so I can know which ones work
Ofborg still says the same thing. Does it have something to do with |
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.
Please squash your mmtc related commits together.
rust = (let | ||
mozilla = fetchFromGitHub { | ||
owner = "mozilla"; | ||
repo = "nixpkgs-mozilla"; |
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.
I don't think this is acceptable in nixpkgs.
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.
I don't think this is acceptable in nixpkgs.
I guess that's why, I will close this then
Motivation for this change
mmtc is a minimal mpd terminal client that aims to be simple yet highly configurable
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)