Fix possible clash of config headers
Review Request #2707 — Created Oct. 25, 2023 and submitted
Both GLib and GStreamer have a config file called
config.h
and use theHAVE_CONFIG_H
macro to conditionally include them.Normally, this works out fine, but if they are subprojects, we may accidentally trigger including a different
config.h
. This is normally harmless, but it causes a bunch of warnings because we defineGETTEXT_PACKAGE
on the command-line, and it won't match the one in a different project.However, we only include this file in source files, not headers, and it's always created, so there's no need to conditionally include it or define the macro. We can also rename it so it can't clash with other instances as well.
Compiled with subprojects and stopped seeing many warnings about
GETTEXT_PACKAGE
and other generic variables being re-defined due to GStreamer'sconfig.h
.
Summary | ID |
---|---|
bc772b1536fe0afa00e2462bb1417eb022a289e6 |