Skip to content

Commit

Permalink
ledger-autosync: don't propagate ledger and hledger
Browse files Browse the repository at this point in the history
The one available in $PATH is automatically selected.
Also remove other superfluous dependencies.
  • Loading branch information
dotlambda committed Aug 12, 2024
1 parent 9ec78f5 commit d11932a
Showing 1 changed file with 7 additions and 24 deletions.
31 changes: 7 additions & 24 deletions pkgs/applications/office/ledger-autosync/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
, fetchpatch2
, ledger
, hledger
, useLedger ? true
, useHledger ? true
}:

python3Packages.buildPythonApplication rec {
Expand Down Expand Up @@ -39,31 +37,16 @@ python3Packages.buildPythonApplication rec {
];

propagatedBuildInputs = with python3Packages; [
asn1crypto
beautifulsoup4
cffi
cryptography
entrypoints
fuzzywuzzy
idna
jeepney
keyring
lxml
mock
nose
ofxclient
ofxhome
ofxparse
pbr
pycparser
secretstorage
six
] ++ lib.optional useLedger ledger
++ lib.optional useHledger hledger;
];

# Checks require ledger as a python package,
# ledger does not support python3 while ledger-autosync requires it.
nativeCheckInputs = with python3Packages; [ ledger hledger nose mock pytestCheckHook ];
nativeCheckInputs = [
hledger
ledger
python3Packages.ledger
python3Packages.pytestCheckHook
];

# Disable some non-passing tests:
# https://github.com/egh/ledger-autosync/issues/127
Expand Down

0 comments on commit d11932a

Please sign in to comment.