all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#26795: gnu: emacs-ag: Propagate the-silver-searcher.
@ 2017-05-06  9:04 Clément Lassieur
  2017-05-06  9:07 ` bug#26795: [PATCH] " Clément Lassieur
  2017-05-06 14:05 ` bug#26795: " Ludovic Courtès
  0 siblings, 2 replies; 5+ messages in thread
From: Clément Lassieur @ 2017-05-06  9:04 UTC (permalink / raw)
  To: 26795

Hi,

emacs-ag is mostly unusable via Tramp (except if the
'the-silver-searcher' store element is also in the remote system, which
will never happen if the remote system is not running Guix).

I believe 'the-silver-searcher' should be propagated, and the executable
should be 'ag' (which is the default value) instead of
'/gnu/store/...-the-silver-searcher-1.0.2/bin/ag', so that it can be
found on remote systems.

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

* bug#26795: [PATCH] gnu: emacs-ag: Propagate the-silver-searcher.
  2017-05-06  9:04 bug#26795: gnu: emacs-ag: Propagate the-silver-searcher Clément Lassieur
@ 2017-05-06  9:07 ` Clément Lassieur
  2017-05-06 14:05 ` bug#26795: " Ludovic Courtès
  1 sibling, 0 replies; 5+ messages in thread
From: Clément Lassieur @ 2017-05-06  9:07 UTC (permalink / raw)
  To: 26795

This allows emacs-ag to be used on remote systems.

* gnu/packages/emacs.scm (emacs-ag)[inputs]: Remove.  Move THE-SILVER-SEARCHER
to...
[propagated-inputs]: ... here.
[arguments]: Remove 'patch-exec-paths' phase.
---
 gnu/packages/emacs.scm | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 74f0ff8b5..3ba5fe102 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -1125,13 +1125,6 @@ than @code{electric-indent-mode}.")
     (arguments
      `(#:phases
        (modify-phases %standard-phases
-         (add-before 'install 'patch-exec-paths
-           (lambda* (#:key inputs #:allow-other-keys)
-             (emacs-substitute-variables "ag.el"
-               ("ag-executable"
-                (string-append (assoc-ref inputs "the-silver-searcher")
-                               "/bin/ag")))
-             #t))
          (add-before 'install 'make-info
            (lambda _
              (with-directory-excursion "docs"
@@ -1142,14 +1135,13 @@ than @code{electric-indent-mode}.")
                     (info (string-append out "/share/info")))
                (install-file "docs/_build/texinfo/agel.info" info)
                #t))))))
-    (inputs
-     `(("the-silver-searcher" ,the-silver-searcher)))
     (native-inputs
      `(("python-sphinx" ,python-sphinx)
        ("texinfo" ,texinfo)))
     (propagated-inputs
      `(("dash" ,emacs-dash)
-       ("s" ,emacs-s)))
+       ("s" ,emacs-s)
+       ("the-silver-searcher" ,the-silver-searcher)))
     (home-page "https://github.com/Wilfred/ag.el")
     (synopsis "Front-end for ag (the-silver-searcher) for Emacs")
     (description "This package provides the ability to use the silver
-- 
2.12.2

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

* bug#26795: gnu: emacs-ag: Propagate the-silver-searcher.
  2017-05-06  9:04 bug#26795: gnu: emacs-ag: Propagate the-silver-searcher Clément Lassieur
  2017-05-06  9:07 ` bug#26795: [PATCH] " Clément Lassieur
@ 2017-05-06 14:05 ` Ludovic Courtès
  2017-05-11 16:44   ` Mathieu Othacehe
  1 sibling, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2017-05-06 14:05 UTC (permalink / raw)
  To: Clément Lassieur; +Cc: 26795

Hello Clément,

Clément Lassieur <clement@lassieur.org> skribis:

> emacs-ag is mostly unusable via Tramp (except if the
> 'the-silver-searcher' store element is also in the remote system, which
> will never happen if the remote system is not running Guix).
>
> I believe 'the-silver-searcher' should be propagated, and the executable
> should be 'ag' (which is the default value) instead of
> '/gnu/store/...-the-silver-searcher-1.0.2/bin/ag', so that it can be
> found on remote systems.

That makes sense to me.  Christopher Baines wrote it; WDYT Chris?

> This allows emacs-ag to be used on remote systems.
>
> * gnu/packages/emacs.scm (emacs-ag)[inputs]: Remove.  Move THE-SILVER-SEARCHER
> to...
> [propagated-inputs]: ... here.
> [arguments]: Remove 'patch-exec-paths' phase.

[...]

>      (propagated-inputs
>       `(("dash" ,emacs-dash)
> -       ("s" ,emacs-s)))
> +       ("s" ,emacs-s)
> +       ("the-silver-searcher" ,the-silver-searcher)))

I’d suggest writing the rationale for propagation in a comment above.

Thanks,
Ludo’.

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

* bug#26795: gnu: emacs-ag: Propagate the-silver-searcher.
  2017-05-06 14:05 ` bug#26795: " Ludovic Courtès
@ 2017-05-11 16:44   ` Mathieu Othacehe
  2017-05-12  8:42     ` Clément Lassieur
  0 siblings, 1 reply; 5+ messages in thread
From: Mathieu Othacehe @ 2017-05-11 16:44 UTC (permalink / raw)
  To: Ludovic Courtès, Clément Lassieur; +Cc: 26795


Hi Clément & Ludo,

I also ran into the same issue. With Ludo's comment below addressed,
it's ok for me !

> I’d suggest writing the rationale for propagation in a comment above.

Thanks,

Mathieu

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

* bug#26795: gnu: emacs-ag: Propagate the-silver-searcher.
  2017-05-11 16:44   ` Mathieu Othacehe
@ 2017-05-12  8:42     ` Clément Lassieur
  0 siblings, 0 replies; 5+ messages in thread
From: Clément Lassieur @ 2017-05-12  8:42 UTC (permalink / raw)
  To: Mathieu Othacehe; +Cc: 26795-done

Mathieu Othacehe <m.othacehe@gmail.com> writes:

> Hi Clément & Ludo,
>
> I also ran into the same issue. With Ludo's comment below addressed,
> it's ok for me !
>
>> I’d suggest writing the rationale for propagation in a comment above.

Ok I pushed then, with the comment.  Thanks for the review :-)

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

end of thread, other threads:[~2017-05-12  8:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-06  9:04 bug#26795: gnu: emacs-ag: Propagate the-silver-searcher Clément Lassieur
2017-05-06  9:07 ` bug#26795: [PATCH] " Clément Lassieur
2017-05-06 14:05 ` bug#26795: " Ludovic Courtès
2017-05-11 16:44   ` Mathieu Othacehe
2017-05-12  8:42     ` Clément Lassieur

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.