[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 4/4] emacs: Add 'guix-devel-lint-package'
From: |
Alex Kost |
Subject: |
[PATCH 4/4] emacs: Add 'guix-devel-lint-package' |
Date: |
Fri, 2 Oct 2015 16:04:06 +0300 |
Suggested by Ludovic Courtès <address@hidden>.
* emacs/guix-devel.el (guix-devel-setup-repl): Add a workaround for
'guix-warning-port'.
(guix-devel-lint-package): New command.
(guix-devel-keys-map): Add key binding for it.
* doc/emacs.texi (Emacs Development): Document it.
---
doc/emacs.texi | 4 ++++
emacs/guix-devel.el | 21 ++++++++++++++++++---
2 files changed, 22 insertions(+), 3 deletions(-)
diff --git a/doc/emacs.texi b/doc/emacs.texi
index 00f86f3..18009fa 100644
--- a/doc/emacs.texi
+++ b/doc/emacs.texi
@@ -667,6 +667,10 @@ this command---for example, with @kbd{C-M-x} (@pxref{To
eval or not to
eval,,, geiser, Geiser User Manual})
(@code{guix-devel-build-package-definition}).
address@hidden C-c . l
+Lint (check) a package defined by the current variable definition
+(@pxref{Invoking guix lint}) (@code{guix-devel-lint-package}).
+
@item C-c . s
Download a source of the package defined by the current variable
definition and print its SHA256 hash. This command has the same meaning
diff --git a/emacs/guix-devel.el b/emacs/guix-devel.el
index a8167a0..4d380ce 100644
--- a/emacs/guix-devel.el
+++ b/emacs/guix-devel.el
@@ -71,11 +71,14 @@ Interactively, use the module defined by the current scheme
file."
(guix-devel-use-modules "(guix monad-repl)"
"(guix packages)"
"(guix scripts)"
- "(guix store)")
- ;; Without this workaround, the build output disappears. See
+ "(guix store)"
+ "(guix ui)")
+ ;; Without this workaround, the warning/build output disappears. See
;; <https://github.com/jaor/geiser/issues/83> for details.
(guix-geiser-eval-in-repl
- "(current-build-output-port (current-error-port))"
+ "(begin
+ (guix-warning-port (current-warning-port))
+ (current-build-output-port (current-error-port)))"
repl 'no-history 'no-display))
(defvar guix-devel-repl-processes nil
@@ -126,6 +129,17 @@ Use this function to compute SHA256 hash of the package
source."
(format "(guix-download (origin-uri (package-source %s)))"
def)))))
+(defun guix-devel-lint-package ()
+ "Check the current package.
+See Info node `(guix) Invoking guix lint' for details."
+ (interactive)
+ (guix-devel-with-definition def
+ (guix-devel-use-modules "(guix scripts lint)")
+ (when (or (not guix-operation-confirm)
+ (y-or-n-p (format "Lint '%s' package?" def)))
+ (guix-geiser-eval-in-repl
+ (format "(run-checkers %s)" def)))))
+
;;; Font-lock
@@ -164,6 +178,7 @@ to find 'modify-phases' keywords."
(defvar guix-devel-keys-map
(let ((map (make-sparse-keymap)))
(define-key map (kbd "b") 'guix-devel-build-package-definition)
+ (define-key map (kbd "l") 'guix-devel-lint-package)
(define-key map (kbd "s") 'guix-devel-download-package-source)
(define-key map (kbd "k") 'guix-devel-copy-module-as-kill)
(define-key map (kbd "u") 'guix-devel-use-module)
--
2.5.0
- emacs: devel: Add lint/download commands., Alex Kost, 2015/10/08
- [PATCH 3/4] lint: Export 'run-checkers'., Alex Kost, 2015/10/08
- [PATCH 4/4] emacs: Add 'guix-devel-lint-package',
Alex Kost <=
- [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Alex Kost, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Ludovic Courtès, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Alex Kost, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Ludovic Courtès, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Alex Kost, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Ludovic Courtès, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Alex Kost, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Ludovic Courtès, 2015/10/08
- Re: [PATCH 2/4] emacs: Add 'guix-devel-download-package-source'., Alex Kost, 2015/10/08