From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Andreas Rottmann Newsgroups: gmane.lisp.guile.devel Subject: Re: SLIB support Date: Wed, 12 Sep 2007 17:30:40 +0200 Message-ID: <87veaf3nq7.fsf@gkar.rotty.yi.org> References: <874pi13xer.fsf@gkar.rotty.yi.org> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-Trace: sea.gmane.org 1189611084 7621 80.91.229.12 (12 Sep 2007 15:31:24 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Wed, 12 Sep 2007 15:31:24 +0000 (UTC) To: Guile Developers Original-X-From: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Wed Sep 12 17:31:19 2007 Return-path: Envelope-to: guile-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1IVUB5-0003AN-J6 for guile-devel@m.gmane.org; Wed, 12 Sep 2007 17:30:56 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IVUB5-00065O-AM for guile-devel@m.gmane.org; Wed, 12 Sep 2007 11:30:55 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IVUB1-00063d-UC for guile-devel@gnu.org; Wed, 12 Sep 2007 11:30:51 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IVUB0-00062Y-CC for guile-devel@gnu.org; Wed, 12 Sep 2007 11:30:51 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IVUB0-00062B-4s for guile-devel@gnu.org; Wed, 12 Sep 2007 11:30:50 -0400 Original-Received: from mail.gmx.net ([213.165.64.20]) by monty-python.gnu.org with smtp (Exim 4.60) (envelope-from ) id 1IVUAy-00005C-MZ for guile-devel@gnu.org; Wed, 12 Sep 2007 11:30:49 -0400 Original-Received: (qmail invoked by alias); 12 Sep 2007 15:30:47 -0000 Original-Received: from chello084112165172.30.11.vie.surfer.at (EHLO localhost.localdomain) [84.112.165.172] by mail.gmx.net (mp043) with SMTP; 12 Sep 2007 17:30:47 +0200 X-Authenticated: #3102804 X-Provags-ID: V01U2FsdGVkX1+gok09ciWgrkBaAmmrBG9OnR98uchM2rFCFuvt3/ 5MJd9yX0DxzHQk Original-Received: from rotty by localhost.localdomain with local (Exim 4.67) (envelope-from ) id 1IVUAq-0005nl-GX for guile-devel@gnu.org; Wed, 12 Sep 2007 17:30:46 +0200 In-Reply-To: <874pi13xer.fsf@gkar.rotty.yi.org> (Andreas Rottmann's message of "Tue\, 11 Sep 2007 19\:49\:16 +0200") User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.1 (gnu/linux) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: a.rottmann@gmx.at X-SA-Exim-Version: 4.2.1 (built Tue, 21 Aug 2007 23:39:36 +0000) X-SA-Exim-Scanned: Yes (on localhost.localdomain) X-Y-GMX-Trusted: 0 X-Detected-Kernel: Linux 2.6, seldom 2.4 (older, 4) X-BeenThere: guile-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Developers list for Guile, the GNU extensibility library" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Errors-To: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.lisp.guile.devel:6784 Archived-At: --=-=-= Andreas Rottmann writes: > Hi! > > Currently on my System (Debian sid), Guile 1.8 does not support > SLIB. I have done a preliminary patch (attached) that makes the module > (ice-9 slib) load the guile.init file from SLIB, which seems to work > fine AFAICS. > > Currently, the patch hard-codes the location of the guile.init file, > (which would be suitable for Debian use, but not for upstream); I had > the intention to improve this (using a path figured out by > configure.ac), if this is the right way to do this; another option > would be placing the contents of guile.init in the (ice-9 slib) > module. So which is the way forward? I'll prepare patches for Guile > upstream once I know what the right approach to this problem is... > I've now prepared a patch (attached) that's hopefully good enough for upstream use. --=-=-= Content-Type: text/x-diff Content-Disposition: attachment; filename=slib.diff Index: guile-1.8.2/ice-9/slib.scm =================================================================== --- guile-1.8.2.orig/ice-9/slib.scm 2007-09-12 17:12:42.000000000 +0200 +++ guile-1.8.2/ice-9/slib.scm 2007-09-12 17:17:13.000000000 +0200 @@ -29,386 +29,8 @@ logical:bit-extract logical:integer-expt logical:ipow-by-squaring slib:eval-load slib:tab slib:form-feed difftime offset-time software-type) - :replace (delete-file open-file provide provided? system) + :replace (provide provided?) :no-backtrace) - -(define (eval-load evl) - (if (not (file-exists? )) - (set! (string-append (scheme-file-suffix)))) - (call-with-input-file - (lambda (port) - (let ((old-load-pathname *load-pathname*)) - (set! *load-pathname* ) - (do ((o (read port) (read port))) - ((eof-object? o)) - (evl o)) - (set! *load-pathname* old-load-pathname))))) - - - -(define slib:exit quit) -(define slib:error error) -(define slib:warn warn) -(define slib:eval (lambda (x) (eval x slib-module))) -(define defmacro:eval (lambda (x) (eval x (interaction-environment)))) -(define logical:logand logand) -(define logical:logior logior) -(define logical:logxor logxor) -(define logical:lognot lognot) -(define logical:ash ash) -(define logical:logcount logcount) -(define logical:integer-length integer-length) -(define logical:bit-extract bit-extract) -(define logical:integer-expt integer-expt) -(define slib:eval-load eval-load) -(define slib:tab #\tab) -(define slib:form-feed #\page) - -(define slib-module (current-module)) - -(define (defined? symbol) - (module-defined? slib-module symbol)) - -;;; *FEATURES* should be set to a list of symbols describing features -;;; of this implementation. Suggestions for features are: -(set! *features* - (append - '( - source ;can load scheme source files - ;(slib:load-source "filename") -; compiled ;can load compiled files - ;(slib:load-compiled "filename") - - ;; Scheme report features - -; rev5-report ;conforms to - eval ;R5RS two-argument eval -; values ;R5RS multiple values - dynamic-wind ;R5RS dynamic-wind -; macro ;R5RS high level macros - delay ;has DELAY and FORCE - multiarg-apply ;APPLY can take more than 2 args. -; rationalize - rev4-optional-procedures ;LIST-TAIL, STRING->LIST, - ;LIST->STRING, STRING-COPY, - ;STRING-FILL!, LIST->VECTOR, - ;VECTOR->LIST, and VECTOR-FILL! - -; rev4-report ;conforms to - -; ieee-p1178 ;conforms to - -; rev3-report ;conforms to - - rev2-procedures ;SUBSTRING-MOVE-LEFT!, - ;SUBSTRING-MOVE-RIGHT!, - ;SUBSTRING-FILL!, - ;STRING-NULL?, APPEND!, 1+, - ;-1+, ?, >=? -; object-hash ;has OBJECT-HASH - - multiarg/and- ;/ and - can take more than 2 args. - with-file ;has WITH-INPUT-FROM-FILE and - ;WITH-OUTPUT-FROM-FILE -; transcript ;TRANSCRIPT-ON and TRANSCRIPT-OFF -; ieee-floating-point ;conforms to IEEE Standard 754-1985 - ;IEEE Standard for Binary - ;Floating-Point Arithmetic. - full-continuation ;can return multiple times - - ;; Other common features - -; srfi ;srfi-0, COND-EXPAND finds all srfi-* -; sicp ;runs code from Structure and - ;Interpretation of Computer - ;Programs by Abelson and Sussman. - defmacro ;has Common Lisp DEFMACRO -; record ;has user defined data structures - string-port ;has CALL-WITH-INPUT-STRING and - ;CALL-WITH-OUTPUT-STRING -; sort -; pretty-print -; object->string -; format ;Common-lisp output formatting -; trace ;has macros: TRACE and UNTRACE -; compiler ;has (COMPILER) -; ed ;(ED) is editor - - ;; core definitions compatible, plus `make-random-state' below - random - ) - - (if (defined? 'getenv) - '(getenv) - '()) - - (if (defined? 'current-time) - '(current-time) - '()) - - (if (defined? 'system) - '(system) - '()) - - (if (defined? 'char-ready?) - '(char-ready?) - '()) - - *features*)) - -;; The array module specified by slib 3a1 is not the same as what guile -;; provides, so we must remove `array' from the features list. -;; -;; The main difference is `create-array' which is similar to -;; `make-uniform-array', but the `Ac64' etc prototype procedures incorporate -;; an initial fill element into the prototype. -;; -;; Believe the array-for-each module will need to be taken from slib when -;; the array module is taken from there, since what the array module creates -;; won't be understood by the guile functions. So remove `array-for-each' -;; from the features list too. -;; -;; Also, slib 3a1 array-for-each specifies an `array-map' which is not in -;; guile (but could be implemented quite easily). -;; -;; ENHANCE-ME: It'd be nice to implement what's necessary, since the guile -;; functions should be more efficient than the implementation in slib. -;; -;; FIXME: Since the *features* variable is shared by slib and the guile -;; core, removing these feature symbols has the unhappy effect of making it -;; look like they aren't in the core either. Let's assume that arrays have -;; been present unconditionally long enough that no guile-specific code will -;; bother to test. An alternative would be to make a new separate -;; *features* variable which the slib stuff operated on, leaving the core -;; mechanism alone. That might be a good thing anyway. -;; -(set! *features* (delq 'array *features*)) -(set! *features* (delq 'array-for-each *features*)) - -;; The random module in slib 3a1 provides a `random:chunk' which is used by -;; the random-inexact module. Guile doesn't provide random:chunk so we must -;; remove 'random from `*features*' to use the slib code. -;; -;; ENHANCE-ME: Maybe Guile could provide a `random:chunk', the rest of the -;; random module is already the same as Guile. -;; -;; FIXME: As per the array bits above, *features* is shared by slib and the -;; guile core, so removing 'random has the unhappy effect of making it look -;; like this isn't in the core. Let's assume random numbers have been -;; present unconditionally long enough that no guile-specific code will -;; bother to test. -;; -(set! *features* (delq 'random *features*)) - - -;;; FIXME: Because uers want require to search the path, this uses -;;; load-from-path, which probably isn't a hot idea. slib -;;; doesn't expect this function to search a path, so I expect to get -;;; bug reports at some point complaining that the wrong file gets -;;; loaded when something accidentally appears in the path before -;;; slib, etc. ad nauseum. However, the right fix seems to involve -;;; changing catalog:get in slib/require.scm, and I don't expect -;;; Aubrey will integrate such a change. So I'm just going to punt -;;; for the time being. -(define (slib:load name) - (save-module-excursion - (lambda () - (set-current-module slib-module) - (let ((errinfo (catch 'system-error - (lambda () - (load-from-path name) - #f) - (lambda args args)))) - (if (and errinfo - (catch 'system-error - (lambda () - (load-from-path - (string-append name ".scm")) - #f) - (lambda args args))) - (apply throw errinfo)))))) - -(define slib:load-source slib:load) -(define defmacro:load slib:load) - -(define slib-parent-dir - (let* ((path (%search-load-path "slib/require.scm"))) - (if path - (substring path 0 (- (string-length path) 17)) - (error "Could not find slib/require.scm in " %load-path)))) - -(define (implementation-vicinity) - (string-append slib-parent-dir "/")) -(define (library-vicinity) - (string-append (implementation-vicinity) "slib/")) -(define home-vicinity - (let ((home-path (getenv "HOME"))) - (lambda () home-path))) -(define (scheme-implementation-type) 'guile) -(define scheme-implementation-version version) -;;; (scheme-implementation-home-page) should return a (string) URI -;;; (Uniform Resource Identifier) for this scheme implementation's home -;;; page; or false if there isn't one. -(define (scheme-implementation-home-page) - "http://www.gnu.org/software/guile/guile.html") - -;; legacy from r3rs, but slib says all implementations provide these -;; ("Legacy" section of the "Miscellany" node in the manual) -(define-public t #t) -(define-public nil #f) - -;; ENHANCE-ME: Could call ioctl TIOCGWINSZ to get the size of a tty (see -;; "man 4 tty_ioctl" on a GNU/Linux system), on systems with that. -(define (output-port-width . arg) 80) -(define (output-port-height . arg) 24) - -;; slib 3a1 and up, straight from Template.scm -(define-public (call-with-open-ports . ports) - (define proc (car ports)) - (cond ((procedure? proc) (set! ports (cdr ports))) - (else (set! ports (reverse ports)) - (set! proc (car ports)) - (set! ports (reverse (cdr ports))))) - (let ((ans (apply proc ports))) - (for-each close-port ports) - ans)) - -;; slib (version 3a1) requires open-file accept a symbol r, rb, w or wb for -;; MODES, so extend the guile core open-file accordingly. -;; -;; slib (version 3a1) also calls open-file with strings "rb" or "wb", not -;; sure if that's intentional, but in any case this extension continues to -;; accept strings to make that work. -;; -(define-public (open-file filename modes) - (if (symbol? modes) - (set! modes (symbol->string modes))) - ((@ (guile) open-file) filename modes)) - -;; returning #t/#f instead of throwing an error for failure -(define-public (delete-file filename) - (catch 'system-error - (lambda () ((@ (guile) delete-file) filename) #t) - (lambda args #f))) - -;; Nothing special to do for this, so straight from Template.scm. Maybe -;; "sensible-browser" for a debian system would be worth trying too (and -;; would be good on a tty). -(define-public (browse-url url) - (define (try cmd end) (zero? (system (string-append cmd url end)))) - (or (try "netscape-remote -remote 'openURL(" ")'") - (try "netscape -remote 'openURL(" ")'") - (try "netscape '" "'&") - (try "netscape '" "'"))) - -;;; {array-for-each} -(define (array-indexes ra) - (let ((ra0 (apply make-array '() (array-shape ra)))) - (array-index-map! ra0 list) - ra0)) - -;;; {Random numbers} -;;; -(define (make-random-state . args) - (let ((seed (if (null? args) *random-state* (car args)))) - (cond ((string? seed)) - ((number? seed) (set! seed (number->string seed))) - (else (let () - (require 'object->string) - (set! seed (object->limited-string seed 50))))) - (seed->random-state seed))) - -;;; {rev2-procedures} -;;; - -(define -1+ 1-) -(define ? >) -(define >=? >=) - -;;; {system} -;;; -;; If the program run is killed by a signal, the shell normally gives an -;; exit code of 128+signum. If the shell itself is killed by a signal then -;; we do the same 128+signum here. -;; -;; "stop-sig" shouldn't arise here, since system shouldn't be calling -;; waitpid with WUNTRACED, but allow for it anyway, just in case. -;; -(if (memq 'system *features*) - (define-public system - (lambda (str) - (let ((st ((@ (guile) system) str))) - (or (status:exit-val st) - (+ 128 (or (status:term-sig st) - (status:stop-sig st)))))))) - -;;; {Time} -;;; - -(define difftime -) -(define offset-time +) - - -(define define - (procedure->memoizing-macro - (lambda (exp env) - (if (= (length env) 1) - `(define-public ,@(cdr exp)) - `(define-private ,@(cdr exp)))))) - -;;; Hack to make syncase macros work in the slib module -(if (nested-ref the-root-module '(app modules ice-9 syncase)) - (set-object-property! (module-local-variable (current-module) 'define) - '*sc-expander* - '(define))) - -(define (software-type) - "Return a symbol describing the current platform's operating system. -This may be one of AIX, VMS, UNIX, COHERENT, WINDOWS, MS-DOS, OS/2, -THINKC, AMIGA, ATARIST, MACH, or ACORN. - -Note that most varieties of Unix are considered to be simply \"UNIX\". -That is because when a program depends on features that are not present -on every operating system, it is usually better to test for the presence -or absence of that specific feature. The return value of -@code{software-type} should only be used for this purpose when there is -no other easy or unambiguous way of detecting such features." - 'UNIX) - -(slib:load (in-vicinity (library-vicinity) "require.scm")) - -(define require require:require) - -;; {Extensions to the require system so that the user can add new -;; require modules easily.} - -(define *vicinity-table* - (list - (cons 'implementation (implementation-vicinity)) - (cons 'library (library-vicinity)))) - -(define (install-require-vicinity name vicinity) - (let ((entry (assq name *vicinity-table*))) - (if entry - (set-cdr! entry vicinity) - (set! *vicinity-table* - (acons name vicinity *vicinity-table*))))) - -(define (install-require-module name vicinity-name file-name) - (if (not *catalog*) ;Fix which loads catalog in slib - (catalog:get 'random)) ;(doesn't load the feature 'random) - (let ((entry (assq name *catalog*)) - (vicinity (cdr (assq vicinity-name *vicinity-table*)))) - (let ((path-name (in-vicinity vicinity file-name))) - (if entry - (set-cdr! entry path-name) - (set! *catalog* - (acons name path-name *catalog*)))))) - -(define (make-exchanger obj) - (lambda (rep) (let ((old obj)) (set! obj rep) old))) +(load "/usr/share/slib/init/guile.init") Index: guile-1.8.2/configure.in =================================================================== --- guile-1.8.2.orig/configure.in 2007-09-12 17:12:42.000000000 +0200 +++ guile-1.8.2/configure.in 2007-09-12 17:14:39.000000000 +0200 @@ -171,6 +171,12 @@ [ --disable-elisp omit Emacs Lisp support],, enable_elisp=yes) +AC_ARG_WITH([slib], + AC_HELP_STRING([--with-slib=LOCATION], [Use SLIB located in LOCATION]), + [SLIB_LOCATION="$withval"], + [SLIB_LOCATION="$prefix/share/slib"]) +AC_SUBST(SLIB_LOCATION) + #-------------------------------------------------------------------- dnl Check for dynamic linking @@ -1352,6 +1358,7 @@ guile-config/Makefile ice-9/Makefile ice-9/debugger/Makefile + ice-9/slib.scm lang/Makefile lang/elisp/Makefile lang/elisp/internals/Makefile Index: guile-1.8.2/ice-9/slib.scm.in =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ guile-1.8.2/ice-9/slib.scm.in 2007-09-12 17:14:39.000000000 +0200 @@ -0,0 +1,36 @@ +;;;; slib.scm --- definitions needed to get SLIB to work with Guile +;;;; +;;;; Copyright (C) 1997, 1998, 2000, 2001, 2002, 2003, 2004, 2006 Free Software Foundation, Inc. +;;;; +;;;; This library is free software; you can redistribute it and/or +;;;; modify it under the terms of the GNU Lesser General Public +;;;; License as published by the Free Software Foundation; either +;;;; version 2.1 of the License, or (at your option) any later version. +;;;; +;;;; This library is distributed in the hope that it will be useful, +;;;; but WITHOUT ANY WARRANTY; without even the implied warranty of +;;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +;;;; Lesser General Public License for more details. +;;;; +;;;; You should have received a copy of the GNU Lesser General Public +;;;; License along with this library; if not, write to the Free Software +;;;; Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA +;;;; +(define-module (ice-9 slib) + :export (slib:load slib:load-source defmacro:load + implementation-vicinity library-vicinity home-vicinity + scheme-implementation-type scheme-implementation-version + output-port-width output-port-height array-indexes + make-random-state + -1+ ? >=? + require slib:error slib:exit slib:warn slib:eval + defmacro:eval logical:logand logical:logior logical:logxor + logical:lognot logical:ash logical:logcount logical:integer-length + logical:bit-extract logical:integer-expt logical:ipow-by-squaring + slib:eval-load slib:tab slib:form-feed difftime offset-time + software-type) + :replace (provide provided?) + :no-backtrace) + + +(load "@SLIB_LOCATION@/init/guile.init") --=-=-= Regards, Rotty -- Andreas Rottmann | Rotty@ICQ | 118634484@ICQ | a.rottmann@gmx.at http://rotty.uttx.net | GnuPG Key: http://rotty.uttx.net/gpg.asc Fingerprint | C38A 39C5 16D7 B69F 33A3 6993 22C8 27F7 35A9 92E7 v2sw7MYChw5pr5OFma7u7Lw2m5g/l7Di6e6t5BSb7en6g3/5HZa2Xs6MSr1/2p7 hackerkey.com To iterate is human; to recurse, divine. --=-=-= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Guile-devel mailing list Guile-devel@gnu.org http://lists.gnu.org/mailman/listinfo/guile-devel --=-=-=--