integrated Raymond Wiker's patches to port RUN-PROGRAM from CMU CL and
[sbcl.git] / src / code / run-program.lisp
index aadfe81..d704435 100644 (file)
@@ -1,20 +1,5 @@
-.. not working .. not working .. not working .. not working ..
-
-KLUDGE: This is CMU CL code which needs more porting before it can
-work on SBCL. At the very least:
-  * Package references need to be renamed from the CMU CL "SYSTEM" style
-    to the SBCL "SB-SYS" style. Possibly some referenced symbols have
-    moved to new packages or been renamed, as well.
-  * The environment-handling needs to be updated to read directly from
-    the Unix environment, since SBCL, unlike CMU CL, doesn't maintain
-    its own local copy.
-  * The DEFCONSTANT #+SVR4 stuff needs to be checked and cleaned up for
-    currently supported OSes, since SBCL doesn't use the :SVR4 feature.
-  * The conditional code for other stuff not supported by SBCL (e.g.
-    HPUX) should probably go away.
--- WHN 20000825
-
-;;;; support for running Unix programs from inside Lisp
+;;;; RUN-PROGRAM and friends, a facility for running Unix programs
+;;;; from inside SBCL
 
 ;;;; This software is part of the SBCL system. See the README file for
 ;;;; more information.
@@ -32,10 +17,10 @@ work on SBCL. At the very least:
 \f
 ;;;; Import wait3(2) from Unix.
 
