[PATCH] meson: Group libipa and libipa_includes in a dependency object

Laurent Pinchart laurent.pinchart at ideasonboard.com
Tue Jun 11 23:30:30 CEST 2024


On Tue, Jun 11, 2024 at 09:28:52PM +0000, Barnabás Pőcze wrote:
> 2024. június 11., kedd 23:24 keltezéssel, Barnabás Pőcze <pobrn at protonmail.com> írta:
> 
> > 2024. június 11., kedd 16:25 keltezéssel, Laurent Pinchart <laurent.pinchart at ideasonboard.com> írta:
> > 
> > > On Tue, Jun 11, 2024 at 01:40:55PM +0000, Barnabás Pőcze wrote:
> > > > 2024. június 11., kedd 14:42 keltezéssel, Laurent Pinchart írta:
> > > >
> > > > > Many build targets link with libipa and need libipa_includes. Group them
> > > > > in a libipa_dep dependency object to simplify the users.
> > > > >
> > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> > > > > [...]
> > > > > diff --git a/src/ipa/libipa/meson.build b/src/ipa/libipa/meson.build
> > > > > index 8ec9c7847348..7f038c90cd87 100644
> > > > > --- a/src/ipa/libipa/meson.build
> > > > > +++ b/src/ipa/libipa/meson.build
> > > > > @@ -27,3 +27,7 @@ libipa_includes = include_directories('..')
> > > > >  libipa = static_library('ipa', [libipa_sources, libipa_headers],
> > > > >                          include_directories : ipa_includes,
> > > > >                          dependencies : libcamera_private)
> > > > > +
> > > > > +libipa_dep = declare_dependency(sources : libipa_headers,
> > > >
> > > > Is this `sources` argument necessary?
> > >
> > > Probably not, as we don't generate any of those headers. They shouldn't
> > > hurt though, as the dependency on sources and headers is already there
> > > implicitly, through 'link_with: libipa'. Should I drop the sources here
> > > or keep them to be ready for code generation in the future ?  :-)
> > 
> > It was just unexpected for me to see it because I hadn't seen anything that
> > would necessitate it, so I would omit it. But as you said, it does not change anything now.
> > [...]
> 
> Oops, never mind, I haven't noticed this change had already been merged.

I can still send a fix on top :-) Your reviews are appreciated, even
after patches get merged.

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list