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
Feature/brother ql #44646
Feature/brother ql #44646
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.
Please add a pythonPackages.
prefix to the appropriate commit messages.
propagatedBuildInputs = [ future packbits pillow pyusb ]; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Python package for the raster language protocol of the Brother QL series label printers (QL-500, QL-550, QL-570, QL-700, QL-710W, QL-720NW, QL-800, QL-820NWB, QL-1050 and more)"; |
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.
Too long
description = "Python package for the raster language protocol of the Brother QL series label printers (QL-500, QL-550, QL-570, QL-700, QL-710W, QL-720NW, QL-800, QL-820NWB, QL-1050 and more)"; | ||
homepage = https://github.com/pklaus/brother_ql; | ||
license = licenses.gpl3; | ||
maintainers = with 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.
Maintainer required.
description = "PackBits encoder/decoder for Python"; | ||
homepage = https://github.com/psd-tools/packbits; | ||
license = [ licenses.bsd2 ]; | ||
maintainers = with 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.
Maintainer required
@@ -232,6 +232,8 @@ in { | |||
|
|||
breathe = callPackage ../development/python-modules/breathe { }; | |||
|
|||
brother_ql = callPackage ../development/python-modules/brother_ql { }; |
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.
Is brother_ql
really a Python module, seems more like a CLI tool only. In that case, it doesn't belong in pythonPackages
and should use 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.
It is a library and application. There is a brother_ql_web program that uses brother_ql as a dependency. It has however also some scripts to create labels.
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.
Okay, then it belongs into pythonPackages
.
9f5167d
to
36f7740
Compare
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 create three commits where one adds yourself to maintainer-list.nix
and one for each package.
Also note that there's a merge conflict. You should rebase on master.
meta = with stdenv.lib; { | ||
description = ''Python package for the raster language protocol of the Brother QL series\ | ||
label printers (QL-500, QL-550, QL-570, QL-700, QL-710W, QL-720NW, QL-800,\ | ||
QL-820NWB, QL-1050 and more)''; |
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.
This must be a short one-liner: https://nixos.org/nixpkgs/manual/#sec-standard-meta-attributes
Any updates on this pull request, please? |
@poelzi Are you still interested in moving this forward? |
Motivation for this change
Allows to print label with the Brother QL printer line. Tested and works with a QL-800 printer on Linux.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)