-(alien:def-alien-routine ("wait3" c-wait3) c-call:int
-  (status c-call:int :out)
-  (options c-call:int)
-  (rusage c-call:int))
+(sb-alien:def-alien-routine ("wait3" c-wait3) sb-c-call:int
+  (status sb-c-call:int :out)
+  (options sb-c-call:int)
+  (rusage sb-c-call:int))
 
 (eval-when (load eval compile)
   (defconstant wait-wnohang #-svr4 1 #+svr4 #o100)
@@ -43,15 +28,15 @@ work on SBCL. At the very least:
   (defconstant wait-wstopped #-svr4 #o177 #+svr4 wait-wuntraced))
 
 (defun wait3 (&optional do-not-hang check-for-stopped)
-  "Return any available status information on child process."
+  "Return any available status information on child process. "
   (multiple-value-bind (pid status)
-                      (c-wait3 (logior (if do-not-hang
-                                           wait-wnohang
-                                           0)
-                                       (if check-for-stopped
-                                           wait-wuntraced
-                                           0))
-                               0)
+      (c-wait3 (logior (if do-not-hang
+                          wait-wnohang
+                          0)
+                      (if check-for-stopped
+                          wait-wuntraced
+                          0))
+              0)
     (cond ((or (minusp pid)
               (zerop pid))
           nil)
@@ -67,39 +52,40 @@ work on SBCL. At the very least:
          (t
           (let ((signal (ldb (byte 7 0) status)))
             (values pid
-                    (if (or (eql signal unix:sigstop)
-                            (eql signal unix:sigtstp)
-                            (eql signal unix:sigttin)
-                            (eql signal unix:sigttou))
-                      :stopped
-                      :signaled)
+                    (if (or (eql signal sb-unix:sigstop)
+                            (eql signal sb-unix:sigtstp)
+                            (eql signal sb-unix:sigttin)
+                            (eql signal sb-unix:sigttou))
+                        :stopped
+                        :signaled)
                     signal
                     (not (zerop (ldb (byte 1 7) status)))))))))
 \f
-;;;; stuff for process control
+;;;; process control stuff
 
 (defvar *active-processes* nil
   "List of process structures for all active processes.")
 
-(defstruct (process (:print-function %print-process))
-  pid                      ; PID of child process
-  %status                  ; either :RUNNING, :STOPPED, :EXITED, or :SIGNALED
-  exit-code                ; either exit code or signal
-  core-dumped              ; T if a core image was dumped
-  pty                      ; stream to child's pty, or NIL
-  input                            ; stream to child's input, or NIL
-  output                   ; stream from child's output, or NIL
-  error                            ; stream from child's error output, or NIL
-  status-hook              ; closure to call when PROC changes status
-  plist                            ; a place for clients to stash things
-  cookie                   ; list of the number of pipes from the subprocess
-  )
-
-(defun %print-process (proc stream depth)
-  (declare (ignore depth))
-  (format stream "#<PROCESS ~D ~S>"
-         (process-pid proc)
-         (process-status proc)))
+(defstruct (process)
+  pid                ; PID of child process
+  %status             ; either :RUNNING, :STOPPED, :EXITED, or :SIGNALED
+  exit-code          ; either exit code or signal
+  core-dumped        ; T if a core image was dumped
+  pty                ; stream to child's pty, or NIL
+  input                      ; stream to child's input, or NIL
+  output             ; stream from child's output, or NIL
+  error                      ; stream from child's error output, or NIL
+  status-hook        ; closure to call when PROC changes status
+  plist                      ; a place for clients to stash things
+  cookie)             ; list of the number of pipes from the subproc
+
+(defmethod print-object ((process process) stream)
+  (print-unreadable-object (process stream :type t)
+    (format stream
+           "~D ~S"
+           (process-pid process)
+           (process-status process)))
+  process)
 
 (defun process-status (proc)
   "Return the current status of process.  The result is one of :RUNNING,
@@ -110,36 +96,37 @@ work on SBCL. At the very least:
 (defun process-wait (proc &optional check-for-stopped)
   "Wait for PROC to quit running for some reason.  Returns PROC."
   (loop
-    (case (process-status proc)
-      (:running)
-      (:stopped
-       (when check-for-stopped
-        (return)))
-      (t
-       (when (zerop (car (process-cookie proc)))
-        (return))))
-    (system:serve-all-events 1))
+      (case (process-status proc)
+       (:running)
+       (:stopped
+        (when check-for-stopped
+          (return)))
+       (t
+        (when (zerop (car (process-cookie proc)))
+          (return))))
+      (sb-sys:serve-all-events 1))
   proc)
 
+#-hpux
 ;;; Find the current foreground process group id.
 (defun find-current-foreground-process (proc)
-  (alien:with-alien ((result c-call:int))
+  (sb-alien:with-alien ((result sb-c-call:int))
     (multiple-value-bind
-       (wonp error)
-       (unix:unix-ioctl (system:fd-stream-fd (ext:process-pty proc))
-                        unix:TIOCGPGRP
-                        (alien:alien-sap (alien:addr result)))
+         (wonp error)
+       (sb-unix:unix-ioctl (sb-sys:fd-stream-fd (process-pty proc))
+                           sb-unix:TIOCGPGRP
+                           (sb-alien:alien-sap (sb-alien:addr result)))
       (unless wonp
        (error "TIOCPGRP ioctl failed: ~S"
-              (unix:get-unix-error-msg error)))
+              (sb-unix:get-unix-error-msg error)))
       result))
   (process-pid proc))
 
 (defun process-kill (proc signal &optional (whom :pid))
-  "Send SIGNAL to PROC.  If WHOM is :PID, then use the kill(2) Unix system
-   call. If WHOM is :PROCESS-GROUP, use the killpg(2) Unix system call.
-   If WHOM is :PTY-PROCESS-GROUP, then deliver the signal to whichever
-   process group is currently in the foreground."
+  "Hand SIGNAL to PROC.  If whom is :pid, use the kill Unix system call.  If
+   whom is :process-group, use the killpg Unix system call.  If whom is
+   :pty-process-group deliver the signal to whichever process group is currently
+   in the foreground."
   (let ((pid (ecase whom
               ((:pid :process-group)
                (process-pid proc))
@@ -147,22 +134,22 @@ work on SBCL. At the very least:
                #-hpux
                (find-current-foreground-process proc)))))
     (multiple-value-bind
-       (okay errno)
+         (okay errno)
        (case whom
          #+hpux
          (:pty-process-group
-          (unix:unix-ioctl (system:fd-stream-fd (process-pty proc))
-                           unix:TIOCSIGSEND
-                           (system:int-sap
-                            (unix:unix-signal-number signal))))
+          (sb-unix:unix-ioctl (sb-sys:fd-stream-fd (process-pty proc))
+                              sb-unix:TIOCSIGSEND
+                              (sb-sys:int-sap
+                               (sb-unix:unix-signal-number signal))))
          ((:process-group #-hpux :pty-process-group)
-          (unix:unix-killpg pid signal))
+          (sb-unix:unix-killpg pid signal))
          (t
-          (unix:unix-kill pid signal)))
+          (sb-unix:unix-kill pid signal)))
       (cond ((not okay)
             (values nil errno))
            ((and (eql pid (process-pid proc))
-                 (= (unix:unix-signal-number signal) unix:sigcont))
+                 (= (sb-unix:unix-signal-number signal) sb-unix:sigcont))
             (setf (process-%status proc) :running)
             (setf (process-exit-code proc) nil)
             (when (process-status-hook proc)
@@ -176,8 +163,8 @@ work on SBCL. At the very least:
   (let ((status (process-status proc)))
     (if (or (eq status :running)
            (eq status :stopped))
-      t
-      nil)))
+       t
+       nil)))
 
 (defun process-close (proc)
   "Close all streams connected to PROC and stop maintaining the status slot."
@@ -187,33 +174,33 @@ work on SBCL. At the very least:
     (frob (process-input  proc)   t) ; .. 'cause it will generate SIGPIPE.
     (frob (process-output proc) nil)
     (frob (process-error  proc) nil))
-  (system:without-interrupts
+  (sb-sys:without-interrupts
    (setf *active-processes* (delete proc *active-processes*)))
   proc)
 
-;;; the handler for SIGCHLD signals that RUN-PROGRAM establishes
+;;; the handler for sigchld signals that RUN-PROGRAM establishes
 (defun sigchld-handler (ignore1 ignore2 ignore3)
   (declare (ignore ignore1 ignore2 ignore3))
   (get-processes-status-changes))
 
 (defun get-processes-status-changes ()
   (loop
-    (multiple-value-bind (pid what code core)
-                        (wait3 t t)
-      (unless pid
-       (return))
-      (let ((proc (find pid *active-processes* :key #'process-pid)))
-       (when proc
-         (setf (process-%status proc) what)
-         (setf (process-exit-code proc) code)
-         (setf (process-core-dumped proc) core)
-         (when (process-status-hook proc)
-           (funcall (process-status-hook proc) proc))
-         (when (or (eq what :exited)
-                   (eq what :signaled))
-           (system:without-interrupts
-             (setf *active-processes*
-                   (delete proc *active-processes*)))))))))
+      (multiple-value-bind (pid what code core)
+         (wait3 t t)
+       (unless pid
+         (return))
+       (let ((proc (find pid *active-processes* :key #'process-pid)))
+         (when proc
+           (setf (process-%status proc) what)
+           (setf (process-exit-code proc) code)
+           (setf (process-core-dumped proc) core)
+           (when (process-status-hook proc)
+             (funcall (process-status-hook proc) proc))
+           (when (or (eq what :exited)
+                     (eq what :signaled))
+             (sb-sys:without-interrupts
+              (setf *active-processes*
+                    (delete proc *active-processes*)))))))))
 \f
 ;;;; RUN-PROGRAM and close friends
 
@@ -224,59 +211,69 @@ work on SBCL. At the very least:
 (defvar *handlers-installed* nil
   "List of handlers installed by RUN-PROGRAM.")
 
-;;; Find a pty that is not in use. Returns three values: the file
+#+FreeBSD
+(def-alien-type nil
+    (struct sgttyb
+           (sg-ispeed sb-c-call:char)  ; input speed
+           (sg-ospeed sb-c-call:char)  ; output speed
+           (sg-erase sb-c-call:char)   ; erase character
+           (sg-kill sb-c-call:char)    ; kill character
+           (sg-flags sb-c-call:short)  ; mode flags
+           ))
+
+;;; Find a pty that is not in use. Return three values: the file
 ;;; descriptor for the master side of the pty, the file descriptor for
 ;;; the slave side of the pty, and the name of the tty device for the
 ;;; slave side.
 (defun find-a-pty ()
-  "Returns the master fd, the slave fd, and the name of the tty"
   (dolist (char '(#\p #\q))
     (dotimes (digit 16)
       (let* ((master-name (format nil "/dev/pty~C~X" char digit))
-            (master-fd (unix:unix-open master-name
-                                       unix:o_rdwr
-                                       #o666)))
+            (master-fd (sb-unix:unix-open master-name
+                                          sb-unix:o_rdwr
+                                          #o666)))
        (when master-fd
          (let* ((slave-name (format nil "/dev/tty~C~X" char digit))
-                (slave-fd (unix:unix-open slave-name
-                                          unix:o_rdwr
-                                          #o666)))
+                (slave-fd (sb-unix:unix-open slave-name
+                                             sb-unix:o_rdwr
+                                             #o666)))
            (when slave-fd
-             ; Maybe put a vhangup here?
-              #-glibc2
-             (alien:with-alien ((stuff (alien:struct unix:sgttyb)))
-               (let ((sap (alien:alien-sap stuff)))
-                 (unix:unix-ioctl slave-fd unix:TIOCGETP sap)
-                 (setf (alien:slot stuff 'unix:sg-flags) #o300) ; EVENP|ODDP
-                 (unix:unix-ioctl slave-fd unix:TIOCSETP sap)
-                 (unix:unix-ioctl master-fd unix:TIOCGETP sap)
-                 (setf (alien:slot stuff 'unix:sg-flags)
-                       (logand (alien:slot stuff 'unix:sg-flags)
+                                       ; Maybe put a vhangup here?
+              #-linux
+             (sb-alien:with-alien ((stuff (sb-alien:struct sgttyb)))
+               (let ((sap (sb-alien:alien-sap stuff)))
+                 (sb-unix:unix-ioctl slave-fd sb-unix:TIOCGETP sap)
+                 (setf (sb-alien:slot stuff 'sg-flags)
+                       #o300)          ; EVENP|ODDP
+                 (sb-unix:unix-ioctl slave-fd sb-unix:TIOCSETP sap)
+                 (sb-unix:unix-ioctl master-fd sb-unix:TIOCGETP sap)
+                 (setf (sb-alien:slot stuff 'sg-flags)
+                       (logand (sb-alien:slot stuff 'sg-flags)
                                (lognot 8))) ; ~ECHO
-                 (unix:unix-ioctl master-fd unix:TIOCSETP sap)))
+                 (sb-unix:unix-ioctl master-fd sb-unix:TIOCSETP sap)))
              (return-from find-a-pty
-                          (values master-fd
-                                  slave-fd
-                                  slave-name)))
-         (unix:unix-close master-fd))))))
+               (values master-fd
+                       slave-fd
+                       slave-name)))
+           (sb-unix:unix-close master-fd))))))
   (error "could not find a pty"))
 
 (defun open-pty (pty cookie)
   (when pty
     (multiple-value-bind
-       (master slave name)
+         (master slave name)
        (find-a-pty)
       (push master *close-on-error*)
       (push slave *close-in-parent*)
       (when (streamp pty)
-       (multiple-value-bind (new-fd errno) (unix:unix-dup master)
+       (multiple-value-bind (new-fd errno) (sb-unix:unix-dup master)
          (unless new-fd
-           (error "could not UNIX:UNIX-DUP ~D: ~A"
-                  master (unix:get-unix-error-msg errno)))
+           (error "could not SB-UNIX:UNIX-DUP ~D: ~S"
+                  master (sb-unix:get-unix-error-msg errno)))
          (push new-fd *close-on-error*)
          (copy-descriptor-to-stream new-fd pty cookie)))
       (values name
-             (system:make-fd-stream master :input t :output t)))))
+             (sb-sys:make-fd-stream master :input t :output t)))))
 
 (defmacro round-bytes-to-words (n)
   `(logand (the fixnum (+ (the fixnum ,n) 3)) (lognot 3)))
@@ -294,19 +291,20 @@ work on SBCL. At the very least:
       (incf string-bytes (round-bytes-to-words (1+ (length s)))))
     ;; Now allocate the memory and fill it in.
     (let* ((total-bytes (+ string-bytes vec-bytes))
-          (vec-sap (system:allocate-system-memory total-bytes))
+          (vec-sap (sb-sys:allocate-system-memory total-bytes))
           (string-sap (sap+ vec-sap vec-bytes))
           (i #-alpha 4 #+alpha 8))
       (declare (type (and unsigned-byte fixnum) total-bytes i)
-              (type system:system-area-pointer vec-sap string-sap))
+              (type sb-sys:system-area-pointer vec-sap string-sap))
       (dolist (s string-list)
        (declare (simple-string s))
        (let ((n (length s)))
          ;; Blast the string into place.
-         (kernel:copy-to-system-area (the simple-string s)
-                                     (* vm:vector-data-offset vm:word-bits)
-                                     string-sap 0
-                                     (* (1+ n) vm:byte-bits))
+         (sb-kernel:copy-to-system-area (the simple-string s)
+                                        (* sb-vm:vector-data-offset
+                                           sb-vm:word-bits)
+                                        string-sap 0
+                                        (* (1+ n) sb-vm:byte-bits))
          ;; Blast the pointer to the string into place.
          (setf (sap-ref-sap vec-sap i) string-sap)
          (setf string-sap (sap+ string-sap (round-bytes-to-words (1+ n))))
@@ -319,71 +317,66 @@ work on SBCL. At the very least:
   (let ((sap (gensym "SAP-"))
        (size (gensym "SIZE-")))
     `(multiple-value-bind
-        (,sap ,var ,size)
-        (string-list-to-c-strvec ,str-list)
-       (unwind-protect
+      (,sap ,var ,size)
+      (string-list-to-c-strvec ,str-list)
+      (unwind-protect
           (progn
             ,@body)
-        (system:deallocate-system-memory ,sap ,size)))))
+       (sb-sys:deallocate-system-memory ,sap ,size)))))
 
-(alien:def-alien-routine spawn c-call:int
-  (program c-call:c-string)
-  (argv (* c-call:c-string))
-  (envp (* c-call:c-string))
-  (pty-name c-call:c-string)
-  (stdin c-call:int)
-  (stdout c-call:int)
-  (stderr c-call:int))
+(sb-alien:def-alien-routine spawn sb-c-call:int
+  (program sb-c-call:c-string)
+  (argv (* sb-c-call:c-string))
+  (envp (* sb-c-call:c-string))
+  (pty-name sb-c-call:c-string)
+  (stdin sb-c-call:int)
+  (stdout sb-c-call:int)
+  (stderr sb-c-call:int))
 
 ;;; RUN-PROGRAM uses fork and execve to run a different program.
 ;;; Strange stuff happens to keep the unix state of the world
 ;;; coherent.
 ;;;
-;;; The child process needs to get it's input from somewhere, and send it's
-;;; output (both standard and error) to somewhere. We have to do different
-;;; things depending on where these somewheres really are.
+;;; The child process needs to get it's input from somewhere, and send
+;;; its output (both standard and error) to somewhere. We have to do
+;;; different things depending on where these somewheres really are.
 ;;;
 ;;; For input, there are five options:
-;;; - T: Just leave fd 0 alone. Pretty simple.
-;;; - "file": Read from the file. We need to open the file and pull the
-;;; descriptor out of the stream. The parent should close this stream after
-;;; the child is up and running to free any storage used in the parent.
-;;; - NIL: Same as "file", but use "/dev/null" as the file.
-;;; - :STREAM: Use unix-pipe to create two descriptors. Use system:make-fd-stream
-;;; to create the output stream on the writeable descriptor, and pass the
-;;; readable descriptor to the child. The parent must close the readable
-;;; descriptor for EOF to be passed up correctly.
-;;; - a stream: If it's a fd-stream, just pull the descriptor out of it.
-;;; Otherwise make a pipe as in :STREAM, and copy everything across.
+;;;  -- T: Just leave fd 0 alone. Pretty simple.
+;;;  -- "file": Read from the file. We need to open the file and
+;;;     pull the descriptor out of the stream. The parent should close
+;;;     this stream after the child is up and running to free any 
+;;;     storage used in the parent.
+;;;  -- NIL: Same as "file", but use "/dev/null" as the file.
+;;;  -- :STREAM: Use unix-pipe to create two descriptors. Use
+;;;     sb-sys:make-fd-stream to create the output stream on the
+;;;     writeable descriptor, and pass the readable descriptor to
+;;;     the child. The parent must close the readable descriptor for
+;;;     EOF to be passed up correctly.
+;;;  -- a stream: If it's a fd-stream, just pull the descriptor out
+;;;     of it. Otherwise make a pipe as in :STREAM, and copy 
+;;;     everything across.
 ;;;
-;;; For output, there are n options:
-;;; - T: Leave descriptor 1 alone.
-;;; - "file": dump output to the file.
-;;; - NIL: dump output to /dev/null.
-;;; - :STREAM: return a stream that can be read from.
-;;; - a stream: if it's a fd-stream, use the descriptor in it. Otherwise, copy
-;;; stuff from output to stream.
+;;; For output, there are five options:
+;;;  -- T: Leave descriptor 1 alone.
+;;;  -- "file": dump output to the file.
+;;;  -- NIL: dump output to /dev/null.
+;;;  -- :STREAM: return a stream that can be read from.
+;;;  -- a stream: if it's a fd-stream, use the descriptor in it.
+;;;     Otherwise, copy stuff from output to stream.
 ;;;
 ;;; For error, there are all the same options as output plus:
-;;; - :OUTPUT: redirect to the same place as output.
+;;;  -- :OUTPUT: redirect to the same place as output.
 ;;;
-;;; RUN-PROGRAM returns a process struct for the process if the fork
-;;; worked, and NIL if it did not.
+;;; RUN-PROGRAM returns a PROCESS structure for the process if
+;;; the fork worked, and NIL if it did not.
 (defun run-program (program args
-                   &key
-                   (env *environment-list*)
-                   (wait t)
-                   pty
-                   input
-                   if-input-does-not-exist
-                   output
-                   (if-output-exists :error)
-                   (error :output)
-                   (if-error-exists :error)
-                   status-hook)
-  "RUN-PROGRAM creates a new process and runs the unix program in the
-   file specified by PROGRAM (a SIMPLE-STRING).  ARGS are the standard
-   arguments that can be passed to a Unix program; for no arguments
+                   &key env (wait t) pty input
+                   if-input-does-not-exist output (if-output-exists :error)
+                   (error :output) (if-error-exists :error) status-hook)
+  "RUN-PROGRAM creates a new process and runs the unix progam in the
+   file specified by the simple-string program.  Args are the standard
+   arguments that can be passed to a Unix program, for no arguments
    use NIL (which means just the name of the program is passed as arg 0).
 
    RUN-PROGRAM will either return NIL or a PROCESS structure.  See the CMU
@@ -391,7 +384,7 @@ work on SBCL. At the very least:
 
    The keyword arguments have the following meanings:
      :env -
-        An alist mapping keyword environment variables to SIMPLE-STRING
+        An A-LIST mapping keyword environment variables to simple-string
        values.
      :wait -
         If non-NIL (default), wait until the created process finishes.  If
@@ -434,65 +427,58 @@ work on SBCL. At the very least:
         This is a function the system calls whenever the status of the
         process changes.  The function takes the process as an argument."
 
-  ;; Make sure that the interrupt handler is installed.
-  (system:enable-interrupt unix:sigchld #'sigchld-handler)
-  ;; Make sure that all the args are okay.
+  ;; Make sure the interrupt handler is installed.
+  (sb-sys:enable-interrupt sb-unix:sigchld #'sigchld-handler)
+  ;; Make sure all the args are okay.
   (unless (every #'simple-string-p args)
-    ;; FIXME: should be some sort of TYPE-ERROR? or perhaps we should
-    ;; just be nice and call (COERCE FOO 'SIMPLE-STRING) on each of
-    ;; our arguments, since it's reasonable for the user to pass in
-    ;; (at least) non-SIMPLE STRING values.
-    (error "All args to program must be simple strings: ~S." args))
-  ;; Prepend the program to the argument list.
+    (error "All arguments to program must be simple strings: ~S" args))
+  ;; Pre-pend the program to the argument list.
   (push (namestring program) args)
   ;; Clear various specials used by GET-DESCRIPTOR-FOR to communicate
-  ;; cleanup info. Also, establish proc at this level so that we can
+  ;; cleanup info.  Also, establish proc at this level so we can
   ;; return it.
   (let (*close-on-error* *close-in-parent* *handlers-installed* proc)
     (unwind-protect
-       (let ((pfile (unix-namestring (merge-pathnames program "path:") t t))
-             (cookie (list 0)))
-         (unless pfile
-           (error "no such program: ~S" program))
-         (multiple-value-bind
-             (stdin input-stream)
-             (get-descriptor-for input cookie
-                                 :direction :input
-                                 :if-does-not-exist if-input-does-not-exist)
-           (multiple-value-bind
-               (stdout output-stream)
-               (get-descriptor-for output cookie
-                                   :direction :output
-                                   :if-exists if-output-exists)
-             (multiple-value-bind
-                 (stderr error-stream)
-                 (if (eq error :output)
-                     (values stdout output-stream)
-                     (get-descriptor-for error cookie
-                                         :direction :output
-                                         :if-exists if-error-exists))
-               (multiple-value-bind (pty-name pty-stream)
-                                    (open-pty pty cookie)
-                 ;; Make sure we are not notified about the child
-                 ;; death before we have installed the process struct
-                 ;; in *ACTIVE-PROCESSES*.
-                 (system:without-interrupts
+        (let ((pfile (unix-namestring (merge-pathnames program "path:") t t))
+              (cookie (list 0)))
+          (unless pfile
+            (error "no such program: ~S" program))
+          (multiple-value-bind
+                (stdin input-stream)
+              (get-descriptor-for input cookie :direction :input
+                                  :if-does-not-exist if-input-does-not-exist)
+            (multiple-value-bind
+                  (stdout output-stream)
+                (get-descriptor-for output cookie :direction :output
+                                    :if-exists if-output-exists)
+              (multiple-value-bind
+                    (stderr error-stream)
+                  (if (eq error :output)
+                      (values stdout output-stream)
+                      (get-descriptor-for error cookie :direction :output
+                                          :if-exists if-error-exists))
+                (multiple-value-bind (pty-name pty-stream)
+                    (open-pty pty cookie)
+                  ;; Make sure we are not notified about the child
+                  ;; death before we have installed the PROCESS
+                  ;; structure in *ACTIVE-PROCESSES*.
+                  (sb-sys:without-interrupts
                    (with-c-strvec (argv args)
                      (with-c-strvec
-                         (envp (mapcar (lambda (entry)
-                                         (concatenate
-                                          'string
-                                          (symbol-name (car entry))
-                                          "="
-                                          (cdr entry)))
+                         (envp (mapcar #'(lambda (entry)
+                                           (concatenate
+                                            'string
+                                            (symbol-name (car entry))
+                                            "="
+                                            (cdr entry)))
                                        env))
                        (let ((child-pid
                               (without-gcing
                                (spawn pfile argv envp pty-name
                                       stdin stdout stderr))))
                          (when (< child-pid 0)
-                           (error "could not fork child process: ~A"
-                                  (unix:get-unix-error-msg)))
+                           (error "could not fork child process: ~S"
+                                  (sb-unix:get-unix-error-msg)))
                          (setf proc (make-process :pid child-pid
                                                   :%status :running
                                                   :pty pty-stream
@@ -501,68 +487,79 @@ work on SBCL. At the very least:
                                                   :error error-stream
                                                   :status-hook status-hook
                                                   :cookie cookie))
-                            (push proc *active-processes*))))))))))
+                         (push proc *active-processes*))))))))))
       (dolist (fd *close-in-parent*)
-       (unix:unix-close fd))
+       (sb-unix:unix-close fd))
       (unless proc
        (dolist (fd *close-on-error*)
-         (unix:unix-close fd))
+         (sb-unix:unix-close fd))
        (dolist (handler *handlers-installed*)
-         (system:remove-fd-handler handler))))
+         (sb-sys:remove-fd-handler handler))))
     (when (and wait proc)
       (process-wait proc))
     proc))
 
-;;; Install a handler for any input that shows up on the file
-;;; descriptor. The handler reads the data and writes it to the stream.
+;;; COPY-DESCRIPTOR-TO-STREAM -- internal
+;;;
+;;;   Installs a handler for any input that shows up on the file descriptor.
+;;; The handler reads the data and writes it to the stream.
+;;; 
 (defun copy-descriptor-to-stream (descriptor stream cookie)
   (incf (car cookie))
   (let ((string (make-string 256))
        handler)
     (setf handler
-         (system:add-fd-handler descriptor :input
-           #'(lambda (fd)
-               (declare (ignore fd))
-               (loop
-                 (unless handler
-                   (return))
-                 (multiple-value-bind
-                     (result readable/errno)
-                     (unix:unix-select (1+ descriptor) (ash 1 descriptor)
-                                       0 0 0)
-                   (cond ((null result)
-                          (error "could not select on sub-process: ~A"
-                                 (unix:get-unix-error-msg readable/errno)))
-                         ((zerop result)
-                          (return))))
-                 (alien:with-alien ((buf (alien:array c-call:char 256)))
-                   (multiple-value-bind
-                       (count errno)
-                       (unix:unix-read descriptor (alien-sap buf) 256)
-                     (cond ((or (and (null count)
-                                     (eql errno unix:eio))
-                                (eql count 0))
-                            (system:remove-fd-handler handler)
-                            (setf handler nil)
-                            (decf (car cookie))
-                            (unix:unix-close descriptor)
-                            (return))
-                           ((null count)
-                            (system:remove-fd-handler handler)
-                            (setf handler nil)
-                            (decf (car cookie))
-                            (error "could not read input from sub-process: ~A"
-                                   (unix:get-unix-error-msg errno)))
-                           (t
-                            (kernel:copy-from-system-area
-                             (alien-sap buf) 0
-                             string (* vm:vector-data-offset vm:word-bits)
-                             (* count vm:byte-bits))
-                            (write-string string stream
-                                          :end count)))))))))))
+         (sb-sys:add-fd-handler
+          descriptor
+          :input #'(lambda (fd)
+                     (declare (ignore fd))
+                     (loop
+                         (unless handler
+                           (return))
+                         (multiple-value-bind
+                               (result readable/errno)
+                             (sb-unix:unix-select (1+ descriptor)
+                                                  (ash 1 descriptor)
+                                                  0 0 0)
+                           (cond ((null result)
+                                  (error "could not select on sub-process: ~S"
+                                         (sb-unix:get-unix-error-msg
+                                          readable/errno)))
+                                 ((zerop result)
+                                  (return))))
+                       (sb-alien:with-alien ((buf (sb-alien:array
+                                                   sb-c-call:char
+                                                   256)))
+                         (multiple-value-bind
+                               (count errno)
+                             (sb-unix:unix-read descriptor
+                                                (alien-sap buf)
+                                                256)
+                           (cond ((or (and (null count)
+                                           (eql errno sb-unix:eio))
+                                      (eql count 0))
+                                  (sb-sys:remove-fd-handler handler)
+                                  (setf handler nil)
+                                  (decf (car cookie))
+                                  (sb-unix:unix-close descriptor)
+                                  (return))
+                                 ((null count)
+                                  (sb-sys:remove-fd-handler handler)
+                                  (setf handler nil)
+                                  (decf (car cookie))
+                                  (error "could not read input from sub-process: ~S"
+                                         (sb-unix:get-unix-error-msg errno)))
+                                 (t
+                                  (sb-kernel:copy-from-system-area
+                                   (alien-sap buf) 0
+                                   string (* sb-vm:vector-data-offset
+                                             sb-vm:word-bits)
+                                   (* count sb-vm:byte-bits))
+                                  (write-string string stream
+                                                :end count)))))))))))
 
 ;;; Find a file descriptor to use for object given the direction.
