Accepting request 344817 from home:Zaitor
New stable OBS-URL: https://build.opensuse.org/request/show/344817 OBS-URL: https://build.opensuse.org/package/show/GNOME:Factory/gtk3?expand=0&rev=209
This commit is contained in:
parent
39df210e17
commit
85aeddc35c
@ -1,3 +0,0 @@
|
||||
version https://git-lfs.github.com/spec/v1
|
||||
oid sha256:5400dcf280d28d24606f33d59ed48c717f7d3db425d4b6fb52e8002f0c76c7eb
|
||||
size 18912812
|
3
gtk+-3.18.5.tar.xz
Normal file
3
gtk+-3.18.5.tar.xz
Normal file
@ -0,0 +1,3 @@
|
||||
version https://git-lfs.github.com/spec/v1
|
||||
oid sha256:107aeb9a4244ce3c044becdd6dffc32d83202595181597180d4c736302a71852
|
||||
size 18912848
|
@ -1,97 +0,0 @@
|
||||
From 0cd4e7ec43a07ae8dc77231bef25ee792ddda529 Mon Sep 17 00:00:00 2001
|
||||
From: Carlos Soriano <csoriano@gnome.org>
|
||||
Date: Wed, 14 Oct 2015 21:42:16 +0200
|
||||
Subject: gtkplacessidebar: improve heuristics for external drives
|
||||
|
||||
Since the change to use GtkPlacesView we don't want to show
|
||||
internal storage on the sidebar.
|
||||
|
||||
In our case we were checking for drive_can_eject and
|
||||
drive_is_media_removable.
|
||||
|
||||
However for some external hard drives it's reported that they
|
||||
are not ejectable nor the have removable media. So the only
|
||||
attribute that they have different from internal drives is that
|
||||
they can be stopped.
|
||||
So check for if the drive can be stopped to decide if it is
|
||||
external or internal.
|
||||
|
||||
On the way realized we don't need to check for the mounts associated
|
||||
with the volume to know if the volume can be ejected or not. So remove
|
||||
that code.
|
||||
|
||||
https://bugzilla.gnome.org/show_bug.cgi?id=756589
|
||||
---
|
||||
gtk/gtkplacessidebar.c | 27 ++++++++++-----------------
|
||||
1 file changed, 10 insertions(+), 17 deletions(-)
|
||||
|
||||
diff --git a/gtk/gtkplacessidebar.c b/gtk/gtkplacessidebar.c
|
||||
index d8a2190..42179c3 100644
|
||||
--- a/gtk/gtkplacessidebar.c
|
||||
+++ b/gtk/gtkplacessidebar.c
|
||||
@@ -823,36 +823,29 @@ out:
|
||||
}
|
||||
|
||||
static gboolean
|
||||
-is_removable_volume (GVolume *volume)
|
||||
+is_external_volume (GVolume *volume)
|
||||
{
|
||||
- gboolean is_removable;
|
||||
+ gboolean is_external;
|
||||
GDrive *drive;
|
||||
- GMount *mount;
|
||||
gchar *id;
|
||||
|
||||
drive = g_volume_get_drive (volume);
|
||||
- mount = g_volume_get_mount (volume);
|
||||
id = g_volume_get_identifier (volume, G_VOLUME_IDENTIFIER_KIND_CLASS);
|
||||
|
||||
- is_removable = g_volume_can_eject (volume);
|
||||
+ is_external = g_volume_can_eject (volume);
|
||||
|
||||
/* NULL volume identifier only happens on removable devices */
|
||||
- is_removable |= !id;
|
||||
+ is_external |= !id;
|
||||
|
||||
if (drive)
|
||||
- {
|
||||
- is_removable |= g_drive_can_eject (drive);
|
||||
- is_removable |= g_drive_is_media_removable (drive);
|
||||
- }
|
||||
-
|
||||
- if (mount)
|
||||
- is_removable |= (g_mount_can_eject (mount) && !g_mount_can_unmount (mount));
|
||||
+ is_external |= g_drive_can_eject (drive) ||
|
||||
+ g_drive_is_media_removable (drive) ||
|
||||
+ g_drive_can_stop (drive);
|
||||
|
||||
g_clear_object (&drive);
|
||||
- g_clear_object (&mount);
|
||||
g_free (id);
|
||||
|
||||
- return is_removable;
|
||||
+ return is_external;
|
||||
}
|
||||
|
||||
static void
|
||||
@@ -991,7 +984,7 @@ update_places (GtkPlacesSidebar *sidebar)
|
||||
}
|
||||
g_free (identifier);
|
||||
|
||||
- if (sidebar->show_other_locations && !is_removable_volume (volume))
|
||||
+ if (sidebar->show_other_locations && !is_external_volume (volume))
|
||||
{
|
||||
g_object_unref (volume);
|
||||
continue;
|
||||
@@ -1095,7 +1088,7 @@ update_places (GtkPlacesSidebar *sidebar)
|
||||
}
|
||||
g_free (identifier);
|
||||
|
||||
- if (sidebar->show_other_locations && !is_removable_volume (volume))
|
||||
+ if (sidebar->show_other_locations && !is_external_volume (volume))
|
||||
{
|
||||
g_object_unref (volume);
|
||||
continue;
|
||||
--
|
||||
cgit v0.11.2
|
||||
|
13
gtk3.changes
13
gtk3.changes
@ -1,3 +1,16 @@
|
||||
-------------------------------------------------------------------
|
||||
Mon Nov 16 21:32:02 UTC 2015 - zaitor@opensuse.org
|
||||
|
||||
- Update to version 3.18.5:
|
||||
+ GtkFileChooser:
|
||||
- Make sure external drives show up either in the sidebar or
|
||||
the places view.
|
||||
- Ignore double-click events.
|
||||
+ Avoid some crashes in CSS parsing error handling.
|
||||
- Drop
|
||||
gtk3-gtkplacessidebar-improve-heuristics-external-drives.patch:
|
||||
Fixed upstream.
|
||||
|
||||
-------------------------------------------------------------------
|
||||
Sat Nov 14 20:04:16 UTC 2015 - zaitor@opensuse.org
|
||||
|
||||
|
@ -29,7 +29,7 @@
|
||||
|
||||
Name: gtk3
|
||||
%define _name gtk+
|
||||
Version: 3.18.4
|
||||
Version: 3.18.5
|
||||
Release: 0
|
||||
Summary: The GTK+ toolkit library (version 3)
|
||||
License: LGPL-2.1+
|
||||
@ -47,8 +47,6 @@ Patch1: gtk3-path-local.patch
|
||||
## PATCH-DISABLED gtk3-bnc130159-bgo319483-async-font-selection.patch - Upstream bug was closed as obsolete in 2011, lets see if anyone complains.
|
||||
# PATCH-FIX-UPSTREAM gtk3-bnc130159-bgo319483-async-font-selection.patch bnc130159 bgo319483 federico@novell.com - Load fonts asynchronously in GtkFontSelection to make it appear faster for CJK languages
|
||||
Patch3: gtk3-bnc130159-bgo319483-async-font-selection.patch
|
||||
# PATCH-FIX-UPSTREAM gtk3-gtkplacessidebar-improve-heuristics-external-drives.patch bgo#756589 zaitor@opensuse.org -- Add a missed commit for the sidebar refactor
|
||||
Patch4: gtk3-gtkplacessidebar-improve-heuristics-external-drives.patch
|
||||
BuildRequires: cups-devel >= 1.2
|
||||
BuildRequires: docbook-xsl-stylesheets
|
||||
BuildRequires: fdupes
|
||||
@ -340,7 +338,6 @@ cp -a %{S:1} .
|
||||
%patch1 -p0
|
||||
## PATCH-DISABLED - Upstream bug was closed as obsolete in 2011, as there was a new fontchooser, lets disable the patch and see if anyone complains.
|
||||
#%%patch3 -p1
|
||||
%patch4 -p1
|
||||
|
||||
%build
|
||||
# Needed for patch1
|
||||
|
Loading…
Reference in New Issue
Block a user