Re: [ptxdist] [PATCH 5/7] cups: new package: Common Unix Printing System

[Roland Hieber <r.hieber@pengutronix.de>]

On 28.08.2017 14:51, Michael Olbrich wrote:
>> +CUPS_CONF_OPT	+= \
>> +	--with-components=all \
>> +	--with-cachedir=/var/cache \
>> +	--with-logdir=/var/log \
>> +	--with-rundir=/run \
>> +	--with-rcdir=/etc \
>> +	--with-languages=none \
>> +	--with-cups-user=daemon \
>> +	--with-cups-group=lp \
>> +	$(call ptx/ifdef,PTXCONF_CUPS_JAVA,--with-java=/usr/bin/java,) \
>> +	$(call ptx/ifdef,PTXCONF_CUPS_PERL,--with-perl=/usr/bin/perl,) \
>> +	$(call ptx/ifdef,PTXCONF_CUPS_PHP,--with-php=/usr/bin/php5,) \
>> +	$(call ptx/ifdef,PTXCONF_CUPS_PYTHON,--with-python=/usr/bin/python,)
> 
> Are these options only to set the executable path? If yes then always set
> those and remove the options and dependencies. Just add a comment to the
> webinterface option.

configure defines HAVE_JAVA etc. in these cases, which influences the 
scheduler code. Nevertheless, configure is a bit broken here, PATCH v2 
will patch it for proper --without-* handling.

  - Roland

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de