Skip to content

cogl: Don't install cogl-gtype-private.h - #869

Open
Fantu wants to merge 1 commit into
linuxmint:masterfrom
Fantu:cogl-dont-install-gtype-private-header
Open

Fantu wants to merge 1 commit into
linuxmint:masterfrom
Fantu:cogl-dont-install-gtype-private-header

Conversation

@Fantu

@Fantu Fantu commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

cogl/cogl/cogl-gtype-private.h is listed in cogl_nonintrospected_headers, so it gets installed in /usr/include/muffin/cogl/cogl/ together with the public headers, but it can't be used from outside:

  • it includes cogl-config.h and cogl-object-private.h, which are not installed, so any program including it fails to build
  • it only holds the COGL_GTYPE_DEFINE_* macros the cogl sources use to define their GTypes; none of the installed headers uses them
  • nothing outside cogl/ includes it

This PR drops it from the installed header list only. The file stays in cogl_sources, so the library build is unchanged, and the GIR is not affected either, since it is generated from cogl_introspected_headers.

The header comes from mutter, which kept installing it the same way (3.36 to 44) until the file went away with the port of cogl from CoglObject to GObject (mutter commit 2c4968fb4).

cogl-gtype-private.h is listed in cogl_nonintrospected_headers, so it
gets installed among the public headers, but it can't be used from
outside: it includes cogl-config.h and cogl-object-private.h, which are
not installed, so any program including it fails to build. It only holds
the COGL_GTYPE_DEFINE_* macros used by the cogl sources to define their
GTypes, none of the public headers uses them, and nothing outside cogl/
includes it.

Drop it from the installed header list. It stays in cogl_sources, so the
library build is unchanged, and the GIR is not affected either, being
generated from cogl_introspected_headers.

The header comes from mutter, which kept installing it the same way
until it went away with the port of cogl from CoglObject to GObject.

Assisted-by: Claude Code:claude-opus-5-5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant