[sane-devel] mystery why sane-backend umax_pp always builds

[Ralph Little]

Hi,

On 2022-07-22 08:56, Ralph Little wrote:
> Hi,
>
> On 2022-07-21 12:19, r. a. schmied wrote:
>> saners
>>
>> the missing '/' on line ending umax
>> (see rcsdiff -uBitwb configure.ac below)
>> might be why the umax_pp always
>> builds even with an envvar like
>> export BACKENDS='genesys net test'
>> passed to configure.
>>
>> yet to determine 1) why the backends
>> umax1220u v4l xerox_mfp p5
>> *do not get* built along with umax_pp and
>> 2) if the added '/' will have any actual effect.
>>
>> i doubt this will have desired effect, as the
>> Makefile(s) do have 'BACKENDS =  genesys net test'
>> and do not have umax_pp (or any other backend)
>> in any obviously related envvar/macro.
>>
>> same missing '/' and build situation exists for
>> sane-backends-1.0.32.86-911be
>> and
>> sane-backends-1.0.28
>>
>>
>> ras
>>
>> --- configure.ac    2022/07/21 17:45:34    1.1
>> +++ configure.ac    2022/07/21 17:45:51
>> @@ -671,7 +671,7 @@
>>          mustek_usb mustek_usb2 nec net niash pie pieusb pint \
>>          pixma plustek plustek_pp qcam ricoh ricoh2 rts8891 s9036 \
>>          sceptre sharp sm3600 sm3840 snapscan sp15c st400 \
>> -        stv680 tamarack teco1 teco2 teco3 test u12 umax
>> +        stv680 tamarack teco1 teco2 teco3 test u12 umax \
>>          umax_pp umax1220u v4l xerox_mfp p5"
>>
>>  # If user specifies backends manually then cause configure
>>
> OK, I looked into this a little bit.
>
> The issue is actually caused by the tools/ directory which builds a 
> tool called umax_pp.
> This tool has a dependency on umax_pp_low which is why, despite the 
> umax_pp backend being disabled, the umax_pp_low.c source is built.
> There doesn't appear to be a configured way to disable this being 
> built so I will add something.
>
> Thanks for raising this issue.
>
> Cheers,
> Ralph
>
OK, more detail on this.

The umax_pp tool in the tools directory is always built. However, if you 
disable the umax_pp backend then it merely sets the umax_pp tool to not 
install, yet it is still built.
I think it is fair to say that if you are disabling the umax_pp backend, 
then you are likely not interested in the umax_pp tool so I have removed 
the build/noinstall option.

I have added a check-in that if you disable the umax_pp backend, the 
tool is not built.

Cheers,
Ralph