-
Notifications
You must be signed in to change notification settings - Fork 262
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use standard gettext, following Geany core. #1183
base: master
Are you sure you want to change the base?
Conversation
Standard gettext should be used over glib's glue these days. AM_GLIB_GNU_GETTEXT and glib-gettextize are deprecated. This helps possible meson build (no concrete plans yet) as well as working with intltool is harder over there. gettext, on the other hand, is supported out of the box in meson. Beware, autopoint is a new dependency when building from git, should not affect tarballs. Also, it's already required by Geany core. Additionally, autogen.sh follows Geany's autogen.sh more closely, allowing for out-of-tree builds.
Requires running update-po to fix the build.
GETTEXT_PACKAGE=geany-plugins | ||
AC_SUBST(GETTEXT_PACKAGE) | ||
AC_DEFINE_UNQUOTED( | ||
[GETTEXT_PACKAGE], | ||
o [GETTEXT_PACKAGE], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a typo or some special M4 syntax?
@@ -16,9 +16,11 @@ AM_PROG_CC_C_O | |||
AC_DISABLE_STATIC | |||
AC_PROG_LIBTOOL | |||
|
|||
dnl i18n | |||
IT_PROG_INTLTOOL([0.35.0]) | |||
GP_I18N |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we don't call GP_I18N
here anymore, couldn't we remove build/i18n.m4
completely?
If so, we probably need to declare LOCALEDIR
here or still use GP_I18N
.
Apart from the comments above, LGBI. |
Standard gettext should be used over glib's glue these days.
AM_GLIB_GNU_GETTEXT and glib-gettextize are deprecated. This helps
possible meson build (no concrete plans yet) as well as working with
intltool is harder over there. gettext, on the other hand, is supported
out of the box in meson.
Beware, autopoint is a new dependency when building from git,
should not affect tarballs. Also, it's already required by Geany core.
Additionally, autogen.sh follows Geany's autogen.sh more closely,
allowing for out-of-tree builds.