[PATCH] gnu packages gnome-console: Remove nautilus extension

  • Done
  • quality assurance status badge
Details
5 participants
  • Dominic Martinez
  • Liliana Marie Prikler
  • Liliana Marie Prikler
  • Christopher Baines
  • Mája Tomášek
Owner
unassigned
Submitted by
Mája Tomášek
Severity
normal

Debbugs page

M
M
Mája Tomášek wrote on 16 Oct 2022 10:34
(address . guix-patches@gnu.org)
87sfjnad0i.fsf@disroot.org
Hi,

this patch removes the gnome-console nautilus extension as it crashes nautilus
(at least on my machine). In my opinion it is safe to remove it, as this
feature is implemented directly into nautilus-43 and the original code is
purged from the gnome-console repo. (see

This sadly means, that in nautilus-42 there will be no "Open in Console", but
the issue is on some code that is already deprecated in the Console repo. (I
have studied gdb info and it appears that kgx_nautilus_get_background_items()
returns a GList* that is { .next = NULL, .prev = NULL, .data = NULL } but
nautilus does only perform a check that GList* is NULL, otherwise it expects a
valid list)

I can maybe supply a patch to gnome-console, but I don't see a reason to.

Kind regards,

Maya
---
gnu/packages/gnome.scm | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

Toggle diff (16 lines)
diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index fbaef9eb6e..c71894bcb9 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -5445,8 +5445,7 @@ (define-public gnome-console
(build-system meson-build-system)
(arguments
(list #:glib-or-gtk? #t
- #:configure-flags #~(list "-Dtests=true"
- "-Dnautilus=enabled")
+ #:configure-flags #~(list "-Dtests=true")
#:phases #~(modify-phases %standard-phases
(add-after 'unpack 'patch-nautilus-extension-path
(lambda _
--
2.37.3
L
L
Liliana Marie Prikler wrote on 17 Oct 2022 01:18
e6c3ddc1e9bd689ef6c7fa1764278309e02a229a.camel@ist.tugraz.at
Am Sonntag, dem 16.10.2022 um 19:34 +0200 schrieb Mája Tomášek:
Toggle quote (5 lines)
>
> Hi,
>
> this patch removes the gnome-console nautilus extension as it crashes
> nautilus (at least on my machine).
I can confirm it working as intended on at least two separate machines,
both configured

Toggle quote (16 lines)
> In my opinion it is safe to remove it, as this
> feature is implemented directly into nautilus-43 and the original
> code is purged from the gnome-console repo. (see
> https://gitlab.gnome.org/GNOME/nautilus/-/merge_requests/911 and
> https://gitlab.gnome.org/GNOME/console/-/commit/c4bd2312f4b432a33810a8dcdf3cadabae5169d7
> )
>
> This sadly means, that in nautilus-42 there will be no "Open in
> Console", but the issue is on some code that is already deprecated in
> the Console repo. (I have studied gdb info and it appears that
> kgx_nautilus_get_background_items() returns a GList* that is { .next
> = NULL, .prev = NULL, .data = NULL } but nautilus does only perform a
> check that GList* is NULL, otherwise it expects a valid list)
>
> I can maybe supply a patch to gnome-console, but I don't see a reason
> to.
In my opinion, it'd be better to find out why it crashes in the first
place. See [1] for the breaking code. My suspicion is that item ends
up NULL, which would be big if true.

Cheers

[1]
M
M
Mája Tomášek wrote on 17 Oct 2022 09:20
875ygie81r.fsf@disroot.org
Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> writes:

Toggle quote (9 lines)
> Am Sonntag, dem 16.10.2022 um 19:34 +0200 schrieb Mája Tomášek:
>>
>> Hi,
>>
>> this patch removes the gnome-console nautilus extension as it crashes
>> nautilus (at least on my machine).
> I can confirm it working as intended on at least two separate machines,
> both configured

I have pinpointed the issue! I tried the debugging steps and suddenly
nautilus opened fine.

So, the issue is that I have some deffective version of
libkgx-nautilus.so, which does return NULL list.

But, if I provide a different version of gnome-console
(ie. --with-debug-info=gnome-console) nautilus suddenly opens fine.

I have run multiple guix pulls, guix system reconfigure, guix gc
--verify=contents but it has not fixed it.

It even works when I use run guix shell --pure nautilus gnome-console.

I am now completely lost.
L
L
Liliana Marie Prikler wrote on 17 Oct 2022 23:19
01dce84767493f80121dee534fbd9edb7225a620.camel@ist.tugraz.at
Am Montag, dem 17.10.2022 um 18:20 +0200 schrieb Mája Tomášek:
Toggle quote (27 lines)
> Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> writes:
>
> > Am Sonntag, dem 16.10.2022 um 19:34 +0200 schrieb Mája Tomášek:
> > >
> > > Hi,
> > >
> > > this patch removes the gnome-console nautilus extension as it
> > > crashes nautilus (at least on my machine).
> > I can confirm it working as intended on at least two separate
> > machines, both configured
>
> I have pinpointed the issue! I tried the debugging steps and suddenly
> nautilus opened fine.
>
> So, the issue is that I have some deffective version of
> libkgx-nautilus.so, which does return NULL list.
>
> But, if I provide a different version of gnome-console
> (ie. --with-debug-info=gnome-console) nautilus suddenly opens fine.
>
> I have run multiple guix pulls, guix system reconfigure, guix gc
> --verify=contents but it has not fixed it.
>
> It even works when I use run guix shell --pure nautilus gnome-
> console.
>
> I am now completely lost.
IIRC you have to --verify=contents,repair. Note that repair might
still not work due to gnome-console – like all gnome packages –
receiving grafts, which is known to mess with guix build --repair.

If everything else fails, you can try rolling back to a generation
before gnome-console was built, delete all future generations, gc, and
then rebuild gnome-console. This should definitely make nautilus work,
under the assumption that your gnome-console build succeeds this time
and no spurious hardware failure turns up.

Cheers
D
D
Dominic Martinez wrote on 18 Oct 2022 14:48
Re: [bug#58569] [PATCH] gnu packages gnome-console: Remove nautilus extension
(name . Liliana Marie Prikler)(address . liliana.prikler@ist.tugraz.at)
87mt9sssxg.fsf@dominicm.dev
Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> writes:

Toggle quote (6 lines)
> IIRC you have to --verify=contents,repair. Note that repair
> might
> still not work due to gnome-console – like all gnome packages –
> receiving grafts, which is known to mess with guix build
> --repair.

I'm encountering the same issue, so it seems strange for it to be
a spurious corruption. Just chiming in in case this doesn't work.
-----BEGIN PGP SIGNATURE-----

iHUEARYIAB0WIQRtp6gAxeTcYmhxQ/1Bnl1fEVBrXQUCY08fiwAKCRBBnl1fEVBr
XeJXAP4hadTMZgwNat1T3s0l/U91sL+RsCh8u0WAGUq+nSi4IgD9GBBF6CiILt3T
aTDN/rK6tKSWI+CyBfc4glrUfll8VQo=
=X9j5
-----END PGP SIGNATURE-----

M
M
Mája Tomášek wrote on 20 Oct 2022 12:58
8735biclnj.fsf@disroot.org
Dominic Martinez <dom@dominicm.dev> writes:

Toggle quote (11 lines)
> Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> writes:
>
>> IIRC you have to --verify=contents,repair. Note that repair
>> might
>> still not work due to gnome-console – like all gnome packages –
>> receiving grafts, which is known to mess with guix build
>> --repair.
>
> I'm encountering the same issue, so it seems strange for it to be
> a spurious corruption. Just chiming in in case this doesn't work.

I have tested everything that has been suggested, but nothing really
worked. When I try apply transform on gnome
(options->transformation
'((with-debug-info . "gnome-console")))
it does not allow me to do so. (Apparently there is a version conflict.

I know that this issue is not present on
guix shell --pure nautilus gnome-console

And is also fixed with
guix shell --with-debug-info=gnome-console nautilus gnome-console

But I cannot upgrade gnome-console, or at least when I tried rollbacking
to a previous version (about a month ago) then deleting all generations,
running guix gc. But the issue is back when I ran guix system
reconfigure.

I have now verified, that it is indeed not a binary corruption issue. As both
in normal and in the pure environment the libkgx-nautilus.so are the
same file.

The fact that it does not occur when using the
--with-debug-info=gnome-console option is kind of a bummer, as I cannot
use gdb to debug gnome-console :D

Regards

Maya
L
L
Liliana Marie Prikler wrote on 20 Oct 2022 23:09
(address . 58569@debbugs.gnu.org)
7c3bda7d322d296244bdd9be5493d53509b928bc.camel@ist.tugraz.at
Am Donnerstag, dem 20.10.2022 um 21:58 +0200 schrieb Mája Tomášek:
Toggle quote (3 lines)
> I have now verified, that it is indeed not a binary corruption issue.
> As both in normal and in the pure environment the libkgx-nautilus.so
> are the same file.
Your nautilus could still be corrupted, though, linking to the wrong
libkgx-nautilus.so

Cheers
M
M
Mája Tomášek wrote on 24 Oct 2022 14:50
87y1t4evsh.fsf@disroot.org
Dominic Martinez <dom@dominicm.dev> writes:

Toggle quote (11 lines)
> Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> writes:
>
>> IIRC you have to --verify=contents,repair. Note that repair
>> might
>> still not work due to gnome-console – like all gnome packages –
>> receiving grafts, which is known to mess with guix build
>> --repair.
>
> I'm encountering the same issue, so it seems strange for it to be
> a spurious corruption. Just chiming in in case this doesn't work.

Just an fyi, I have temporarily fixed the issue by unsetting
NAUTILUS_EXTENSION_PATH environment variable

This is only a temporary fix, but it means that you can use your desktop
normally in the meantime.
M
M
Mája Tomášek wrote on 24 Oct 2022 14:54
(address . 58569@debbugs.gnu.org)
87pmegevl9.fsf@disroot.org
Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> writes:

Toggle quote (9 lines)
> Am Donnerstag, dem 20.10.2022 um 21:58 +0200 schrieb Mája Tomášek:
>> I have now verified, that it is indeed not a binary corruption issue.
>> As both in normal and in the pure environment the libkgx-nautilus.so
>> are the same file.
> Your nautilus could still be corrupted, though, linking to the wrong
> libkgx-nautilus.so
>
> Cheers

I think that nautilus loads this library by checking for objects in
NAUTILUS_EXTENSION_PATH and loading them into the system at
runtime. Simply unsetting the variable fixes the issue. So in my opinion
nautilus cannot link the wrong file, as it does on runtime and that is
set to a variable that essentialy links it to the same file.

Sidenote, nautilus is actually not involved in the process, as
gnome-console is not it's input. (It's actually the other way around)

Cheers
L
L
Liliana Marie Prikler wrote on 24 Oct 2022 22:59
(address . 58569@debbugs.gnu.org)
19e4c0df760dd4328e96cfb2d87bc612f3e18239.camel@ist.tugraz.at
Am Montag, dem 24.10.2022 um 23:54 +0200 schrieb Mája Tomášek:
Toggle quote (6 lines)
> I think that nautilus loads this library by checking for objects in
> NAUTILUS_EXTENSION_PATH and loading them into the system at
> runtime. Simply unsetting the variable fixes the issue. So in my
> opinion nautilus cannot link the wrong file, as it does on runtime
> and that is set to a variable that essentialy links it to the same
> file.t
Ahh, you're right of course. Still, when debugging against guix shell,
you ought to take the shell cache into account, which can obscure bugs.
In either case, check very carefully that the two libkgx-nautilus
actually point to the same file.


Cheers
C
C
Christopher Baines wrote on 5 Nov 2022 03:27
tag 58569 moreinfo
(address . control@debbugs.gnu.org)
87v8nthf1o.fsf@cbaines.net
tags 58569 + moreinfo
quit
L
L
Liliana Marie Prikler wrote on 26 Nov 2022 04:45
e17db2e4cfe47f439fd35a7f394a2897ada35d6b.camel@gmail.com
close 58569 99ba4ddb03b396f56764a25317f40d4501380704
thanks
?
Your comment

This issue is archived.

To comment on this conversation send an email to 58569@patchwise.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 58569
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch