-
Notifications
You must be signed in to change notification settings - Fork 272
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
Rename GeanyLipsum to just Lipsum #472
Conversation
config_file = g_strconcat(geany->app->configdir, | ||
G_DIR_SEPARATOR_S, "plugins", G_DIR_SEPARATOR_S, | ||
"geanylipsum", G_DIR_SEPARATOR_S, "lipsum.conf", NULL); | ||
|
||
#ifndef G_OS_WIN32 | ||
/* We try only to move if we are on not Windows platform */ | ||
config_dir_old = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S, |
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.
could use g_build_filename()
instead for these.
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.
yep, would probably be neater
e565d8a
to
a129fc4
Compare
@@ -1,12 +1,12 @@ | |||
GeanyLipsum | |||
Lipsum | |||
----------- |
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.
could change the underline length too (not necessary though)
Otherwise LGBI |
if (dialogs_show_question( | ||
_("Renamed plugin detected!\n" | ||
"\n" | ||
"GeanyLipsum has been renamed to Lipsum -- you surely have " |
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.
as you will have already recognised
a129fc4
to
7f33eb6
Compare
@@ -0,0 +1,9 @@ | |||
AC_DEFUN([GP_CHECK_GEANYLIPSUM], |
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.
you could want to rename that (and in the configure.ac too).
Otherwise LGBI |
7f33eb6
to
9abd912
Compare
There is no need for Geany prefix on plugins.