diff options
author | Colin Walters <walters@verbum.org> | 2015-02-05 13:51:10 -0500 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2015-02-05 19:11:43 -0500 |
commit | c394ae44e99e3edc6d3ae926fb00e58d63aefe4c (patch) | |
tree | 46992fea183702378909791bdb75017611399867 | |
parent | 7e73247c57af28d7b6de71ec7d7e01e18f249922 (diff) | |
download | gnome-desktop-c394ae44e99e3edc6d3ae926fb00e58d63aefe4c.tar.gz |
Drop libgsystem submodule, stop using O_NOATIME
The entire rationale for adding this _noatime function was fairly
bogus.
See https://bugzilla.gnome.org/show_bug.cgi?id=680326
But the Linux kernel doesn't permit non-root users to use it for
root-owned files - which is the default for real world GNOME
installations. We end up doing twice the syscall traffic with no
benefit at all.
The patch does work in jhbuild, but honestly the relatime handling is
generally OK for this, and where it's not, change your system to mount
O_NOATIME.
https://bugzilla.gnome.org/show_bug.cgi?id=708453
-rw-r--r-- | .gitmodules | 3 | ||||
-rwxr-xr-x | autogen.sh | 7 | ||||
-rw-r--r-- | configure.ac | 1 | ||||
-rw-r--r-- | docs/reference/gnome-desktop3/Makefile.am | 1 | ||||
-rw-r--r-- | libgnome-desktop/Makefile.am | 14 | ||||
-rw-r--r-- | libgnome-desktop/gnome-desktop-thumbnail.c | 22 | ||||
m--------- | libgnome-desktop/libgsystem | 0 |
7 files changed, 7 insertions, 41 deletions
diff --git a/.gitmodules b/.gitmodules index 5e4c4f62..e69de29b 100644 --- a/.gitmodules +++ b/.gitmodules @@ -1,3 +0,0 @@ -[submodule "libgnome-desktop/libgsystem"] - path = libgnome-desktop/libgsystem - url = git://git.gnome.org/libgsystem @@ -20,11 +20,4 @@ which gnome-autogen.sh || { exit 1 } -# Fetch submodules if needed -if test ! -f libgnome-desktop/libgsystem/README; then - echo "+ Setting up submodules" - git submodule init -fi -git submodule update - . gnome-autogen.sh diff --git a/configure.ac b/configure.ac index 5f173c24..26567622 100644 --- a/configure.ac +++ b/configure.ac @@ -133,7 +133,6 @@ AC_CHECK_LIB([rt], [clock_gettime], [LIBRT="-lrt"], [LIBRT=""]) AC_SUBST(LIBRT) AC_CHECK_FUNCS([openat]) -AM_CONDITIONAL([USE_LIBGSYSTEM], [test x$ac_cv_func_openat = xyes]) dnl we need x11 for GnomeBG diff --git a/docs/reference/gnome-desktop3/Makefile.am b/docs/reference/gnome-desktop3/Makefile.am index d4f0779e..65d01007 100644 --- a/docs/reference/gnome-desktop3/Makefile.am +++ b/docs/reference/gnome-desktop3/Makefile.am @@ -48,7 +48,6 @@ CFILE_GLOB=$(top_srcdir)/libgnome-desktop/*.c # Header files to ignore when scanning. # e.g. IGNORE_HFILES=gtkdebug.h gtkintl.h IGNORE_HFILES= \ - libgsystem \ meta-dbus-idle-monitor.h \ meta-dbus-xrandr.h \ meta-xrandr-shared.h diff --git a/libgnome-desktop/Makefile.am b/libgnome-desktop/Makefile.am index 2389eb18..d6cac820 100644 --- a/libgnome-desktop/Makefile.am +++ b/libgnome-desktop/Makefile.am @@ -7,7 +7,6 @@ lib_LTLIBRARIES = libgnome-desktop-3.la AM_CPPFLAGS = \ -I$(top_srcdir) \ - -I$(srcdir)/libgsystem \ $(GNOME_DESKTOP_CFLAGS) \ $(XLIB_CFLAGS) \ -DG_LOG_DOMAIN=\"GnomeDesktop\" \ @@ -20,15 +19,6 @@ AM_CPPFLAGS = \ AM_CFLAGS = $(WARN_CFLAGS) -if USE_LIBGSYSTEM -libgsystem_srcpath := libgsystem -libgsystem_cflags = $(GNOME_DESKTOP_CFLAGS) -libgsystem_libs = $(GNOME_DESKTOP_LIBS) -include libgsystem/Makefile-libgsystem.am - -noinst_LTLIBRARIES += libgsystem.la -endif - introspection_sources = \ gnome-desktop-thumbnail.c \ gnome-thumbnail-pixbuf-utils.c \ @@ -84,10 +74,6 @@ libgnome_desktop_3_la_LIBADD = \ $(LIBRT) \ $(NULL) -if USE_LIBGSYSTEM -libgnome_desktop_3_la_LIBADD += libgsystem.la -endif - libgnome_desktop_3_la_LDFLAGS = \ -version-info $(LT_VERSION) \ -export-symbols-regex "^gnome_.*" \ diff --git a/libgnome-desktop/gnome-desktop-thumbnail.c b/libgnome-desktop/gnome-desktop-thumbnail.c index 7a1963bd..fa1904b0 100644 --- a/libgnome-desktop/gnome-desktop-thumbnail.c +++ b/libgnome-desktop/gnome-desktop-thumbnail.c @@ -145,9 +145,6 @@ #define GNOME_DESKTOP_USE_UNSTABLE_API #include "gnome-desktop-thumbnail.h" #include <glib/gstdio.h> -#ifdef HAVE_OPENAT -#include <libgsystem.h> -#endif #define SECONDS_BETWEEN_STATS 10 @@ -487,18 +484,13 @@ _gdk_pixbuf_new_from_uri_at_scale (const char *uri, } if (input_stream == NULL) { -#ifdef HAVE_OPENAT - if (g_file_is_native (file)) - input_stream = gs_file_read_noatime (file, NULL, &error); - else -#endif - input_stream = G_INPUT_STREAM (g_file_read (file, NULL, &error)); - if (input_stream == NULL) { - g_warning ("Unable to create an input stream for %s: %s", uri, error->message); - g_clear_error (&error); - g_object_unref (file); - return NULL; - } + input_stream = G_INPUT_STREAM (g_file_read (file, NULL, &error)); + if (input_stream == NULL) { + g_warning ("Unable to create an input stream for %s: %s", uri, error->message); + g_clear_error (&error); + g_object_unref (file); + return NULL; + } } has_frame = FALSE; diff --git a/libgnome-desktop/libgsystem b/libgnome-desktop/libgsystem deleted file mode 160000 -Subproject 1bd16c24706b9053d7d9c509c137fe963cd5e31 |