From 5c08caccbec41d9c14781ece9a336f91da2c7ab8 Mon Sep 17 00:00:00 2001 From: Jose Antonio Ortega Ruiz Date: Wed, 25 Sep 2013 05:10:00 +0200 Subject: Scheme version checks And, if you happen to be launching it all the time, a way of skipping them via a customizable variable. Should address issue #15. --- NEWS | 3 ++- doc/repl.texi | 23 +++++++++++++++++------ elisp/geiser-base.el | 11 ++++++++++- elisp/geiser-repl.el | 25 +++++++++++++++++++++++++ 4 files changed, 54 insertions(+), 8 deletions(-) diff --git a/NEWS b/NEWS index 5cb751c..e5e4606 100644 --- a/NEWS +++ b/NEWS @@ -2,7 +2,7 @@ New features: - - Geiser is now available from MELPA. + - Geiser is now available from MELPA, with the help of Steve Purcell. - Racket: new commands geiser-racket-{show, hide, toggle}-submodules, for folding submodule forms in code buffers. - Racket: interaction with submodules (entering them and evaluation @@ -12,6 +12,7 @@ - Pressing return on a previous expression in the REPL will resend it, by Darren Hoo. - Improvements to syntax highlighting (define/match in racket). + - Version checks for the underlying Scheme process. Bug fixes: diff --git a/doc/repl.texi b/doc/repl.texi index 8b32de7..acab26e 100644 --- a/doc/repl.texi +++ b/doc/repl.texi @@ -345,16 +345,27 @@ not the case, the variables to tweak are @code{geiser-guile-binary} and @code{geiser-racket-binary}, which should be set to a string with the full path to the requisite binary. +@cindex Version checking +Before starting the REPL, Geiser will check wether the version of your +Scheme interpreter is good enough. This means that it will spend a +couple tenths of a second launching and quickly discarding a Scheme +process, but also that the error message you'll get if you're on the +wrong Scheme version will be much more informative. If you one to +avoid version checks, just check +@code{geiser-repl-skip-version-check-p} to @code{t} in your +configuration. + @cindex scheme load path @cindex scheme init file @cindex GUILE_LOAD_PATH +@cindex GUILE_LOAD_COMPILED_PATH @cindex PLTCOLLECTS -You can also specify a couple more initialisation parameters. For Guile, -@code{geiser-guile-load-path} is a list of paths to add to its load path -when it's started, while @code{geiser-guile-init-file} is the path to an -initialisation file to be loaded on start-up. The equivalent variables -for Racket are @code{geiser-racket-collects} and -@code{geiser-racket-init-file}. +You can also specify a couple more initialisation parameters. For +Guile, @code{geiser-guile-load-path} is a list of paths to add to its +load path (and its compiled load path) when it's started, while +@code{geiser-guile-init-file} is the path to an initialisation file to +be loaded on start-up. The equivalent variables for Racket are +@code{geiser-racket-collects} and @code{geiser-racket-init-file}. Note, however, that specifying @code{geiser-guile-init-file} is @i{not} equivalent to changing Guile's initialization file (@file{~/.guile}), diff --git a/elisp/geiser-base.el b/elisp/geiser-base.el index 2569034..e564b0f 100644 --- a/elisp/geiser-base.el +++ b/elisp/geiser-base.el @@ -1,6 +1,6 @@ ;;; geiser-base.el --- shared bits -;; Copyright (C) 2009, 2010, 2012 Jose Antonio Ortega Ruiz +;; Copyright (C) 2009, 2010, 2012, 2013 Jose Antonio Ortega Ruiz ;; This program is free software; you can redistribute it and/or ;; modify it under the terms of the Modified BSD License. You should @@ -74,5 +74,14 @@ (let ((thing (thing-at-point 'symbol))) (and thing (make-symbol thing)))) +(defun geiser--cut-version (v) + (when (string-match "\\([0-9]+\\.[0-9]\\(?:\\.[0-9]+\\)?\\).*" v) + (match-string 1 v))) + +(defun geiser--version< (v1 v2) + (let ((v1 (geiser--cut-version v1)) + (v2 (geiser--cut-version v2))) + (and v1 v2 (version< v1 v2)))) + (provide 'geiser-base) diff --git a/elisp/geiser-repl.el b/elisp/geiser-repl.el index 68e4708..ebb74c2 100644 --- a/elisp/geiser-repl.el +++ b/elisp/geiser-repl.el @@ -105,6 +105,16 @@ expression, if any." :type 'boolean :group 'geiser-repl) +(geiser-custom--defcustom geiser-repl-skip-version-check-p nil + "Whether to skip version checks for the Scheme executable. + +When set, Geiser won't check the version of the Scheme +interpreter when starting a REPL, saving a few tenths of a +second. +" + :type 'boolean + :group 'geiser-repl) + (geiser-custom--defcustom geiser-repl-query-on-exit-p nil "Whether to prompt for confirmation on \\[geiser-repl-exit]." :type 'boolean @@ -188,6 +198,13 @@ module command as a string") (geiser-impl--define-caller geiser-repl--exit-cmd exit-command () "Function returning the REPL exit command as a string") +(geiser-impl--define-caller geiser-repl--version version-command (binary) + "Function returning the version of the corresponding scheme process, + given its full path.") + +(geiser-impl--define-caller geiser-repl--min-version minimum-version () + "A variable providing the minimum required scheme version, as a string.") + ;;; Geiser REPL buffers and processes: @@ -355,6 +372,13 @@ module command as a string") geiser-repl-query-on-kill-p) (message "%s up and running!" (geiser-repl--repl-name impl)))) +(defun geiser-repl--check-version (impl) + (when (not geiser-repl-skip-version-check-p) + (let ((v (geiser-repl--version impl (geiser-repl--binary impl))) + (r (geiser-repl--min-version impl))) + (when (geiser--version< v r) + (error "Geiser requires %s version %s but detected %s" impl r v))))) + (defun geiser-repl--start-scheme (impl address prompt) (setq comint-prompt-regexp prompt) (let* ((name (geiser-repl--repl-name impl)) @@ -363,6 +387,7 @@ module command as a string") `(,(geiser-repl--binary impl) nil ,@(geiser-repl--arglist impl))))) + (when (not address) (geiser-repl--check-version impl)) (condition-case err (apply 'make-comint-in-buffer `(,name ,buff ,@args)) (error (insert "Unable to start REPL:\n" -- cgit v1.2.3