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
t.py: init at 2020-04-11 #98474
t.py: init at 2020-04-11 #98474
Conversation
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.
Otherwise lgtm. You can also probably use pname = "t";
attribute name and package name don't always match in nixpkgs.
|
||
python3Packages.buildPythonApplication { | ||
pname = "t.py"; | ||
version = "2020-04-11"; |
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.
version = "2020-04-11"; | |
version = "unstable-2020-04-11"; |
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.
Just pass buildPythonApplication
from python3Packages
instead of the whole package set. This doesn't make a huge difference, but is more in line with other python applications in top level nixpkgs.
pkgs/top-level/all-packages.nix
Outdated
@@ -7307,6 +7307,8 @@ in | |||
|
|||
t = callPackage ../tools/misc/t { }; | |||
|
|||
t_py = callPackage ../applications/misc/t_py { }; |
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.
t_py = callPackage ../applications/misc/t_py { }; | |
t_py = python3Packages.callPackage ../applications/misc/t_py { }; |
@@ -0,0 +1,20 @@ | |||
{ lib, python3Packages, fetchFromGitHub }: |
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.
{ lib, python3Packages, fetchFromGitHub }: | |
{ lib, buildPythonApplication, fetchFromGitHub }: |
@@ -0,0 +1,20 @@ | |||
{ lib, python3Packages, fetchFromGitHub }: | |||
|
|||
python3Packages.buildPythonApplication { |
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.
python3Packages.buildPythonApplication { | |
buildPythonApplication { |
Result of 1 package built:- t_py |
|
Seems like steve is using cram: https://github.com/brodie/cram |
I initially didn't bother with the tests because I didn't know what I was looking at, but that was super easy 🤷 |
@GrahamcOfBorg build t_py |
Last (hopefully) order of business would be squashing all the commits into a single |
Please rebase against |
I marked this as stale due to inactivity. → More info |
Reopen and rebase if you're still interested in this. |
Motivation for this change
It's a nice, simple todo tool. It's called
t
upstream, but since there's already at
in nixpkgs I've called itt.py
.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)