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
qbittorrent: extending path to include Python3 for default functionality #106735
qbittorrent: extending path to include Python3 for default functionality #106735
Conversation
@GrahamcOfBorg build qbittorrent |
is it not compatible with |
Oh, yes, the naming. |
ed32615
to
755432f
Compare
Done. |
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.
The first two commits look good, I tested the search function and it worked as expected. The last commit looks a bit odd to me:
[pbb@onyx:~/proj/nixpkgs]$ tree result/
result
├── bin
│ └── qbittorrent
└── share
├── applications
│ └── org.qbittorrent.qBittorrent.desktop
├── icons
[...]
├── metainfo
│ └── org.qbittorrent.qBittorrent.appdata.xml
└── pixmaps
└── qbittorrent.png
[pbb@onyx:~/proj/nixpkgs]$ tree result-dev/
result-dev
└── nix-support
└── propagated-build-inputs
[pbb@onyx:~/proj/nixpkgs]$ tree result-bin/
result-bin
└── share
└── man
└── man1
└── qbittorrent.1.gz
[pbb@onyx:~/proj/nixpkgs]$
But only the bin
output would be installed to a profile, so no binary or .desktop file would be available.
Maybe you can move the last commit to its own PR so we can merge the rest.
Since people regularly open reports about tracker search not working because python is not found. And since Tracker Search in qBittorrent can not be disabled & at the same time the Pythong is needed for it to work - including Python for default functionality to work.
755432f
to
b951431
Compare
Rebased to Removed the last commit of split outputs. |
Motivation for this change
Since people regularly open reports about tracker search not working because
python
is not found. The most recent of reports/discussions being #104221.And since Tracker Search in qBittorrent can not be disabled & at the same time
the Python is needed for it to work - including Python for default
functionality to work.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
(no dependent packages)
./result/bin/
)nix path-info -S
before and after)