XEmacs 21.2-b3
[chise/xemacs-chise.git.1] / lisp / package-get.el
index ba0f0ed..023fe91 100644 (file)
 ;;; Code:
 
 (require 'package-admin)
-(require 'package-get-base)
+;; (require 'package-get-base)
 
+(defgroup package-tools nil
+  "Tools to manipulate packages."
+  :group 'emacs)
+
+(defgroup package-get nil
+  "Automatic Package Fetcher and Installer."
+  :prefix "package-get"
+  :group 'package-tools)
+  
 (defvar package-get-base nil
   "List of packages that are installed at this site.
 For each element in the alist,  car is the package name and the cdr is
@@ -145,80 +154,347 @@ recent to least recent -- in other words, the version names don't have to
 be lexically ordered.  It is debatable if it makes sense to have more than
 one version of a package available.")
 
-(defvar package-get-dir (temp-directory)
-  "*Where to store temporary files for staging.")
+(defcustom package-get-dir (temp-directory)
+  "*Where to store temporary files for staging."
+  :tag "Temporary directory"
+  :type 'directory
+  :group 'package-get)
 
-(defvar package-get-remote
-  '(
-    ("ftp.xemacs.org" "/pub/xemacs/beta/xemacs-21.0/packages/binary-packages")
-    ("ftp.xemacs.org" "/pub/xemacs/beta/xemacs-21.0/packages/single-file-packages")
-    ("ftp.xemacs.org" "/pub/xemacs/package"))
+;; JV Any Custom expert know to get "Host" and "Dir" for the remote option
+(defcustom package-get-remote
+  '(("ftp.xemacs.org" "/pub/xemacs/packages"))
   "*List of remote sites to contact for downloading packages.
 List format is '(site-name directory-on-site).  Each site is tried in
-order until the package is found.")
-
-(defvar package-get-remove-copy nil
+order until the package is found.  As a special case, `site-name' can be
+`nil', in which case `directory-on-site' is treated as a local directory."
+  :tag "Package repository"
+  :type '(repeat (choice (list :tag "Local" (const :tag "Local" nil) directory )
+                        (list :tag "Remote" string string) ))
+  :group 'package-get)
+
+(defcustom package-get-remove-copy nil
   "*After copying and installing a package, if this is T, then remove the
-copy.  Otherwise, keep it around.")
+copy.  Otherwise, keep it around."
+  :type 'boolean
+  :group 'package-get)
+
+(defcustom package-get-base-filename
+  "/ftp.xemacs.org:/pub/xemacs/packages/package-index.LATEST"
+  "*Name of the default package database file, usually on ftp.xemacs.org."
+  :type 'file
+  :group 'package-get)
+
+;;;###autoload
+(defun package-get-require-base ()
+  "Require that a package-get database has been loaded."
+  (when (or (not (boundp 'package-get-base))
+            (not package-get-base))
+    (package-get-update-base))
+  (when (or (not (boundp 'package-get-base))
+            (not package-get-base))
+    (error "Package-get database not loaded")))
+
+(defconst package-get-pgp-signed-begin-line "^-----BEGIN PGP SIGNED MESSAGE-----"
+  "Text for start of PGP signed messages.")
+(defconst package-get-pgp-signature-begin-line "^-----BEGIN PGP SIGNATURE-----"
+  "Text for beginning of PGP signature.")
+(defconst package-get-pgp-signature-end-line "^-----END PGP SIGNATURE-----"
+  "Text for end of PGP signature.")
+
+;;;###autoload
+(defun package-get-update-base-entry (entry)
+  "Update an entry in `package-get-base'."
+  (let ((existing (assoc (car entry) package-get-base)))
+    (if existing
+        (setcdr existing (cdr entry))
+      (setq package-get-base (cons entry package-get-base)))))
+
+;;;###autoload
+(defun package-get-update-base (&optional db-file)
+  "Update the package-get database file with entries from DB-FILE."
+  (interactive (list
+                (read-file-name "Load package-get database: "
+                                (file-name-directory package-get-base-filename)
+                                package-get-base-filename
+                                t
+                                (file-name-nondirectory package-get-base-filename))))
+  (setq db-file (expand-file-name (or db-file package-get-base-filename)))
+  (if (not (file-exists-p db-file))
+      (error "Package-get database file `%s' does not exist" db-file))
+  (if (not (file-readable-p db-file))
+      (error "Package-get database file `%s' not readable" db-file))
+  (let ((buf (get-buffer-create "*package database*")))
+    (unwind-protect
+        (save-excursion
+          (set-buffer buf)
+          (erase-buffer buf)
+          (insert-file-contents-internal db-file)
+          (package-get-update-base-from-buffer buf))
+      (kill-buffer buf))))
+
+;;;###autoload
+(defun package-get-update-base-from-buffer (&optional buf)
+  "Update the package-get database with entries from BUFFER.
+BUFFER defaults to the current buffer.  This command can be
+used interactively, for example from a mail or news buffer."
+  (interactive)
+  (setq buf (or buf (current-buffer)))
+  (let (content-beg content-end beg end)
+    (save-excursion
+      (set-buffer buf)
+      (goto-char (point-min))
+      (setq content-beg (point))
+      (setq content-end (save-excursion (goto-char (point-max)) (point)))
+      (when (re-search-forward package-get-pgp-signed-begin-line nil t)
+        (setq beg (match-beginning 0))
+        (setq content-beg (match-end 0)))
+      (when (re-search-forward package-get-pgp-signature-begin-line nil t)
+        (setq content-end (match-beginning 0)))
+      (when (re-search-forward package-get-pgp-signature-end-line nil t)
+        (setq end (point)))
+      (if (not (and content-beg content-end beg end))
+          (or (yes-or-no-p "Package-get entries not PGP signed, continue? ")
+              (error "Package-get database not updated")))
+      (if (and content-beg content-end beg end)
+          (if (not (condition-case nil
+                       (or (fboundp 'mc-pgp-verify-region)
+                           (load-library "mc-pgp")
+                           (fboundp 'mc-pgp-verify-region))
+                     (error nil)))
+              (or (yes-or-no-p
+                   "No mailcrypt; can't verify package-get DB signature, continue? ")
+                  (error "Package-get database not updated"))))
+      (if (and beg end
+               (fboundp 'mc-pgp-verify-region)
+               (or (not
+                    (condition-case err
+                        (mc-pgp-verify-region beg end)
+                      (file-error
+                       (and (string-match "No such file" (nth 2 err))
+                            (yes-or-no-p
+                             "Can't find PGP, continue without package-get DB verification? ")))
+                      (t nil)))))
+          (error "Package-get PGP signature failed to verify"))
+      (package-get-update-base-entries content-beg content-end)
+      (message "Updated package-get database"))))
+
+(defun package-get-update-base-entries (beg end)
+  "Update the package-get database with the entries found between
+BEG and END in the current buffer."
+  (save-excursion
+    (goto-char beg)
+    (if (not (re-search-forward "^(package-get-update-base-entry" nil t))
+        (error "Buffer does not contain package-get database entries"))
+    (beginning-of-line)
+    (let ((count 0))
+      (while (and (< (point) end)
+                  (re-search-forward "^(package-get-update-base-entry" nil t))
+        (beginning-of-line)
+        (let ((entry (read (current-buffer))))
+          (if (or (not (consp entry))
+                  (not (eq (car entry) 'package-get-update-base-entry)))
+              (error "Invalid package-get database entry found"))
+          (package-get-update-base-entry
+           (car (cdr (car (cdr entry)))))
+          (setq count (1+ count))))
+      (message "Got %d package-get database entries" count))))
+
+(defun package-get-interactive-package-query (get-version package-symbol)
+  "Perform interactive querying for package and optional version.
+Query for a version if GET-VERSION is non-nil.  Return package name as
+a symbol instead of a string if PACKAGE-SYMBOL is non-nil.
+The return value is suitable for direct passing to `interactive'."
+  (package-get-require-base)
+  (let ( (table (mapcar '(lambda (item)
+                          (let ( (name (symbol-name (car item))) )
+                            (cons name name)
+                            ))
+                       package-get-base)) 
+        package package-symbol default-version version)
+    (save-window-excursion
+      (setq package (completing-read "Package: " table nil t))
+      (setq package-symbol (intern package))
+      (if get-version
+         (progn
+           (setq default-version 
+                 (package-get-info-prop 
+                  (package-get-info-version
+                   (package-get-info-find-package package-get-base
+                                                  package-symbol) nil)
+                  'version))
+           (while (string=
+                   (setq version (read-string "Version: " default-version))
+                   "")
+             )
+           (if package-symbol
+               (list package-symbol version)
+             (list package version))
+           )
+       (if package-symbol
+           (list package-symbol)
+         (list package)))
+      )))
+
+;;;###autoload
+(defun package-get-delete-package (package &optional pkg-topdir)
+  "Delete an installation of PACKAGE below directory PKG-TOPDIR.
+PACKAGE is a symbol, not a string.
+This is just an interactive wrapper for `package-admin-delete-binary-package'."
+  (interactive (package-get-interactive-package-query nil t))
+  (package-admin-delete-binary-package package pkg-topdir))
 
 ;;;###autoload
 (defun package-get-update-all ()
   "Fetch and install the latest versions of all currently installed packages."
   (interactive)
+  (package-get-require-base)
   ;; Load a fresh copy
-  (mapcar (lambda (pkg)
-           (package-get-all
-            (car pkg) nil))
-          packages-package-list))
+  (catch 'exit
+    (mapcar (lambda (pkg)
+             (if (not (package-get (car pkg) nil 'never))
+                 (throw 'exit nil)             ;; Bail out if error detected
+                 ))
+           packages-package-list)))
 
 ;;;###autoload
-(defun package-get-all (package version &optional fetched-packages)
+(defun package-get-all (package version &optional fetched-packages install-dir)
   "Fetch PACKAGE with VERSION and all other required packages.
 Uses `package-get-base' to determine just what is required and what
 package provides that functionality.  If VERSION is nil, retrieves
 latest version.  Optional argument FETCHED-PACKAGES is used to keep
-track of packages already fetched."
-  (interactive "sPackage: \nsVersion: ")
+track of packages already fetched.  Optional argument INSTALL-DIR,
+if non-nil, specifies the package directory where fetched packages
+should be installed.
+
+Returns nil upon error."
+  (interactive (package-get-interactive-package-query t nil))
   (let* ((the-package (package-get-info-find-package package-get-base
                                                     package))
         (this-package (package-get-info-version
                        the-package version))
         (this-requires (package-get-info-prop this-package 'requires))
         )
-    (setq version (package-get-info-prop this-package 'version))
-    (unless (package-get-installedp package version)
-      (package-get package version))
-    (setq fetched-packages
-         (append (list package)
-                 (package-get-info-prop this-package 'provides)
-                 fetched-packages))
-    ;; grab everything that this package requires plus recursively
-    ;; grab everything that the requires require.  Keep track
-    ;; in `fetched-packages' the list of things provided -- this
-    ;; keeps us from going into a loop
-    (while this-requires
-      (if (not (member (car this-requires) fetched-packages))
-         (let* ((reqd-package (package-get-package-provider
-                               (car this-requires)))
-                (reqd-version (cadr reqd-package))
-                (reqd-name (car reqd-package)))
-           (if (null reqd-name)
-               (error "Unable to find a provider for %s" (car this-requires)))
-           (setq fetched-packages
-                 (package-get-all reqd-name reqd-version fetched-packages)))
-       )
-      (setq this-requires (cdr this-requires)))
+    (catch 'exit
+      (setq version (package-get-info-prop this-package 'version))
+      (unless (package-get-installedp package version)
+       (if (not (package-get package version nil install-dir))
+           (progn
+             (setq fetched-packages nil)
+             (throw 'exit nil))))
+      (setq fetched-packages
+           (append (list package)
+                   (package-get-info-prop this-package 'provides)
+                   fetched-packages))
+      ;; grab everything that this package requires plus recursively
+      ;; grab everything that the requires require.  Keep track
+      ;; in `fetched-packages' the list of things provided -- this
+      ;; keeps us from going into a loop
+      (while this-requires
+       (if (not (member (car this-requires) fetched-packages))
+           (let* ((reqd-package (package-get-package-provider
+                                 (car this-requires)))
+                  (reqd-version (cadr reqd-package))
+                  (reqd-name (car reqd-package)))
+             (if (null reqd-name)
+                 (error "Unable to find a provider for %s"
+                        (car this-requires)))
+             (if (not (setq fetched-packages
+                            (package-get-all reqd-name reqd-version
+                                             fetched-packages
+                                              install-dir)))
+                 (throw 'exit nil)))
+         )
+       (setq this-requires (cdr this-requires)))
+      )
     fetched-packages
     ))
 
 ;;;###autoload
-(defun package-get (package &optional version conflict)
+(defun package-get-dependencies (packages)
+  "Compute dependencies for PACKAGES.
+Uses `package-get-base' to determine just what is required and what
+package provides that functionality.  Returns the list of packages
+required by PACKAGES."
+  (package-get-require-base)
+  (let ((orig-packages packages)
+        dependencies provided)
+    (while packages
+      (let* ((package (car packages))
+             (the-package (package-get-info-find-package
+                           package-get-base package))
+             (this-package (package-get-info-version
+                            the-package nil))
+             (this-requires (package-get-info-prop this-package 'requires))
+             (new-depends   (set-difference
+                             (mapcar
+                              #'(lambda (reqd)
+                                  (let* ((reqd-package (package-get-package-provider reqd))
+                                         (reqd-version (cadr reqd-package))
+                                         (reqd-name    (car reqd-package)))
+                                    (if (null reqd-name)
+                                        (error "Unable to find a provider for %s" reqd))
+                                    reqd-name))
+                              this-requires)
+                             dependencies))
+             (this-provides (package-get-info-prop this-package 'provides)))
+        (setq dependencies
+              (union dependencies new-depends))
+        (setq provided
+              (union provided (union (list package) this-provides)))
+        (setq packages
+              (union new-depends (cdr packages)))))
+    (set-difference dependencies orig-packages)))
+
+(defun package-get-load-package-file (lispdir file)
+  (let (pathname)
+    (setq pathname (expand-file-name file lispdir))
+    (condition-case err
+       (progn
+         (load pathname t)
+         t)
+      (t
+       (message "Error loading package file \"%s\" %s!" pathname err)
+       nil))
+    ))
+
+(defun package-get-init-package (lispdir)
+  "Initialize the package.
+This really assumes that the package has never been loaded.  Updating
+a newer package can cause problems, due to old, obsolete functions in
+the old package.
+
+Return `t' upon complete success, `nil' if any errors occurred."
+  (progn
+    (if (and lispdir
+            (file-accessible-directory-p lispdir))
+       (progn
+         ;; Add lispdir to load-path if it doesn't already exist.
+         ;; NOTE: this does not take symlinks, etc., into account.
+         (if (let ( (dirs load-path) )
+               (catch 'done
+                 (while dirs
+                   (if (string-equal (car dirs) lispdir)
+                       (throw 'done nil))
+                   (setq dirs (cdr dirs))
+                   )
+                 t))
+             (setq load-path (cons lispdir load-path)))
+         (if (not (package-get-load-package-file lispdir "auto-autoloads"))
+             (package-get-load-package-file lispdir "_pkg"))
+         t)
+      nil)
+    ))
+
+;;;###autoload
+(defun package-get (package &optional version conflict install-dir)
   "Fetch PACKAGE from remote site.
 Optional arguments VERSION indicates which version to retrieve, nil
 means most recent version.  CONFLICT indicates what happens if the
 package is already installed.  Valid values for CONFLICT are:
 'always        always retrieve the package even if it is already installed
 'never do not retrieve the package if it is installed.
+INSTALL-DIR, if non-nil, specifies the package directory where
+fetched packages should be installed.
 
 The value of `package-get-base' is used to determine what files should 
 be retrieved.  The value of `package-get-remote' is used to determine
@@ -227,59 +503,160 @@ order so one is better off listing easily reached sites first.
 
 Once the package is retrieved, its md5 checksum is computed.  If that
 sum does not match that stored in `package-get-base' for this version
-of the package, an error is signalled."
-  (interactive "xPackage List: ")
+of the package, an error is signalled.
+
+Returns `t' upon success, the symbol `error' if the package was
+successfully installed but errors occurred during initialization, or
+`nil' upon error."
+  (interactive (package-get-interactive-package-query nil t))
   (let* ((this-package
          (package-get-info-version
           (package-get-info-find-package package-get-base
                                          package) version))
+        (this-requires (package-get-info-prop this-package 'requires))
         (found nil)
         (search-dirs package-get-remote)
-        (filename (package-get-info-prop this-package 'filename)))
+        (base-filename (package-get-info-prop this-package 'filename))
+        (package-status t)
+        filenames full-package-filename)
     (if (null this-package)
        (error "Couldn't find package %s with version %s"
               package version))
-    (if (null filename)
+    (if (null base-filename)
        (error "No filename associated with package %s, version %s"
               package version))
+    (setq install-dir
+         (package-admin-get-install-dir package install-dir
+               (or (eq package 'mule-base) (memq 'mule-base this-requires))))
+
+    ;; Contrive a list of possible package filenames.
+    ;; Ugly.  Is there a better way to do this?
+    (setq filenames (cons base-filename nil))
+    (if (string-match "^\\(..*\\)\.tar\.gz$" base-filename)
+       (setq filenames (append filenames
+                               (list (concat (match-string 1 base-filename)
+                                             ".tgz")))))
+
     (setq version (package-get-info-prop this-package 'version))
     (unless (and (eq conflict 'never)
                 (package-get-installedp package version))
-      ;; Find the package from search list in package-get-remote
+      ;; Find the package from the search list in package-get-remote
       ;; and copy it into the staging directory.  Then validate
       ;; the checksum.  Finally, install the package.
-      (while (and search-dirs
-                 (not (file-exists-p (package-get-staging-dir filename))))
-       (if (file-exists-p (package-get-remote-filename
-                           (car search-dirs) filename))
-           (copy-file (package-get-remote-filename (car search-dirs) filename)
-                      (package-get-staging-dir filename))
-         (setq search-dirs (cdr search-dirs))
+      (catch 'done
+       (let (search-filenames current-dir-entry host dir current-filename
+                              dest-filename)
+         ;; In each search directory ...
+         (while search-dirs
+           (setq current-dir-entry (car search-dirs)
+                 host (car current-dir-entry)
+                 dir (car (cdr current-dir-entry))
+                 search-filenames filenames
+                 )
+
+           ;; Look for one of the possible package filenames ...
+           (while search-filenames
+             (setq current-filename (car search-filenames)
+                   dest-filename (package-get-staging-dir current-filename))
+             (cond
+              ;; No host means look on the current system.
+              ( (null host)
+                (setq full-package-filename
+                      (substitute-in-file-name
+                       (expand-file-name current-filename
+                                         (file-name-as-directory dir))))
+                )
+
+              ;; If it's already on the disk locally, and the size is
+              ;; greater than zero ...
+              ( (and (file-exists-p dest-filename)
+                     (let (attrs)
+                       ;; file-attributes could return -1 for LARGE files,
+                       ;; but, hopefully, packages won't be that large.
+                       (and (setq attrs (file-attributes dest-filename))
+                            (> (nth 7 attrs) 0))))
+                (setq full-package-filename dest-filename)
+                )
+
+              ;; If the file exists on the remote system ...
+              ( (file-exists-p (package-get-remote-filename
+                                current-dir-entry current-filename))
+                ;; Get it
+                (setq full-package-filename dest-filename)
+                (message "Retrieving package `%s' ..." 
+                         current-filename)
+                (sit-for 0)
+                (copy-file (package-get-remote-filename current-dir-entry
+                                                        current-filename)
+                           full-package-filename t)
+                )
+              )
+
+             ;; If we found it, we're done.
+             (if (and full-package-filename
+                      (file-exists-p full-package-filename))
+                 (throw 'done nil))
+             ;; Didn't find it.  Try the next possible filename.
+             (setq search-filenames (cdr search-filenames))
+             )
+           ;; Try looking in the next possible directory ...
+           (setq search-dirs (cdr search-dirs))
+           )
          ))
-      (if (not (file-exists-p (package-get-staging-dir filename)))
-         (error "Unable to find file %s" filename))
+
+      (if (or (not full-package-filename)
+             (not (file-exists-p full-package-filename)))
+         (error "Unable to find file %s" base-filename))
       ;; Validate the md5 checksum
       ;; Doing it with XEmacs removes the need for an external md5 program
+      (message "Validating checksum for `%s'..." package) (sit-for 0)
       (with-temp-buffer
        ;; What ever happened to i-f-c-literally
        (let (file-name-handler-alist)
-         (insert-file-contents-internal (package-get-staging-dir filename)))
+         (insert-file-contents-internal full-package-filename))
        (if (not (string= (md5 (current-buffer))
                          (package-get-info-prop this-package
                                                 'md5sum)))
-           (error "Package %s does not match md5 checksum" filename)))
-      (message "Retrieved package %s" filename) (sit-for 0)
+           (error "Package %s does not match md5 checksum" base-filename)))
+
+      (package-admin-delete-binary-package package install-dir)
+
+      (message "Installing package `%s' ..." package) (sit-for 0)
       (let ((status
-            (package-admin-add-binary-package
-             (package-get-staging-dir filename))))
-       (when (not (= status 0))
-         (message "Package failed.")
-         (switch-to-buffer package-admin-temp-buffer)))
-      (sit-for 0)
-      (message "Added package") (sit-for 0)
+            (package-admin-add-binary-package full-package-filename
+                                              install-dir)))
+       (if (= status 0)
+           (progn
+             ;; clear messages so that only messages from
+             ;; package-get-init-package are seen, below.
+             (clear-message)
+             (if (package-get-init-package (package-admin-get-lispdir
+                                            install-dir package))
+                 (progn
+                   (message "Added package `%s'" package)
+                   (sit-for 0)
+                   )
+               (progn
+                 ;; display message only if there isn't already one.
+                 (if (not (current-message))
+                     (progn
+                       (message "Added package `%s' (errors occurred)"
+                                package)
+                       (sit-for 0)
+                       ))
+                 (if package-status
+                     (setq package-status 'errors))
+                 ))
+             )
+         (message "Installation of package %s failed." base-filename)
+         (sit-for 0)
+         (switch-to-buffer package-admin-temp-buffer)
+         (setq package-status nil)
+         ))
       (setq found t))
     (if (and found package-get-remove-copy)
-       (delete-file (package-get-staging-dir filename)))
+       (delete-file full-package-filename))
+    package-status
     ))
 
 (defun package-get-info-find-package (which name)
@@ -304,7 +681,7 @@ from a version returned by `package-get-info-version'."
   `package-get-info-find-package'.  If VERSION is nil, then return the 
   first (aka most recent) version.  Use `package-get-info-find-prop'
   to retrieve a particular property from the value returned by this."
-  (interactive "xPackage Info: \nsVersion: ")
+  (interactive (package-get-interactive-package-query t t))
   (while (and version package (not (string= (plist-get (car package) 'version) version)))
     (setq package (cdr package)))
   (if package (car package)))
@@ -345,10 +722,11 @@ Creates `package-get-dir'  it it doesn't exist."
   (interactive "FPackage filename: ")
   (if (not (file-exists-p package-get-dir))
       (make-directory package-get-dir))
-  (concat 
-   (file-name-as-directory package-get-dir)
-   (file-name-nondirectory (or (nth 2 (efs-ftp-path filename)) filename))))
-       
+  (expand-file-name
+   (file-name-nondirectory (or (and (fboundp 'efs-ftp-path)
+                                   (nth 2 (efs-ftp-path filename)))
+                              filename))
+   (file-name-as-directory package-get-dir)))
 
 (defun package-get-remote-filename (search filename)
   "Return FILENAME as a remote filename.
@@ -386,6 +764,7 @@ some built in variables.  For now, use packages-package-list."
   consp, then it must match a corresponding (provide (SYM VERSION)) from 
   the package."
   (interactive "SSymbol: ")
+  (package-get-require-base)
   (let ((packages package-get-base)
        (done nil)
        (found nil))
@@ -417,6 +796,7 @@ some built in variables.  For now, use packages-package-list."
 (defun package-get-custom ()
   "Fetch and install the latest versions of all customized packages."
   (interactive)
+  (package-get-require-base)
   ;; Load a fresh copy
   (load "package-get-custom.el")
   (mapcar (lambda (pkg)
@@ -458,10 +838,12 @@ Entries in the customization file are retrieved from package-get-base.el."
   (let ((custom-buffer (find-file-noselect 
                        (or (package-get-file-installed-p 
                             "package-get-custom.el")
-                           (concat (file-name-directory 
-                                    (package-get-file-installed-p 
-                                     "package-get-base.el"))
-                                   "package-get-custom.el"))))
+                           (expand-file-name
+                            "package-get-custom.el"
+                            (file-name-directory 
+                             (package-get-file-installed-p 
+                              "package-get-base.el"))
+                            ))))
        (pkg-groups nil))
 
     ;; clear existing stuff
@@ -493,6 +875,8 @@ Entries in the customization file are retrieved from package-get-base.el."
 (provide 'package-get)
 
 ;; potentially update the custom dependencies every time we load this
+(when nil ;; #### disable for now... -gk
+(unless noninteractive
 (let ((custom-file (package-get-file-installed-p "package-get-custom.el"))
       (package-file (package-get-file-installed-p "package-get-base.el")))
   ;; update custom file if it doesn't exist
@@ -506,6 +890,7 @@ Entries in the customization file are retrieved from package-get-base.el."
        (set-buffer (package-get-create-custom))
        (save-buffer)
        (message "generating package customizations...done")))
-  (load "package-get-custom.el"))
+  (load "package-get-custom.el")))
+)
 
 ;;; package-get.el ends here