Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] tracker3{-miners} update (bugfix)
@ 2020-12-22 22:37 st3r4g
  2020-12-22 23:31 ` [PR PATCH] [Updated] " st3r4g
  2020-12-25 14:01 ` [PR PATCH] [Merged]: " q66
  0 siblings, 2 replies; 3+ messages in thread
From: st3r4g @ 2020-12-22 22:37 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 590 bytes --]

There is a new pull request by st3r4g against master on the void-packages repository

https://github.com/st3r4g/void-packages tracker3-3.0.3
https://github.com/void-linux/void-packages/pull/27363

tracker3{-miners} update (bugfix)
Today I noticed a very annoying bug. Moving a text file in my home resulted in processes consuming a large amount of cpu and spamming gdm log. I can reproduce by:
```
echo x > x
mv x y
```
and the cpu drain/gdm log spam goes away only after:
`tracker3 reset -s`

A patch file from https://github.com/void-linux/void-packages/pull/27363.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-tracker3-3.0.3-27363.patch --]
[-- Type: text/x-diff, Size: 6956 bytes --]

From e151d7af5c6392e66cf85f87b8c487ba901cbc4a Mon Sep 17 00:00:00 2001
From: Stefano Ragni <st3r4g@protonmail.com>
Date: Tue, 22 Dec 2020 23:29:44 +0100
Subject: [PATCH 1/2] tracker3: update to 3.0.2

---
 srcpkgs/tracker3/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/tracker3/template b/srcpkgs/tracker3/template
index 1c2100f26ca..b83fe874626 100644
--- a/srcpkgs/tracker3/template
+++ b/srcpkgs/tracker3/template
@@ -1,6 +1,6 @@
 # Template file for 'tracker3'
 pkgname=tracker3
-version=3.0.0
+version=3.0.2
 revision=1
 wrksrc=tracker-${version}
 build_style=meson
@@ -17,7 +17,7 @@ maintainer="q66 <daniel@octaforge.org>"
 license="GPL-2.0-or-later, LGPL-2.1-or-later"
 homepage="https://live.gnome.org/Tracker"
 distfiles="${GNOME_SITE}/tracker/${version%.*}/tracker-${version}.tar.xz"
-checksum=70864515f5752b0596f9c442d7e86585734f42b82a9233e55dae6ac2b0d33837
+checksum=a60041cf47947ebbcfbba926d3b470bfdead21f3c12ce7aa93eaf1969a185c60
 
 if [ "$CROSS_BUILD" ]; then
 	hostmakedepends+=" dbus"

From 3a4ecdaf8b36315d746ba243237f5525c4212e6d Mon Sep 17 00:00:00 2001
From: Stefano Ragni <st3r4g@protonmail.com>
Date: Tue, 22 Dec 2020 23:29:08 +0100
Subject: [PATCH 2/2] tracker3-miners: update to 3.0.3

---
 srcpkgs/tracker3-miners/patches/build.patch   |  25 -----
 .../patches/dont-use-malloc-trim.patch        | 104 ------------------
 srcpkgs/tracker3-miners/template              |   4 +-
 3 files changed, 2 insertions(+), 131 deletions(-)
 delete mode 100644 srcpkgs/tracker3-miners/patches/build.patch
 delete mode 100644 srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch

