mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH v3 5/7] cups: new package: Common Unix Printing System
Date: Wed, 1 Nov 2017 11:45:27 +0100	[thread overview]
Message-ID: <20171101104526.mrhreeotw7du6aeo@pengutronix.de> (raw)
In-Reply-To: <e4b27e80-0718-e581-ea6b-93f34920764c@pengutronix.de>

On Tue, Oct 31, 2017 at 11:31:45PM +0100, Roland Hieber wrote:
> On 27.10.2017 19:54, Michael Olbrich wrote:
> > On Wed, Oct 25, 2017 at 10:57:49PM +0200, Roland Hieber wrote:
> >> diff --git a/rules/cups.in b/rules/cups.in
> >> new file mode 100644
> >> index 000000000..13b7142d6
> >> --- /dev/null
> >> +++ b/rules/cups.in
> >> @@ -0,0 +1,113 @@
> >> +## SECTION=applications
> >> +
> >> +menuconfig CUPS
> >> +	tristate
> >> +	prompt "cups                          "
> >> +
> >> +	select ROOTFS_VAR_LOG
> >> +	select ROOTFS_VAR_RUN
> >> +	select LIBC_M
> >> +	select LIBC_CRYPT
> >> +	select ZLIB
> >> +
> >> +	select LIBC_PTHREAD		if CUPS_THREADING
> >> +	select SYSTEMD			if CUPS_SYSTEMD_UNIT
> >> +	select LIBPAPER			if CUPS_LIBPAPER
> >> +	select LIBUSB			if CUPS_LIBUSB
> >> +	select DBUS			if CUPS_DBUS
> >> +	select GNUTLS			if CUPS_SSL
> >> +	select AVAHI			if CUPS_AVAHI
> >> +	select AVAHI_LIBAVAHI_CLIENT	if CUPS_AVAHI
> >> +	select ORACLE_JAVA7_JRE		if CUPS_JAVA
> > 
> > This is not good. ORACLE_JAVA7_JRE is the x86-only binary packages.
> > Just add a comment to the help text, that some java is needed.
> > 
> >> +	select PERL			if CUPS_PERL
> >> +	select PHP5			if CUPS_PHP
> >> +	select PYTHON			if CUPS_PYTHON
> > 
> > Is python2 the only option or could python3 be used as well?
> 
> Python 3 is possible. But I have to pass the path to the executable to
> CUPS' configure, so I would need to reconfigure CUPS if anything of
> PYTHON || PYTHON3 || PYTHON3_SYMLINK changes. Is there a way to ensure
> this? Same for PHP5_CGI || PHP5_CLI.

If really works with both, then I'd say, make it a choice (2/3/no) and
/usr/bin/python{2,3} explicitly.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2017-11-01 10:45 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-25 20:57 [ptxdist] [PATCH v3 1/7] libpaper: new package: system-wide papersize configuration Roland Hieber
2017-10-25 20:57 ` [ptxdist] [PATCH v3 2/7] qpdf: new package: PDF inspection library and tools Roland Hieber
2017-10-25 20:57 ` [ptxdist] [PATCH v3 3/7] poppler-data: new package: optional encoding data for poppler Roland Hieber
2017-10-25 20:57 ` [ptxdist] [PATCH v3 4/7] poppler: version bump 0.18.0 -> 0.59.0 Roland Hieber
2017-10-25 21:15   ` Roland Hieber
2017-10-28  9:46   ` Michael Olbrich
2017-10-25 20:57 ` [ptxdist] [PATCH v3 5/7] cups: new package: Common Unix Printing System Roland Hieber
2017-10-27 17:54   ` Michael Olbrich
2017-10-31 22:31     ` Roland Hieber
2017-11-01 10:45       ` Michael Olbrich [this message]
2017-10-28  9:54   ` Michael Olbrich
2017-10-31 21:17     ` Roland Hieber
2017-10-25 20:57 ` [ptxdist] [PATCH v3 6/7] host-cups: new package: CUPS host tools Roland Hieber
2017-10-28  9:57   ` Michael Olbrich
2017-10-25 20:57 ` [ptxdist] [PATCH v3 7/7] cups-filters: new package: additional filters and backends for CUPS Roland Hieber
2017-11-01 10:51 ` [ptxdist] [PATCH v3 1/7] libpaper: new package: system-wide papersize configuration Michael Olbrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171101104526.mrhreeotw7du6aeo@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox