GNU bug report logs - #40753
gnu: spacefm: Add missing dependencies.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 40753 in the body.
You can then email your comments to 40753 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: guix-patches <at> gnu.org
Subject: gnu: spacefm: Add missing dependencies.
Date: Tue, 21 Apr 2020 17:28:12 -0400

[Message part 1 (text/plain, inline)]

BLOCKERS: #40630 and #40752.

[0003-gnu-spacefm-Add-missing-dependencies.patch (text/x-patch, attachment)]

Message #8 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: 40753 <at> debbugs.gnu.org
Subject: gnu: spacefm: Add and configure missing dependencies. (v2)
Date: Tue, 21 Apr 2020 20:15:50 -0400

[Message part 1 (text/plain, inline)]

BLOCKERS: #40630, #40752 and #40756.

[0004-gnu-spacefm-Add-and-configure-missing-dependencies.patch (text/x-patch, attachment)]

Message #11 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add missing dependencies.
Date: Wed, 22 Apr 2020 08:46:38 +0200

Raghav Gururajan writes:

Hello!

> BLOCKERS: #40630 and #40752.

...what does this mean?

>>From fac881e2820c08e09c8f3e42699ba25b8b9f4004 Mon Sep 17 00:00:00 2001
> From: Raghav Gururajan <raghavgururajan <at> disroot.org>
> Date: Tue, 21 Apr 2020 17:23:21 -0400
> Subject: [PATCH 3/3] gnu: spacefm: Add missing dependencies.

> * gnu/packages/lxde.scm (spacefm): Add missing dependencies.

It builds OK without this patch; it does not seem that any dependencies
are missing.  Could it be that these dependencies change or amend its
behaviour?

After applying this patch, spacefm does not build for me, I get

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env guix build spacefm
ice-9/eval.scm:223:20: In procedure proc:
error: eject: unbound variable
hint: Did you forget a `use-modules' form?
--8<---------------cut here---------------end--------------->8---

I am feeling a bit uncomfortable about the fact that you pinged me to
"push" this patch, it seems it needs a bit more attention than that.

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #14 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add missing dependencies.
Date: Wed, 22 Apr 2020 12:28:33 -0400

Hi Jan!

> > BLOCKERS: #40630 and #40752.  
> 
> ...what does this mean?

They are dependencies of this patch, that needs to be pushed before this patch.

> It builds OK without this patch; it does not seem that any dependencies
> are missing.  Could it be that these dependencies change or amend its
> behaviour?

It is to change and amend the behaviour. It adds missing features and fixes
icons+privilege-management+disk-management.

> After applying this patch, spacefm does not build for me, I get
> 
> --8<---------------cut here---------------start------------->8---
> $ ./pre-inst-env guix build spacefm
> ice-9/eval.scm:223:20: In procedure proc:
> error: eject: unbound variable
> hint: Did you forget a `use-modules' form?
> --8<---------------cut here---------------end--------------->8---

That's correct. Eject is provided by one of the blockers.

> I am feeling a bit uncomfortable about the fact that you pinged me to
> "push" this patch, it seems it needs a bit more attention than that.

I apologize. I spent whole day to create these patches and throughly testing
them. If I missed anything, please understand it would not have been
intentional. Also, there is a v2 patch in #40753 thread that has 3 blockers.

Thank you for considering my request.

Regards,
RG.

Message #17 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add missing dependencies.
Date: Wed, 22 Apr 2020 19:59:41 +0200

Raghav Gururajan writes:

Hello Raghav,

>> > BLOCKERS: #40630 and #40752.  
>> 
>> ...what does this mean?
>
> They are dependencies of this patch, that needs to be pushed before
> this patch.

Right.  So we'll have to wait for those then, right?

>> It builds OK without this patch; it does not seem that any dependencies
>> are missing.  Could it be that these dependencies change or amend its
>> behaviour?
>
> It is to change and amend the behaviour. It adds missing features and fixes
> icons+privilege-management+disk-management.