diff --git a/srcpkgs/tracker3-miners/patches/build.patch b/srcpkgs/tracker3-miners/patches/build.patch
deleted file mode 100644
index d58a0d8c0f8..00000000000
--- a/srcpkgs/tracker3-miners/patches/build.patch
+++ /dev/null
@@ -1,25 +0,0 @@
-From 8d709c3b55dc92621d5871fb429a02c78cc4b2d8 Mon Sep 17 00:00:00 2001
-From: Daniel Kolesa <daniel@octaforge.org>
-Date: Wed, 16 Sep 2020 23:57:32 +0200
-Subject: [PATCH] fix build
-
----
- src/libtracker-miner/meson.build | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git src/libtracker-miner/meson.build src/libtracker-miner/meson.build
-index a226623..9eacee3 100644
---- src/libtracker-miner/meson.build
-+++ src/libtracker-miner/meson.build
-@@ -40,7 +40,7 @@ endif
- 
- libtracker_miner = library(
-     'tracker-miner-' + tracker_api_version,
--    miner_enums[0], miner_enums[1], miner_sources,
-+    miner_enums[0], miner_enums[1], tracker_miners_common_enum_header, miner_sources,
-     c_args: tracker_c_args,
-     install: true,
-     install_dir: tracker_internal_libs_dir,
--- 
-2.28.0
-
diff --git a/srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch b/srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch
deleted file mode 100644
index 3f42a912097..00000000000
--- a/srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch
+++ /dev/null
@@ -1,104 +0,0 @@
-Reason: This reverts 191cc5218b53fba85baec1c8c17bb246a74044c0 and
-ccb0b4ebbff4dfacf17ea67ce56bb27c39741811. malloc_trim is glibc specific
-API, so this causes the build to fail on musl.
-Upstream: https://gitlab.gnome.org/GNOME/tracker-miners/-/issues/135
-diff --git a/src/miners/fs/tracker-main.c b/src/miners/fs/tracker-main.c
-index c60070d..220916d 100644
---- src/miners/fs/tracker-main.c
-+++ src/miners/fs/tracker-main.c
-@@ -57,7 +57,6 @@
- static GMainLoop *main_loop;
- static GDBusProxy *index_proxy;
- static GPtrArray *proxy_folders;
--static guint cleanup_id;
- 
- static gint initial_sleep = -1;
- static gboolean no_daemon;
-@@ -433,37 +432,6 @@ miner_start (TrackerMiner  *miner,
- 	                                           miner);
- }
- 
--static gboolean
--cleanup_cb (gpointer user_data)
--{
--	/* Reclaim as much memory as possible */
--	malloc_trim (0);
--
--	cleanup_id = 0;
--
--	return G_SOURCE_REMOVE;
--}
--
--#if GLIB_CHECK_VERSION (2, 64, 0)
--static void
--on_low_memory (GMemoryMonitor            *monitor,
--               GMemoryMonitorWarningLevel level,
--               gpointer                   user_data)
--{
--	if (level > G_MEMORY_MONITOR_WARNING_LEVEL_LOW)
--		malloc_trim (0);
--}
--#endif
--
--static void
--miner_started_cb (TrackerMinerFS *fs)
--{
--	if (cleanup_id) {
--		g_source_remove (cleanup_id);
--		cleanup_id = 0;
--	}
--}
--
- static void
- miner_finished_cb (TrackerMinerFS *fs,
-                    gdouble         seconds_elapsed,
-@@ -483,8 +451,6 @@ miner_finished_cb (TrackerMinerFS *fs,
- 							 TRUE);
- 	}
- 
--	cleanup_id = g_timeout_add_seconds (30, cleanup_cb, NULL);
--
- 	/* We're not sticking around for file updates, so stop
- 	 * the mainloop and exit.
- 	 */
-@@ -941,9 +907,6 @@ main (gint argc, gchar *argv[])
- 	TrackerEndpointDBus *endpoint;
- 	TrackerDomainOntology *domain_ontology;
- 	GCancellable *cancellable;
--#if GLIB_CHECK_VERSION (2, 64, 0)
--	GMemoryMonitor *memory_monitor;
--#endif
- 	gchar *domain_name, *dbus_name;
- 
- 	main_loop = NULL;
-@@ -1129,18 +1092,10 @@ main (gint argc, gchar *argv[])
- 		tracker_miner_files_set_mtime_checking (TRACKER_MINER_FILES (miner_files), do_mtime_checking);
- 	}
- 
--	g_signal_connect (miner_files, "started",
--			  G_CALLBACK (miner_started_cb),
--			  NULL);
- 	g_signal_connect (miner_files, "finished",
- 			  G_CALLBACK (miner_finished_cb),
- 			  NULL);
- 
--#if GLIB_CHECK_VERSION (2, 64, 0)
--	memory_monitor = g_memory_monitor_dup_default ();
--	g_signal_connect (memory_monitor, "low-memory-warning", on_low_memory, NULL);
--#endif
--
- 	/* Preempt creation of graphs */
- 	tracker_sparql_connection_update_async (tracker_miner_get_connection (miner_files),
- 	                                        "CREATE SILENT GRAPH tracker:FileSystem; "
-@@ -1183,11 +1138,6 @@ main (gint argc, gchar *argv[])
- 	tracker_sparql_connection_close (sparql_conn);
- 	g_object_unref (sparql_conn);
- 
--#if GLIB_CHECK_VERSION (2, 64, 0)
--	g_signal_handlers_disconnect_by_func (memory_monitor, on_low_memory, NULL);
--	g_object_unref (memory_monitor);
--#endif
--
- 	g_print ("\nOK\n\n");
- 
- 	return EXIT_SUCCESS;
diff --git a/srcpkgs/tracker3-miners/template b/srcpkgs/tracker3-miners/template
index 0bf14b48479..7b773dfb494 100644
--- a/srcpkgs/tracker3-miners/template
+++ b/srcpkgs/tracker3-miners/template
@@ -1,6 +1,6 @@
 # Template file for 'tracker3-miners'
 pkgname=tracker3-miners
-version=3.0.0
+version=3.0.3
 revision=1
 wrksrc=tracker-miners-${version}
 build_style=meson
@@ -26,7 +26,7 @@ maintainer="q66 <daniel@octaforge.org>"
 license="GPL-2.0-or-later"
 homepage="https://developer.gnome.org/libtracker-miner/stable"
 distfiles="${GNOME_SITE}/tracker-miners/${version%.*}/tracker-miners-${version}.tar.xz"
-checksum=bb481a7c23c5def4f215627a0630fbe9d1c64f3319825859d7a73645738f4042
+checksum=eb6c956aec200ec441e20f0c8a7e5baedd59532299c6f735774f30db9bff949d
 
 # relies on unsupported ops in chroot
 do_check() {

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PR PATCH] [Updated] tracker3{-miners} update (bugfix)
  2020-12-22 22:37 [PR PATCH] tracker3{-miners} update (bugfix) st3r4g
@ 2020-12-22 23:31 ` st3r4g
  2020-12-25 14:01 ` [PR PATCH] [Merged]: " q66
  1 sibling, 0 replies; 3+ messages in thread
From: st3r4g @ 2020-12-22 23:31 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 619 bytes --]

