Skip to content

add meson support#735

Open
mbkma wants to merge 1 commit into
masterfrom
meson
Open

add meson support#735
mbkma wants to merge 1 commit into
masterfrom
meson

Conversation

@mbkma
Copy link
Copy Markdown
Member

@mbkma mbkma commented Mar 22, 2026

No description provided.

@mbkma mbkma force-pushed the meson branch 5 times, most recently from 031ffde to 34c3063 Compare March 22, 2026 21:03
Comment thread meson.build
Comment on lines +89 to +96
have_introspection = false
gobject_introspection_dep = dependency('gobject-introspection-1.0',
version: '>= 0.9.3',
required: get_option('introspection'))
if gobject_introspection_dep.found()
have_introspection = true
config_h.set('HAVE_INTROSPECTION', 1)
endif
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to also add the same check but for girepository-2.0 and use it to set HAVE_GIREPOSITORY_2 in the config_h. This is almost certainly why the CI is failing.

Comment thread pluma/meson.build
Comment on lines +123 to +130
pluma_cflags = [
'-I' + meson.current_source_dir(),
'-DDATADIR="@0@"'.format(datadir),
'-DPLUMA_DATADIR="@0@"'.format(pkgdatadir),
'-DPLUMA_LOCALEDIR="@0@"'.format(localedir),
'-DLIBDIR="@0@"'.format(libdir),
'-DPLUMA_LIBDIR="@0@"'.format(pkglibdir),
]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You seem to be duplicating these in config_h as well - is this intentional?

Comment thread meson.build
Comment on lines +62 to +63
sm_dep = dependency('sm', version: '>= 1.0.0')
ice_dep = dependency('ice', version: '>= 1.0.0')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're not using these two...

Comment thread meson.build
Comment on lines +122 to +126
subdir('pluma')
subdir('data')
subdir('po')
subdir('plugins')
subdir('help')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're missing the pixmaps dir

Comment thread meson.build

gnome.post_install(
glib_compile_schemas: true,
update_desktop_database: find_program('update-desktop-database', required: false).found(),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just set this to true directly...

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.

2 participants