[bug#40753] gnu: spacefm: Add dependencies for extra functionality.

[Raghav Gururajan]

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
Description: Text Data