summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorColin Walters <walters@verbum.org>2015-02-05 13:51:10 -0500
committerColin Walters <walters@verbum.org>2015-02-05 19:11:43 -0500
commitc394ae44e99e3edc6d3ae926fb00e58d63aefe4c (patch)
tree46992fea183702378909791bdb75017611399867
parent7e73247c57af28d7b6de71ec7d7e01e18f249922 (diff)
downloadgnome-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--.gitmodules3
-rwxr-xr-xautogen.sh7
-rw-r--r--configure.ac1
-rw-r--r--docs/reference/gnome-desktop3/Makefile.am1
-rw-r--r--libgnome-desktop/Makefile.am14
-rw-r--r--libgnome-desktop/gnome-desktop-thumbnail.c22
m---------libgnome-desktop/libgsystem0
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
diff --git a/autogen.sh b/autogen.sh
index f4fcbd82..b305efa9 100755
--- a/autogen.sh
+++ b/autogen.sh
@@ -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