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:
Well, it's not my pipeline, it's upstream's ditto.
On 2021-02-17 06:41, Pat Suwalski wrote: low.cpp: 636:76: error: unused parameter 'sensor' [-Werror enable_ gamma(const ScanSession& session, const enable_ gamma(const ScanSession& session, const
> genesys/
> =unused-parameter]
>
> You can either change the compiler flags or change the function
> signature from:
>
> bool should_
> Genesys_Sensor& sensor)
>
> to:
>
> bool should_
> 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+.