There is an updated pull request by st3r4g against master on the void-packages repository

https://github.com/st3r4g/void-packages tracker3-3.0.3
https://github.com/void-linux/void-packages/pull/27363

tracker3{-miners} update (bugfix)
Today I noticed a very bad bug. Moving a text file in my home resulted in processes consuming a large amount of cpu and spamming gdm log. I can reproduce by:
```
echo x > x
mv x y
```
and the cpu drain/gdm log spam goes away only after:
`tracker3 reset -s`

Updating seems to fix it.

A patch file from https://github.com/void-linux/void-packages/pull/27363.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-tracker3-3.0.3-27363.patch --]
[-- Type: text/x-diff, Size: 5769 bytes --]

From e151d7af5c6392e66cf85f87b8c487ba901cbc4a Mon Sep 17 00:00:00 2001
From: Stefano Ragni <st3r4g@protonmail.com>
Date: Tue, 22 Dec 2020 23:29:44 +0100
Subject: [PATCH 1/2] tracker3: update to 3.0.2

---
 srcpkgs/tracker3/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/tracker3/template b/srcpkgs/tracker3/template
index 1c2100f26ca..b83fe874626 100644
--- a/srcpkgs/tracker3/template
+++ b/srcpkgs/tracker3/template
@@ -1,6 +1,6 @@
 # Template file for 'tracker3'
 pkgname=tracker3
-version=3.0.0
+version=3.0.2
 revision=1
 wrksrc=tracker-${version}
 build_style=meson
