Comment 30 for bug 1327801

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

Well, it's not my pipeline, it's upstream's ditto.

On 2021-02-17 06:41, Pat Suwalski wrote:
> genesys/low.cpp:636:76: error: unused parameter 'sensor' [-Werror
> =unused-parameter]
>
> You can either change the compiler flags or change the function
> signature from:
>
> bool should_enable_gamma(const ScanSession& session, const
> Genesys_Sensor& sensor)
>
> to:
>
> bool should_enable_gamma(const ScanSession& session, const
> Genesys_Sensor&)

Yep, that's it. Thanks a lot! (I failed to find the actual log yesterday.)

> But I don't think that build is important to the distro issue.

Right, it apparently isn't. But I'm disinclined to propose a fix in Ubuntu only which is not present in Debian and/or upstream. So I submitted a merge request:

https://gitlab.com/sane-project/backends/-/merge_requests/597

Once that (or something to the same effect) has been committed upstream, we can include it in Ubuntu 20.04+.