[gparted] Increase minimum required gtkmm to 2.11.1 (#794253)
- From: Curtis Gedak <gedakc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gparted] Increase minimum required gtkmm to 2.11.1 (#794253)
- Date: Mon, 26 Mar 2018 16:49:42 +0000 (UTC)
commit 0ab2adb67fd3ef8e75698c596245b81762b8c7c2
Author: Mike Fleetwood <mike fleetwood googlemail com>
Date: Wed Mar 7 13:09:35 2018 +0000
Increase minimum required gtkmm to 2.11.1 (#794253)
Increase the minimum required version of gtkmm to 2.11.1, thus allowing
removal of:
* HAVE_SET_DEFAULT_ICON_NAME autoconf definition and associated
optional code.
* INSTALL_PIXMAPS_DIR automake conditional and associated make
instructions.
This is reversing these 3 commits, except for the higher minimum gtkmm
version:
1) a04210788399736ff7f097cb75650ebcbd0a4950
Only use Gtk::Window::set_default_icon_name method when available (#695279)
2) b09d6035cdca90debb145628b0c62a0213ee1225
Add fallback method for specifying GParted icon (#695279)
3) d6baac254677b7863af413a38f382e9a2e0252bd
Only install fallback icon when required (#695279)
Bug 794253 - Desupport RHEL / CentOS 5 and raise minimum required
versions to glibmm 2.14.0 and gtkmm 2.16.0
configure.ac | 24 +-----------------------
data/icons/Makefile.am | 27 ++-------------------------
src/Makefile.am | 1 -
src/Win_GParted.cc | 4 ----
4 files changed, 3 insertions(+), 53 deletions(-)
---
diff --git a/configure.ac b/configure.ac
index fe4d4e4..740476e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -219,7 +219,7 @@ AC_SUBST([GTHREAD_LIBS])
AC_SUBST([GTHREAD_CFLAGS])
dnl GTKMM
-PKG_CHECK_MODULES([GTKMM], [gtkmm-2.4 > 2.8])
+PKG_CHECK_MODULES([GTKMM], [gtkmm-2.4 >= 2.11.1])
AC_SUBST([GTKMM_LIBS])
AC_SUBST([GTKMM_CFLAGS])
@@ -228,28 +228,6 @@ dnl Check for glibmm minimum required version.
PKG_CHECK_MODULES([GLIBMM], [glibmm-2.4 >= 2.14.0])
-dnl Check for gtkmm >= 2.11.1 to determine availability of Gtk::Window::set_default_icon_name()
-dnl NOTE:
-dnl The documentation says that set_default_icon_name() is available in gtkmm >= 2.6,
-dnl however the code reveals that it is only available in gtkmm >= 2.11.1.
-dnl * gtkmm GTK::Window Class Reference
-dnl https://developer.gnome.org/gtkmm/3.6/classGtk_1_1Window.html#a533d03e9b92d8ccd142ab3a44005cae4
-dnl * gtkmm NEWS file
-dnl https://git.gnome.org/browse/gtkmm/tree/NEWS?h=gtkmm-2.14.0#n565
-AC_MSG_CHECKING([for Gtk::Window::set_default_icon_name() method])
-PKG_CHECK_EXISTS(
- [gtkmm-2.4 >= 2.11.1],
- [AC_DEFINE([HAVE_SET_DEFAULT_ICON_NAME], 1,
- [Define to 1 if gtkmm-2.4 provides Gtk::Window::set_default_icon_name() method.])
- AM_CONDITIONAL([INSTALL_PIXMAPS_DIR], false)
- AC_MSG_RESULT([yes])
- ],
- [AM_CONDITIONAL([INSTALL_PIXMAPS_DIR], true)
- AC_MSG_RESULT([no])
- ]
-)
-
-
dnl GTKMM 2.16 needed for gtk_show_uri()
AC_MSG_CHECKING([for gtk_show_uri function])
PKG_CHECK_EXISTS(
diff --git a/data/icons/Makefile.am b/data/icons/Makefile.am
index 9572d17..d98bd10 100644
--- a/data/icons/Makefile.am
+++ b/data/icons/Makefile.am
@@ -16,16 +16,6 @@ public_icons = \
private_icons = \
$(NULL)
-# Name of the fallback icon to install in /usr/share/pixmaps [1]
-# --
-# The GNOME 2 desktop wants a 16x16 "Menu" type icon [2] for the title
-# bar and the open window list, and a 32x32 icon for the Alt-Tab task
-# switcher. GParted only specifies a single icon file in the fallback
-# case so just use the 16x16 one and let it be scaled as required.
-# [1] https://developer.gnome.org/icon-theme-spec/
-# [2] https://developer.gnome.org/hig-book/2.32/hig-book.html#icons
-pixmap = hicolor_apps_16x16_gparted.png
-
noinst_DATA = \
$(NULL)
@@ -36,12 +26,6 @@ EXTRA_DIST = \
$(noinst_DATA) \
$(NULL)
-# Only install fallback icon to /usr/share/pixmaps if required
-if INSTALL_PIXMAPS_DIR
-INSTALL_PIXMAP = install-pixmap
-UNINSTALL_PIXMAP = uninstall-pixmap
-endif
-
###############################################################################
gtk_update_icon_cache = gtk-update-icon-cache -f -t
@@ -93,13 +77,6 @@ uninstall-icons:
rm -f $(DESTDIR)$(pkgdatadir)/icons/$$THEME/$$SIZE/$$CONTEXT/$$ICONFILE; \
done
-install-pixmap:
- mkdir -p $(DESTDIR)$(datadir)/pixmaps; \
- $(INSTALL_DATA) $(srcdir)/$(pixmap) $(DESTDIR)$(datadir)/pixmaps/gparted.png
-
-uninstall-pixmap:
- rm -f $(DESTDIR)$(datadir)/pixmaps/gparted.png
-
-install-data-local: install-icons $(INSTALL_PIXMAP) update-icon-cache
+install-data-local: install-icons update-icon-cache
-uninstall-local: uninstall-icons $(UNINSTALL_PIXMAP) update-icon-cache
+uninstall-local: uninstall-icons update-icon-cache
diff --git a/src/Makefile.am b/src/Makefile.am
index 2e8a03f..4314657 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -2,7 +2,6 @@ AM_CPPFLAGS = \
-I$(top_srcdir)/include \
$(GTHREAD_CFLAGS) \
$(GTKMM_CFLAGS) \
- -DGNOME_ICONDIR=\""$(datadir)/pixmaps"\" \
-DGNOMELOCALEDIR=\""$(datadir)/locale"\"
AM_CFLAGS = -Wall
diff --git a/src/Win_GParted.cc b/src/Win_GParted.cc
index edb81a3..e3ecd69 100644
--- a/src/Win_GParted.cc
+++ b/src/Win_GParted.cc
@@ -85,11 +85,7 @@ Win_GParted::Win_GParted( const std::vector<Glib::ustring> & user_devices )
try
{
-#ifdef HAVE_SET_DEFAULT_ICON_NAME
this ->set_default_icon_name( "gparted" ) ;
-#else
- this ->set_icon_from_file( GNOME_ICONDIR "/gparted.png" ) ;
-#endif
}
catch ( Glib::Exception & e )
{
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]