[Buildroot] [PATCH 1/1] package/python-pycups: new package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Tue Aug 25 08:08:21 UTC 2020
Hello Asaf,
Thanks for reviewing!
On Tue, 25 Aug 2020 09:41:28 +0300
Asaf Kahlon <asafka7 at gmail.com> wrote:
> > + bool "python-pycups"
> > + depends on BR2_PACKAGE_CUPS
> Since this package wraps the CUPS API and uses the cups package, you
> should select BR2_PACKAGE_CUPS instead of depending on it (and add the
> additional "depends on" sections from cups).
I perhaps wouldn't be so strict on this. When a dependency is
"obvious", it is quite OK to have a "depends on". In this case, I
believe the user who wants python-pycups will realize that CUPS is
needed in the system, so I find a "depends on" dependency to be quite
OK in this case. Using "depends on" has the advantage that we don't
have to replicate the "depends on" from BR2_PACKAGE_CUPS.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list