[libcamera-devel] [PATCH 0/2] libcamera: Fix compilation with gcc 8.3.0

[Laurent Pinchart]

Hi Kieran,

On Mon, Oct 10, 2022 at 02:28:46PM +0100, Kieran Bingham wrote:
> Quoting Jacopo Mondi via libcamera-devel (2022-10-10 09:36:49)
> > On Sun, Oct 09, 2022 at 07:44:38AM +0300, Laurent Pinchart via libcamera-devel wrote:
> > > Hello,
> > >
> > > It is with great sadness that I have to announce the removal of
> > > utils::defopt. This feature causes compilation failures with gcc 8.0.0
> > > to 8.3.0 due to (I believe) a gcc bug ([1]), and I haven't been able to
> > > find a suitable workaround. Debian 10 ships gcc 8.3.0, and it has LTS
> > > support until June 2024, making it an important target.
> > >
> > > I'm sure nobody but me will miss utils::defopt, so I'll get over it.
> > 
> > I'm sorry :(
> > But happy gcc8.3 gets fixed
> 
> Seeing it go, I can see some of it's benefits. ... but I agree removing
> it will help fix things for now.
> 
> Seems like we should note somewhere that we might want to re-introduce
> this when we bump our minimum gcc version - but I'm not sure of where
> would be suitable.

Maybe in a bugzilla entry ?

I considered keeping the implementation of defopt in utils.h and
utils.cpp, with a comment to tell it shouldn't be used for now, but I
don't really like carrying dead code.

> > Rafael, would you be able to test this and provide a tag ?
> 
> Worth waiting for this still but:
> 
> 
> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> 
> > For the series
> > Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
> > 
> > Thanks
> >   j
> > 
> > >
> > > [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86521
> > >
> > > Laurent Pinchart (2):
> > >   ipa: rkisp1: Drop use of utils::defopt
> > >   libcamera: base: utils: Drop defopt
> > >
> > >  include/libcamera/base/utils.h    | 14 --------
> > >  src/ipa/rkisp1/algorithms/dpf.cpp |  6 ++--
> > >  src/ipa/rkisp1/algorithms/gsl.cpp |  8 ++---
> > >  src/ipa/rkisp1/algorithms/lsc.cpp |  4 +--
> > >  src/libcamera/base/utils.cpp      | 21 ------------
> > >  test/utils.cpp                    | 53 -------------------------------
> > >  6 files changed, 9 insertions(+), 97 deletions(-)
> > >
> > >
> > > base-commit: 4bf68fdbe1493e57118dad4e94c2a46c50a9c402

-- 
Regards,

Laurent Pinchart