Okay, it would be nice to say something about that in the commit
message.  What about

    gnu: spacefm: Add dependencies for extra functionality.

    Adding these inputs provides adds icons, privilege-management and
    disk-management.

Other than that, LGTM (assuming that it builds).  I'm not really able
to test it, I assume it works for you as you expected?

>> I am feeling a bit uncomfortable about the fact that you pinged me to
>> "push" this patch, it seems it needs a bit more attention than that.
>
> I apologize. I spent whole day to create these patches and throughly testing
> them. If I missed anything, please understand it would not have been
> intentional. Also, there is a v2 patch in #40753 thread that has 3 blockers.
>
> Thank you for considering my request.

No worries!  I did not have much time this morning and from your ping
assumed a trivial task.  Teaches me not to assume things.

I would suggest that when you get an OK on the last dependency for this
one, you mention this patch again.

Please consider sending patches that depend on eachother in one bug
report if that makes sense.

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #20 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add missing dependencies. (v3)
Date: Wed, 22 Apr 2020 16:22:39 -0400

[Message part 1 (text/plain, inline)]

Hi Jan!

> Right.  So we'll have to wait for those then, right?

Pardon me, I should have communicated better. I was asking to push including
the blockers. Would you be able to please? If not, that's okay.

> Okay, it would be nice to say something about that in the commit
> message.  What about
> 
>     gnu: spacefm: Add dependencies for extra functionality.
> 
>     Adding these inputs provides adds icons, privilege-management and
>     disk-management.

Thanks. Please find the revised patch attached with this email.

> Other than that, LGTM (assuming that it builds).  I'm not really able
> to test it, I assume it works for you as you expected?

Yes, all works good. :-)

> No worries!  I did not have much time this morning and from your ping
> assumed a trivial task.  Teaches me not to assume things.

No, you did not do anything wrong. I should have communicated better.

> I would suggest that when you get an OK on the last dependency for this
> one, you mention this patch again.

Sure. But let me know if you would be able to process the blockers as well.

> Please consider sending patches that depend on eachother in one bug
> report if that makes sense.

Sorry, I did not plan on fixing spacefm. I came across the issues with spacefm
after sending those other patches.

Regards,
RG.

[0004-gnu-spacefm-Add-dependencies-for-extra-functionality.patch (text/x-patch, attachment)]

Message #23 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: 40753 <at> debbugs.gnu.org
Subject: gnu: spacefm: Add dependencies for extra functionality. (v3)
Date: Fri, 24 Apr 2020 15:35:58 -0400

[Message part 1 (text/plain, inline)]

[0002-gnu-spacefm-Add-dependencies-for-extra-functionality.patch (text/x-patch, attachment)]

Message #26 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality. (v3)
Date: Fri, 24 Apr 2020 22:09:15 +0200

Raghav Gururajan writes:

Hi Raghav,

Pushed to master with small cosmetic change below.

Thanks for the patch!
janneke

>>From c8ffa0953bd26a4ce22cae17b55d0392908b0f71 Mon Sep 17 00:00:00 2001
> From: Raghav Gururajan <raghavgururajan <at> disroot.org>
> Date: Fri, 24 Apr 2020 15:33:15 -0400
> Subject: [PATCH 2/2] gnu: spacefm: Add dependencies for extra functionality.
>    #:use-module (gnu packages wm)
>    #:use-module (gnu packages xml)
> +  #:use-module (gnu packages xdisorg)
>    #:use-module (gnu packages xorg)

Changed this to

    #:use-module (gnu packages wm)
    #:use-module (gnu packages xdisorg)
    #:use-module (gnu packages xml)
    #:use-module (gnu packages xorg)

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #29 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Sat, 25 Apr 2020 22:11:43 -0400

[Message part 1 (text/plain, inline)]

Hi Jan!

> Pushed to master with small cosmetic change below.
> 
> Thanks for the patch!