-;;; Return the descriptor. If object is :STREAM, return the created
+;;; Returns the descriptor. If object is :STREAM, returns the created
 ;;; stream as the second value.
 (defun get-descriptor-for (object
                           cookie
@@ -575,88 +572,90 @@ work on SBCL. At the very least:
        ((eq object nil)
         ;; Use /dev/null.
         (multiple-value-bind
-            (fd errno)
-            (unix:unix-open "/dev/null"
-                            (case direction
-                              (:input unix:o_rdonly)
-                              (:output unix:o_wronly)
-                              (t unix:o_rdwr))
-                            #o666)
+              (fd errno)
+            (sb-unix:unix-open "/dev/null"
+                               (case direction
+                                 (:input sb-unix:o_rdonly)
+                                 (:output sb-unix:o_wronly)
+                                 (t sb-unix:o_rdwr))
+                               #o666)
           (unless fd
-            (error "could not open \"/dev/null\": ~A"
-                   (unix:get-unix-error-msg errno)))
+            (error "could not open \"/dev/null\": ~S"
+                   (sb-unix:get-unix-error-msg errno)))
           (push fd *close-in-parent*)
           (values fd nil)))
        ((eq object :stream)
         (multiple-value-bind
-            (read-fd write-fd)
-            (unix:unix-pipe)
+              (read-fd write-fd)
+            (sb-unix:unix-pipe)
           (unless read-fd
-            (error "could not create pipe: ~A"
-                   (unix:get-unix-error-msg write-fd)))
+            (error "could not create pipe: ~S"
+                   (sb-unix:get-unix-error-msg write-fd)))
           (case direction
             (:input
              (push read-fd *close-in-parent*)
              (push write-fd *close-on-error*)
-             (let ((stream (system:make-fd-stream write-fd :output t)))
+             (let ((stream (sb-sys:make-fd-stream write-fd :output t)))
                (values read-fd stream)))
             (:output
              (push read-fd *close-on-error*)
              (push write-fd *close-in-parent*)
-             (let ((stream (system:make-fd-stream read-fd :input t)))
+             (let ((stream (sb-sys:make-fd-stream read-fd :input t)))
                (values write-fd stream)))
             (t
-             (unix:unix-close read-fd)
-             (unix:unix-close write-fd)
-             (error "direction must be either :INPUT or :OUTPUT, not ~S"
+             (sb-unix:unix-close read-fd)
+             (sb-unix:unix-close write-fd)
+             (error "Direction must be either :INPUT or :OUTPUT, not ~S."
                     direction)))))
        ((or (pathnamep object) (stringp object))
         (with-open-stream (file (apply #'open object keys))
           (multiple-value-bind
-              (fd errno)
-              (unix:unix-dup (system:fd-stream-fd file))
+                (fd errno)
+              (sb-unix:unix-dup (sb-sys:fd-stream-fd file))
             (cond (fd
                    (push fd *close-in-parent*)
                    (values fd nil))
                   (t
-                   (error "could not duplicate file descriptor: ~A"
-                          (unix:get-unix-error-msg errno)))))))
-       ((system:fd-stream-p object)
-        (values (system:fd-stream-fd object) nil))
+                   (error "could not duplicate file descriptor: ~S"
+                          (sb-unix:get-unix-error-msg errno)))))))
+       ((sb-sys:fd-stream-p object)
+        (values (sb-sys:fd-stream-fd object) nil))
        ((streamp object)
         (ecase direction
           (:input
+           ;; FIXME: We could use a better way of setting up
+           ;; temporary files, both here and in LOAD-FOREIGN.
            (dotimes (count
-                     256
+                      256
                      (error "could not open a temporary file in /tmp"))
              (let* ((name (format nil "/tmp/.run-program-~D" count))
-                    (fd (unix:unix-open name
-                                        (logior unix:o_rdwr
-                                                unix:o_creat
-                                                unix:o_excl)
-                                        #o666)))
-               (unix:unix-unlink name)
+                    (fd (sb-unix:unix-open name
+                                           (logior sb-unix:o_rdwr
+                                                   sb-unix:o_creat
+                                                   sb-unix:o_excl)
+                                           #o666)))
+               (sb-unix:unix-unlink name)
                (when fd
                  (let ((newline (string #\Newline)))
                    (loop
-                     (multiple-value-bind
-                         (line no-cr)
-                         (read-line object nil nil)
-                       (unless line
-                         (return))
-                       (unix:unix-write fd line 0 (length line))
-                       (if no-cr
-                         (return)
-                         (unix:unix-write fd newline 0 1)))))
-                 (unix:unix-lseek fd 0 unix:l_set)
+                       (multiple-value-bind
+                             (line no-cr)
+                           (read-line object nil nil)
+                         (unless line
+                           (return))
+                         (sb-unix:unix-write fd line 0 (length line))
+                         (if no-cr
+                             (return)
+                             (sb-unix:unix-write fd newline 0 1)))))
+                 (sb-unix:unix-lseek fd 0 sb-unix:l_set)
                  (push fd *close-in-parent*)
                  (return (values fd nil))))))
           (:output
            (multiple-value-bind (read-fd write-fd)
-                                (unix:unix-pipe)
+               (sb-unix:unix-pipe)
              (unless read-fd
-               (error "could not create pipe: ~A"
-                      (unix:get-unix-error-msg write-fd)))
+               (error "could not create pipe: ~S"
+                      (sb-unix:get-unix-error-msg write-fd)))
              (copy-descriptor-to-stream read-fd object cookie)
              (push read-fd *close-on-error*)
              (push write-fd *close-in-parent*)