@@ -17,7 +17,7 @@ maintainer="q66 <daniel@octaforge.org>"
 license="GPL-2.0-or-later, LGPL-2.1-or-later"
 homepage="https://live.gnome.org/Tracker"
 distfiles="${GNOME_SITE}/tracker/${version%.*}/tracker-${version}.tar.xz"
-checksum=70864515f5752b0596f9c442d7e86585734f42b82a9233e55dae6ac2b0d33837
+checksum=a60041cf47947ebbcfbba926d3b470bfdead21f3c12ce7aa93eaf1969a185c60
 
 if [ "$CROSS_BUILD" ]; then
 	hostmakedepends+=" dbus"

From 313021a3e16a4fb71875d49b0a872f7b7dfaa342 Mon Sep 17 00:00:00 2001
From: Stefano Ragni <st3r4g@protonmail.com>
Date: Wed, 23 Dec 2020 00:30:14 +0100
Subject: [PATCH 2/2] tracker3-miners: update to 3.0.3

---
 .../patches/dont-use-malloc-trim.patch        | 104 ------------------
 srcpkgs/tracker3-miners/template              |   4 +-
 2 files changed, 2 insertions(+), 106 deletions(-)
 delete mode 100644 srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch

diff --git a/srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch b/srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch
deleted file mode 100644
index 3f42a912097..00000000000
--- a/srcpkgs/tracker3-miners/patches/dont-use-malloc-trim.patch
+++ /dev/null
@@ -1,104 +0,0 @@
-Reason: This reverts 191cc5218b53fba85baec1c8c17bb246a74044c0 and
-ccb0b4ebbff4dfacf17ea67ce56bb27c39741811. malloc_trim is glibc specific
-API, so this causes the build to fail on musl.
-Upstream: https://gitlab.gnome.org/GNOME/tracker-miners/-/issues/135
-diff --git a/src/miners/fs/tracker-main.c b/src/miners/fs/tracker-main.c
-index c60070d..220916d 100644
---- src/miners/fs/tracker-main.c
-+++ src/miners/fs/tracker-main.c
-@@ -57,7 +57,6 @@
- static GMainLoop *main_loop;
- static GDBusProxy *index_proxy;
- static GPtrArray *proxy_folders;
--static guint cleanup_id;
- 
- static gint initial_sleep = -1;
- static gboolean no_daemon;
-@@ -433,37 +432,6 @@ miner_start (TrackerMiner  *miner,
- 	                                           miner);
- }
- 
--static gboolean
--cleanup_cb (gpointer user_data)
--{
--	/* Reclaim as much memory as possible */
--	malloc_trim (0);
--
--	cleanup_id = 0;
--
--	return G_SOURCE_REMOVE;
--}
--
--#if GLIB_CHECK_VERSION (2, 64, 0)
--static void
--on_low_memory (GMemoryMonitor            *monitor,
--               GMemoryMonitorWarningLevel level,
--               gpointer                   user_data)
--{
--	if (level > G_MEMORY_MONITOR_WARNING_LEVEL_LOW)
--		malloc_trim (0);
--}
--#endif
--
--static void
--miner_started_cb (TrackerMinerFS *fs)
--{
--	if (cleanup_id) {
--		g_source_remove (cleanup_id);
--		cleanup_id = 0;
--	}
--}
--
- static void
- miner_finished_cb (TrackerMinerFS *fs,
-                    gdouble         seconds_elapsed,
-@@ -483,8 +451,6 @@ miner_finished_cb (TrackerMinerFS *fs,
- 							 TRUE);
- 	}
- 
--	cleanup_id = g_timeout_add_seconds (30, cleanup_cb, NULL);
--
- 	/* We're not sticking around for file updates, so stop
- 	 * the mainloop and exit.
- 	 */
-@@ -941,9 +907,6 @@ main (gint argc, gchar *argv[])
- 	TrackerEndpointDBus *endpoint;
- 	TrackerDomainOntology *domain_ontology;
- 	GCancellable *cancellable;
--#if GLIB_CHECK_VERSION (2, 64, 0)
--	GMemoryMonitor *memory_monitor;
--#endif
- 	gchar *domain_name, *dbus_name;
- 
- 	main_loop = NULL;
-@@ -1129,18 +1092,10 @@ main (gint argc, gchar *argv[])
- 		tracker_miner_files_set_mtime_checking (TRACKER_MINER_FILES (miner_files), do_mtime_checking);
- 	}
- 
--	g_signal_connect (miner_files, "started",
--			  G_CALLBACK (miner_started_cb),
--			  NULL);
- 	g_signal_connect (miner_files, "finished",
- 			  G_CALLBACK (miner_finished_cb),
- 			  NULL);
- 
--#if GLIB_CHECK_VERSION (2, 64, 0)
--	memory_monitor = g_memory_monitor_dup_default ();
--	g_signal_connect (memory_monitor, "low-memory-warning", on_low_memory, NULL);
--#endif
--
- 	/* Preempt creation of graphs */
- 	tracker_sparql_connection_update_async (tracker_miner_get_connection (miner_files),
- 	                                        "CREATE SILENT GRAPH tracker:FileSystem; "
-@@ -1183,11 +1138,6 @@ main (gint argc, gchar *argv[])
- 	tracker_sparql_connection_close (sparql_conn);
- 	g_object_unref (sparql_conn);
- 
--#if GLIB_CHECK_VERSION (2, 64, 0)
--	g_signal_handlers_disconnect_by_func (memory_monitor, on_low_memory, NULL);
--	g_object_unref (memory_monitor);
--#endif
--
- 	g_print ("\nOK\n\n");
- 
- 	return EXIT_SUCCESS;
diff --git a/srcpkgs/tracker3-miners/template b/srcpkgs/tracker3-miners/template
index 0bf14b48479..7b773dfb494 100644
--- a/srcpkgs/tracker3-miners/template
+++ b/srcpkgs/tracker3-miners/template
@@ -1,6 +1,6 @@
 # Template file for 'tracker3-miners'
 pkgname=tracker3-miners
-version=3.0.0
+version=3.0.3
 revision=1
 wrksrc=tracker-miners-${version}
 build_style=meson
@@ -26,7 +26,7 @@ maintainer="q66 <daniel@octaforge.org>"
 license="GPL-2.0-or-later"
 homepage="https://developer.gnome.org/libtracker-miner/stable"
 distfiles="${GNOME_SITE}/tracker-miners/${version%.*}/tracker-miners-${version}.tar.xz"
-checksum=bb481a7c23c5def4f215627a0630fbe9d1c64f3319825859d7a73645738f4042
+checksum=eb6c956aec200ec441e20f0c8a7e5baedd59532299c6f735774f30db9bff949d
 
 # relies on unsupported ops in chroot
 do_check() {

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PR PATCH] [Merged]: tracker3{-miners} update (bugfix)
  2020-12-22 22:37 [PR PATCH] tracker3{-miners} update (bugfix) st3r4g
  2020-12-22 23:31 ` [PR PATCH] [Updated] " st3r4g
@ 2020-12-25 14:01 ` q66
  1 sibling, 0 replies; 3+ messages in thread
From: q66 @ 2020-12-25 14:01 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 462 bytes --]

There's a merged pull request on the void-packages repository

tracker3{-miners} update (bugfix)
https://github.com/void-linux/void-packages/pull/27363

Description:
Today I noticed an ugly bug. Moving a text file in my home resulted in processes consuming a large amount of cpu and spamming gdm log. I can reproduce by:
```
echo x > x
mv x y
```
and the cpu drain/gdm log spam goes away only after:
`tracker3 reset -s`

Updating seems to have fixed it.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-12-25 14:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-22 22:37 [PR PATCH] tracker3{-miners} update (bugfix) st3r4g
2020-12-22 23:31 ` [PR PATCH] [Updated] " st3r4g
2020-12-25 14:01 ` [PR PATCH] [Merged]: " q66

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).