I accidentally sent the wrong patch that missed some things. So sorry. I have
attached a patch with this email to append the previous. Could you please push
it as new commit. Thanks!

Regards,
RG.

[0001-gnu-spacefm-Fix-privilege-management-and-icons.patch (text/x-patch, attachment)]

Message #32 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Sun, 26 Apr 2020 16:00:57 +0200

Raghav Gururajan writes:

Hello Raghav,

>> Pushed to master with small cosmetic change below.
>> 
>> Thanks for the patch!
>
> I accidentally sent the wrong patch that missed some things. So sorry. I have
> attached a patch with this email to append the previous. Could you please push
> it as new commit. Thanks!

Don't worry, we all make mistakes.

> From ea9e9909c2929639c1b6574ae61ca9fb0b319177 Mon Sep 17 00:00:00 2001
> From: Raghav Gururajan <raghavgururajan <at> disroot.org>
> Date: Sat, 25 Apr 2020 21:43:25 -0400
> Subject: [PATCH] gnu: spacefm: Fix privilege management and icons.

>>>HERE<<<

> * gnu/packages/lxde.scm (spacefm): Fix privilege management and icons.

Please move this remark:

> This commit contains changes that were accidentally left out in
> commit bf37b49cdd345bcedeb7142f61968d3a6e15e8d8.

beteen subject and "* gnu/packages/lxde", so to >>>HERE<<<

