strepsirrhini/office-gobmx
9
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

avmediagtk: Use gb_Library_set_include for GTK4_CFLAGS

Browse source 
In the same way that `vcl/Library_vclplug_gtk4.mk` does
it, set the `GTK4_CFLAGS` (which have the includes) via
`gb_Library_set_include`, not `gb_Library_add_cxxflags`.

This makes sure that they end up in the `INCLUDE`
section in `workdir/GbuildToJson/Library/libavmediagtk.so`
and thus end up in the IDE integrations as includes,
which makes Qt Creator's Clang Code Model  find them
instead of showing an error
("gtkplayer.cxx:28:10: 'gtk/gtk.h' file not found").

See also this similar commit for more details:

    commit 6a53832080
    Author: Michael Weghorn <m.weghorn@posteo.de>
    Date:   Mon Dec 17 15:35:58 2018 +0100

        Use 'gb_Library_set_include' for GTK3_CFLAGS

Change-Id: I5ef01aff32073d37b8add0b2250402ccaa48b2dc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167857
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
This commit is contained in:
Michael Weghorn 2024-05-20 10:28:05 +02:00
parent 5a602ed852
commit ba41718095
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
avmedia/Library_avmediagtk.mk
View file

@ -13,16 +13,13 @@ $(eval $(call gb_Library_set_componentfile,avmediagtk,avmedia/source/gtk/avmedia
$(eval $(call gb_Library_set_include,avmediagtk,\
$$(INCLUDE) \
$$(GTK4_CFLAGS) \
-I$(SRCDIR)/avmedia/source/inc \
-I$(SRCDIR)/avmedia/source/gstreamer \
))
$(eval $(call gb_Library_use_external,avmediagtk,boost_headers))
$(eval $(call gb_Library_add_cxxflags,avmediagtk,\
$$(GTK4_CFLAGS) \
))
$(eval $(call gb_Library_use_sdk_api,avmediagtk))
$(eval $(call gb_Library_use_libraries,avmediagtk,\