Bug#904309: RFP: tilda -- [SHORT DESCRIPTION]

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

Bug#904309: RFP: tilda -- [SHORT DESCRIPTION]

ValessioBrito-2
Package: wnpp
Severity: wishlist
X-Debbugs-CC: [hidden email]

--- Please fill out the fields below. ---

   Package name: tilda
        Version: 1.4.1-2
Upstream Author: Sebastian Geiger <[hidden email]>
            URL: http://github.com/lanoxx/tilda


[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-
gnu/libthread_db.so.1".
[New Thread 0x7fffedc0d700 (LWP 10935)]
[New Thread 0x7fffed40c700 (LWP 10936)]
[New Thread 0x7fffecc02700 (LWP 10937)]
[New Thread 0x7fffdf85c700 (LWP 10938)]
[New Thread 0x7fffdf05b700 (LWP 10939)]
[New Thread 0x7fffde5e8700 (LWP 10940)]
[New Thread 0x7fffddde7700 (LWP 10941)]
[New Thread 0x7fffdd5e6700 (LWP 10942)]
[New Thread 0x7fffdcde5700 (LWP 10943)]

Thread 1 "tilda" received signal SIGSEGV, Segmentation fault.
0x00007ffff5ea4339 in XGetModifierMapping () from /usr/lib/x86_64-
linux-gnu/libX11.so.6

Reply | Threaded
Open this post in threaded view
|

Bug#904309: RFP: tilda -- [SHORT DESCRIPTION]

Ben Hutchings-3
Control: reassign -1 tilda 1.4.1-2
Control: tag -1 moreinfo

Don't use "wnpp" for actual bugs.

You need to explain how to reproduce this bug.

Ben.

On Mon, 2018-07-23 at 14:45 +0800, Valessio Brito wrote:

> Package: wnpp
> Severity: wishlist
> X-Debbugs-CC: [hidden email]
>
> --- Please fill out the fields below. ---
>
>    Package name: tilda
>         Version: 1.4.1-2
> Upstream Author: Sebastian Geiger <[hidden email]>
>             URL: http://github.com/lanoxx/tilda
>
>
> [Thread debugging using libthread_db enabled]
> Using host libthread_db library "/lib/x86_64-linux-
> gnu/libthread_db.so.1".
> [New Thread 0x7fffedc0d700 (LWP 10935)]
> [New Thread 0x7fffed40c700 (LWP 10936)]
> [New Thread 0x7fffecc02700 (LWP 10937)]
> [New Thread 0x7fffdf85c700 (LWP 10938)]
> [New Thread 0x7fffdf05b700 (LWP 10939)]
> [New Thread 0x7fffde5e8700 (LWP 10940)]
> [New Thread 0x7fffddde7700 (LWP 10941)]
> [New Thread 0x7fffdd5e6700 (LWP 10942)]
> [New Thread 0x7fffdcde5700 (LWP 10943)]
>
> Thread 1 "tilda" received signal SIGSEGV, Segmentation fault.
> 0x00007ffff5ea4339 in XGetModifierMapping () from /usr/lib/x86_64-
> linux-gnu/libX11.so.6
>
--
Ben Hutchings
Anthony's Law of Force: Don't force it, get a larger hammer.


signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Bug#904309:

Franciscarlos Santos Soares
In reply to this post by ValessioBrito-2
It seems that this segmentation fault occurs when run under wayland. So it happened here. gnome under wayaland.

--

Att,
prof. Franciscarlos Soares
CEIN Raimundo Araújo
CE Dr. Otávio Vieira Passos
Chapadinha - Ma
Currículo Lattes/CNPq:
Reply | Threaded
Open this post in threaded view
|

Bug#904309: RFP: tilda -- [SHORT DESCRIPTION]

Pedro Ângelo
In reply to this post by ValessioBrito-2
I can confirm this issue (Segfault when trying to start under Wayland) is valid
in Debian Testing.

The issue is known upstream (https://github.com/lanoxx/tilda/issues/314) andthere's a workaround in commit daa64f5.

I tried cherry-picking this commit and installing the rebuilt Debian package,
and found that tilda starts up normally again.

Best regards,
P.

Reply | Threaded
Open this post in threaded view
|

Bug#904309: tilda crashes under Wayland

Vlad Orlov
In reply to this post by ValessioBrito-2
Hi,

Crash under Wayland is also confirmed in [1].
Maybe the two reports should be merged, with setting severity to Grave for this one?


[1] https://bugs.debian.org/925483
Reply | Threaded
Open this post in threaded view
|

Bug#904309: appropriate priority for bugs under Wayland?

Jonathan Dowland
In reply to this post by ValessioBrito-2
severity 904309 normal
thanks

Folks, I've seen a few occurences now of bugs like this which are being given
RC severity because Wayland is currently the default desktop technology for
Buster. The consequence of this is that the software (here tilda, elsewhere
synaptic, and perhaps others) are at risk of being dropped from Buster
entirely due to incompatibility under Wayland, despite working fine under X.

I'm not sure that this is fair or the right way to address issues of Wayland
compatibility with other (longer established) software. I'm directing this at
-release to ask the Release Team whether they have a position on the matter.
Thanks!

Related I'm not sure that Wayland is a suitable choice for the default desktop
technology yet either (see Bug #927667 for discussion of that, as well as a
subset of these bugs[1]). Please direct any thoughts on *that* to #927667
rather than here.

[1] https://udd.debian.org/cgi-bin/bts-usertags.cgi?tag=wayland&user=pkg-gnome-maintainers%40lists.alioth.debian.org


Best wishes

--

⢀⣴⠾⠻⢶⣦⠀
⣾⠁⢠⠒⠀⣿⡁ Jonathan Dowland
⢿⡄⠘⠷⠚⠋⠀ https://jmtd.net
⠈⠳⣄⠀⠀⠀⠀ Please do not CC me, I am subscribed to the list.

Reply | Threaded
Open this post in threaded view
|

Bug#904309: tilda: diff for NMU version 1.4.1-2.1

Niels Thykier
In reply to this post by ValessioBrito-2
Control: tags 904309 + patch
Control: tags 904309 + pending


Dear maintainer,

I've prepared an NMU for tilda (versioned as 1.4.1-2.1) and
uploaded it to DELAYED/7. Please feel free to tell me if I
should delay it longer.

Regards.

diff -Nru tilda-1.4.1/debian/changelog tilda-1.4.1/debian/changelog
--- tilda-1.4.1/debian/changelog 2018-02-18 20:08:42.000000000 +0000
+++ tilda-1.4.1/debian/changelog 2019-05-11 13:04:23.000000000 +0000
@@ -1,3 +1,15 @@
+tilda (1.4.1-2.1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Import patch from upstream to set ensure tilda is run
+    under the X11 backend.  This prevents crashes when the
+    window manager uses Wayland by default.
+    (Closes: #904309)
+  * Ret Rules-Requires-Root to no as tilda does not need
+    (fake)root for building the debs.
+
+ -- Niels Thykier <[hidden email]>  Sat, 11 May 2019 13:04:23 +0000
+
 tilda (1.4.1-2) unstable; urgency=medium
 
   [ Sebastian Geiger ]
diff -Nru tilda-1.4.1/debian/control tilda-1.4.1/debian/control
--- tilda-1.4.1/debian/control 2018-02-18 19:46:04.000000000 +0000
+++ tilda-1.4.1/debian/control 2019-05-11 11:30:38.000000000 +0000
@@ -5,6 +5,7 @@
 Build-Depends: debhelper (>= 11), autopoint, pkg-config, libgtk-3-dev, libvte-2.91-dev, libconfuse-dev
 Standards-Version: 4.1.3
 Homepage: http://github.com/lanoxx/tilda
+Rules-Requires-Root: no
 Vcs-Git: https://salsa.debian.org/debian/tilda.git
 Vcs-Browser: https://salsa.debian.org/debian/tilda
 
diff -Nru tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch
--- tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch 1970-01-01 00:00:00.000000000 +0000
+++ tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch 2019-05-11 11:29:06.000000000 +0000
@@ -0,0 +1,25 @@
+From 606b655d8d811f880d610dead7d6943a33199deb Mon Sep 17 00:00:00 2001
+From: Roman Hoellen <[hidden email]>
+Date: Mon, 30 Apr 2018 09:18:49 +0200
+Subject: [PATCH] Set supported backend to X11
+Upstream-Commit: https://github.com/lanoxx/tilda/commit/606b655d8d811f880d610dead7d6943a33199deb.patch
+Forwarded: Not-needed
+
+---
+ src/tilda.c | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/src/tilda.c b/src/tilda.c
+index 8aa69828..92ce46c4 100644
+--- a/src/tilda.c
++++ b/src/tilda.c
+@@ -686,6 +686,9 @@ int main (int argc, char *argv[])
+     g_log_set_default_handler (tilda_log_handler, NULL);
+ #endif
+
++    /* Set supported backend to X11 */
++    gdk_set_allowed_backends ("x11");
++
+     tilda_window tw;
+     /* NULL set the tw pointers so we can get a clean exit on initialization failure */
+     memset(&tw, 0, sizeof(tilda_window));
diff -Nru tilda-1.4.1/debian/patches/series tilda-1.4.1/debian/patches/series
--- tilda-1.4.1/debian/patches/series 2018-02-18 19:48:15.000000000 +0000
+++ tilda-1.4.1/debian/patches/series 2019-05-11 11:30:38.000000000 +0000
@@ -1 +1,2 @@
 01_install_metadata_in_new_location.patch
+02_set_supported_backend_to_X11.patch

Reply | Threaded
Open this post in threaded view
|

Bug#904309: tilda: diff for NMU version 1.4.1-2.1

Sebastian Geiger
In reply to this post by ValessioBrito-2
Hi Niels,

Thanks for the upload. It looks good to me. I think you can push that to the repository.

Best Regards
Mit freundlichen Grüßen
Dipl. Ing. Sebastian Geiger

> On 11. May 2019, at 15:04, Niels Thykier <[hidden email]> wrote:
>
> Control: tags 904309 + patch
> Control: tags 904309 + pending
>
>
> Dear maintainer,
>
> I've prepared an NMU for tilda (versioned as 1.4.1-2.1) and
> uploaded it to DELAYED/7. Please feel free to tell me if I
> should delay it longer.
>
> Regards.
>
> diff -Nru tilda-1.4.1/debian/changelog tilda-1.4.1/debian/changelog
> --- tilda-1.4.1/debian/changelog    2018-02-18 20:08:42.000000000 +0000
> +++ tilda-1.4.1/debian/changelog    2019-05-11 13:04:23.000000000 +0000
> @@ -1,3 +1,15 @@
> +tilda (1.4.1-2.1) unstable; urgency=medium
> +
> +  * Non-maintainer upload.
> +  * Import patch from upstream to set ensure tilda is run
> +    under the X11 backend.  This prevents crashes when the
> +    window manager uses Wayland by default.
> +    (Closes: #904309)
> +  * Ret Rules-Requires-Root to no as tilda does not need
> +    (fake)root for building the debs.
> +
> + -- Niels Thykier <[hidden email]>  Sat, 11 May 2019 13:04:23 +0000
> +
> tilda (1.4.1-2) unstable; urgency=medium
>
>   [ Sebastian Geiger ]
> diff -Nru tilda-1.4.1/debian/control tilda-1.4.1/debian/control
> --- tilda-1.4.1/debian/control    2018-02-18 19:46:04.000000000 +0000
> +++ tilda-1.4.1/debian/control    2019-05-11 11:30:38.000000000 +0000
> @@ -5,6 +5,7 @@
> Build-Depends: debhelper (>= 11), autopoint, pkg-config, libgtk-3-dev, libvte-2.91-dev, libconfuse-dev
> Standards-Version: 4.1.3
> Homepage: http://github.com/lanoxx/tilda
> +Rules-Requires-Root: no
> Vcs-Git: https://salsa.debian.org/debian/tilda.git
> Vcs-Browser: https://salsa.debian.org/debian/tilda
>
> diff -Nru tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch
> --- tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch    1970-01-01 00:00:00.000000000 +0000
> +++ tilda-1.4.1/debian/patches/02_set_supported_backend_to_X11.patch    2019-05-11 11:29:06.000000000 +0000
> @@ -0,0 +1,25 @@
> +From 606b655d8d811f880d610dead7d6943a33199deb Mon Sep 17 00:00:00 2001
> +From: Roman Hoellen <[hidden email]>
> +Date: Mon, 30 Apr 2018 09:18:49 +0200
> +Subject: [PATCH] Set supported backend to X11
> +Upstream-Commit: https://github.com/lanoxx/tilda/commit/606b655d8d811f880d610dead7d6943a33199deb.patch
> +Forwarded: Not-needed
> +
> +---
> + src/tilda.c | 3 +++
> + 1 file changed, 3 insertions(+)
> +
> +diff --git a/src/tilda.c b/src/tilda.c
> +index 8aa69828..92ce46c4 100644
> +--- a/src/tilda.c
> ++++ b/src/tilda.c
> +@@ -686,6 +686,9 @@ int main (int argc, char *argv[])
> +     g_log_set_default_handler (tilda_log_handler, NULL);
> + #endif
> +
> ++    /* Set supported backend to X11 */
> ++    gdk_set_allowed_backends ("x11");
> ++
> +     tilda_window tw;
> +     /* NULL set the tw pointers so we can get a clean exit on initialization failure */
> +     memset(&tw, 0, sizeof(tilda_window));
> diff -Nru tilda-1.4.1/debian/patches/series tilda-1.4.1/debian/patches/series
> --- tilda-1.4.1/debian/patches/series    2018-02-18 19:48:15.000000000 +0000
> +++ tilda-1.4.1/debian/patches/series    2019-05-11 11:30:38.000000000 +0000
> @@ -1 +1,2 @@
> 01_install_metadata_in_new_location.patch
> +02_set_supported_backend_to_X11.patch

Reply | Threaded
Open this post in threaded view
|

Bug#904309: tilda: diff for NMU version 1.4.1-2.1

Niels Thykier
Sebastian Geiger:
> Hi Niels,
>
> Thanks for the upload. It looks good to me. I think you can push that to the repository.
>
> Best Regards
> Mit freundlichen Grüßen
> Dipl. Ing. Sebastian Geiger
>
> [...]

Hi Sebastian,

Thanks for your feedback.

Just to confirm, when you wrote "push that to the repository", did you
mean push changes to the git repo on salsa (master branch) or remove the
delay of the upload to the archive (or both)?

Thanks,
~Niels