>      (arguments
> -     `(#:configure-flags (list (string-append "--with-bash-path="
> -                                              (assoc-ref %build-inputs "bash")
> -                                              "/bin/bash")
> -                               (string-append "--sysconfdir="
> -                                              (assoc-ref %outputs "out")
> -                                              "/etc"))))
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'patch-spacefm-conf
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "etc/spacefm.conf"
> +               (("#terminal_su=/bin/su")
> +                (string-append "terminal_su="
> +                               (string-append (assoc-ref inputs "sudo")
> +                                              "/bin/sudo"))))

Does this work; SU is not SUDO?  Also, I don't see how <sudo>/bin/sudo
would work.  When I run this

    $(guix build sudo)/bin/sudo 

I get

    sudo: /gnu/store/l320ig872ny66d1yi6v7n4zb93iz50dx-sudo-1.8.31p1/bin/sudo must be owned by uid 0 and have the setuid bit set

Am I missing something?

> +             (substitute* "etc/spacefm.conf"
> +               (("#graphical_su=/usr/bin/gksu")
> +                (string-append "graphical_su="
> +                               (string-append (assoc-ref inputs "ktsuss")
> +                                              "/bin/ktsuss"))))
> +             #t)))

Changing the default configurations does not seem to work for me.  When
I select

    /File/Root Window

there is a pop-up that says: Please configure a valid Terminal SU
command.  Does it work for You?

> +       #:configure-flags (list 
                                 ^
Trailing space.

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #35 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Mon, 27 Apr 2020 02:29:56 -0400

[Message part 1 (text/plain, inline)]

Hi Jan!

> >>>HERE<<<  
> 
> > * gnu/packages/lxde.scm (spacefm): Fix privilege management and icons.  
> 
> Please move this remark:
> 
> > This commit contains changes that were accidentally left out in
> > commit bf37b49cdd345bcedeb7142f61968d3a6e15e8d8.  
> 
> beteen subject and "* gnu/packages/lxde", so to >>>HERE<<<

Sure.

> >      (arguments
> > -     `(#:configure-flags (list (string-append "--with-bash-path="
> > -                                              (assoc-ref %build-inputs
> > "bash")
> > -                                              "/bin/bash")
> > -                               (string-append "--sysconfdir="
> > -                                              (assoc-ref %outputs "out")
> > -                                              "/etc"))))
> > +     `(#:phases
> > +       (modify-phases %standard-phases
> > +         (add-after 'unpack 'patch-spacefm-conf
> > +           (lambda* (#:key inputs #:allow-other-keys)
> > +             (substitute* "etc/spacefm.conf"
> > +               (("#terminal_su=/bin/su")
> > +                (string-append "terminal_su="
> > +                               (string-append (assoc-ref inputs "sudo")
> > +                                              "/bin/sudo"))))  
> 
> Does this work; SU is not SUDO?  Also, I don't see how <sudo>/bin/sudo
> would work.  When I run this
> 
>     $(guix build sudo)/bin/sudo 
> 
> I get
> 
>     sudo: /gnu/store/l320ig872ny66d1yi6v7n4zb93iz50dx-sudo-1.8.31p1/bin/sudo
> must be owned by uid 0 and have the setuid bit set
> 
> Am I missing something?

The option just has the name "su". It can be sudo too. I wouldn't worry about
this terminal_su option. This is just has to be set, so that spacefm doesn't
throw "no valid program found" error, now and then, because of default FHS path.
SpaceFM primarily uses graphical_su, which has been set to 'ktsuss'.

> > +             (substitute* "etc/spacefm.conf"
> > +               (("#graphical_su=/usr/bin/gksu")
> > +                (string-append "graphical_su="
> > +                               (string-append (assoc-ref inputs "ktsuss")
> > +                                              "/bin/ktsuss"))))
> > +             #t)))  
> 
> Changing the default configurations does not seem to work for me.  When
> I select
> 
>     /File/Root Window
> 
> there is a pop-up that says: Please configure a valid Terminal SU
> command.  Does it work for You?

This is exactly the error I was talking about above. Once we apply this patch,
the error appearing stops and ktsuss will pop right up.

> 
> > +       #:configure-flags (list   
>                                  ^
> Trailing space.

Thanks! I have attached revised patch with this email.

Regards,
RG.

[0001-gnu-spacefm-Fix-privilege-management.patch (text/x-patch, attachment)]

Message #38 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Efraim Flashner <efraim <at> flashner.co.il>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753 <at> debbugs.gnu.org, Jan Nieuwenhuizen <janneke <at> gnu.org>
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Mon, 27 Apr 2020 09:38:12 +0300

[Message part 1 (text/plain, inline)]

On Guix System $(guix build sudo)/bin/sudo isn't setuid, we'd need the
one from /run/setuid-programs.

(ins)efraim <at> E5400 ~$ which sudo
/run/setuid-programs/sudo


-- 
Efraim Flashner   <efraim <at> flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[signature.asc (application/pgp-signature, inline)]

Message #41 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753 <at> debbugs.gnu.org
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Mon, 27 Apr 2020 10:37:52 +0200

Raghav Gururajan writes:

Hello Raghav,

>> Does this work; SU is not SUDO?  Also, I don't see how <sudo>/bin/sudo
>> would work.  When I run this
>> 
>>     $(guix build sudo)/bin/sudo 
>> 
>> I get
>> 
>>     sudo: /gnu/store/l320ig872ny66d1yi6v7n4zb93iz50dx-sudo-1.8.31p1/bin/sudo
>> must be owned by uid 0 and have the setuid bit set
>> 
>> Am I missing something?
>
> The option just has the name "su". It can be sudo too. I wouldn't worry about
> this terminal_su option.

That's okay, I'm not really worried :-)

When I read a patch, and I cannot imagine how that would work, I would
like to learn more to understand it beter, or remove it.  I do not like
adding code that I do not understand, and also does not work.

> This is just has to be set, so that spacefm doesn't throw "no valid
> program found" error, now and then, because of default FHS path.
> SpaceFM primarily uses graphical_su, which has been set to 'ktsuss'.

Okay, that would explain something.  So, TERMINAL_SU just needs to be
set to "some" existing executable, but is not used?  What about

     `(#:phases
       (modify-phases %standard-phases
         (add-after 'unpack 'patch-spacefm-conf
           (lambda _
             ;; If terminal_su is unset, users get a popup:
             ;; "Please configure a valid Terminal SU command."
             (substitute* "etc/spacefm.conf"
               (("#terminal_su=/bin/su")
                "terminal_su=/run/current-system/profile/bin/false"))
             #t)))

I tested this, and it hase the same, partially functional result, as
using sudo and setting #graphical_su.  WDYT?

>> > +             (substitute* "etc/spacefm.conf"
>> > +               (("#graphical_su=/usr/bin/gksu")
>> > +                (string-append "graphical_su="
>> > +                               (string-append (assoc-ref inputs "ktsuss")
>> > +                                              "/bin/ktsuss"))))
>> > +             #t)))  
>> 
>> Changing the default configurations does not seem to work for me.  When
>> I select
>> 
>>     /File/Root Window
>> 
>> there is a pop-up that says: Please configure a valid Terminal SU
>> command.  Does it work for You?
>
> This is exactly the error I was talking about above. Once we apply this patch,
> the error appearing stops and ktsuss will pop right up.

Hmm.  Can you double check?  I tested your patch again today it (still)
does not work.  I get

    Please configure a valid Terminal SU command.

Only when I manually point spacefm to it's configuration file, like so

    /gnu/store/y24705ci3dcjiqdig3k3x18pc6aymnzc-spacefm-1.0.6/bin/spacefm -c /gnu/store/y24705ci3dcjiqdig3k3x18pc6aymnzc-spacefm-1.0.6/etc/spacefm/

then the popup disappears.  So now I am wondering, is the problem with
your setup or with mine?  What do you think?

By the way, the KTSUSS popup (when I see it) does not work for me.  Have
you tested KTSUSS?

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #44 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Efraim Flashner <efraim <at> flashner.co.il>
Cc: 40753 <at> debbugs.gnu.org, Raghav Gururajan <raghavgururajan <at> disroot.org>
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Mon, 27 Apr 2020 10:42:45 +0200

Efraim Flashner writes:

> On Guix System $(guix build sudo)/bin/sudo isn't setuid, we'd need the
> one from /run/setuid-programs.
>
> (ins)efraim <at> E5400 ~$ which sudo
> /run/setuid-programs/sudo

Yes, that's what I wanted to suggest...but then I saw that the setting
is called TERMINAL_SU (not SUDO).  That got me thinking: this is
possibly not used at all...could be dead code?

janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #47 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 40753 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Tue, 28 Apr 2020 05:11:08 -0400

[Message part 1 (text/plain, inline)]

Hi Jan!

Sorry again for messing this up. I have attached revised patch with this email.

This, along with #40922 and new patch at #40901, makes SpaceFM to work
perfectly. :-)

Regards,
RG.

[0003-gnu-spacefm-Fix-privilege-and-disk-management.patch (text/x-patch, attachment)]

Message #50 received at 40753 <at> debbugs.gnu.org (full text, mbox):

From: Raghav Gururajan <raghavgururajan <at> disroot.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: 40753 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Thu, 30 Apr 2020 09:57:07 -0400

Hi Jan!

Since #40753 is already closed. I have sent revised patch as #40982.

So #40982 and #40922, should fix current spacefm disk issues.

Regards,
RG.

Message #55 received at 40753-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Raghav Gururajan <raghavgururajan <at> disroot.org>
Cc: 40753-done <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>,
 Jan Nieuwenhuizen <janneke <at> gnu.org>
Subject: Re: [bug#40753] gnu: spacefm: Add dependencies for extra
 functionality.
Date: Sat, 30 May 2020 01:08:47 -0400

Raghav Gururajan <raghavgururajan <at> disroot.org> writes:

> Hi Jan!
>
> Since #40753 is already closed. I have sent revised patch as #40982.
>
> So #40982 and #40922, should fix current spacefm disk issues.
>
> Regards,
> RG.

Alright, I'm closing this one then, by sending a reply to
40753-done <at> debbugs.gnu.org (that is, simply add the '-done' suffix to
the bug number in the debbugs email).

Thank you,

Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.