[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] elpa-admin 81a6a01 088/357: Fix up scripts to build archive from
From: |
Stefan Monnier |
Subject: |
[elpa] elpa-admin 81a6a01 088/357: Fix up scripts to build archive from Git |
Date: |
Thu, 10 Dec 2020 18:06:21 -0500 (EST) |
branch: elpa-admin
commit 81a6a01d5d4e56204024134f8d295efab5a7385c
Author: Stefan Monnier <monnier@iro.umontreal.ca>
Commit: Stefan Monnier <monnier@iro.umontreal.ca>
Fix up scripts to build archive from Git
---
GNUmakefile | 37 ++++----
admin/archive-contents.el | 230 ++++++++++++++++++++++------------------------
2 files changed, 131 insertions(+), 136 deletions(-)
diff --git a/GNUmakefile b/GNUmakefile
index 3199fcc..d1baf58 100644
--- a/GNUmakefile
+++ b/GNUmakefile
@@ -1,6 +1,6 @@
# Makefile for GNU Emacs Lisp Package Archive.
-EMACS=emacs
+EMACS=emacs --batch
ARCHIVE_TMP=archive-tmp
SITE_DIR=site
@@ -23,14 +23,16 @@ process-archive:
# FIXME, we could probably speed this up significantly with
# rules like "%.tar: ../%/ChangeLog" so we only rebuild the packages
# that have indeed changed.
- cd $(ARCHIVE_TMP)/packages; $(EMACS) -batch -l
$(CURDIR)/admin/archive-contents.el -f batch-make-archive
+ cd $(ARCHIVE_TMP)/packages; \
+ $(EMACS) -l $(CURDIR)/admin/archive-contents.el \
+ -f batch-make-archive
@cd $(ARCHIVE_TMP)/packages; \
- for pt in *; do \
- if [ -d $$pt ]; then \
- echo "Creating tarball $${pt}.tar" && \
- tar -cf $${pt}.tar $$pt --remove-files; \
- fi; \
- done
+ for pt in *; do \
+ if [ -d $$pt ]; then \
+ echo "Creating tarball $${pt}.tar" && \
+ tar -cf $${pt}.tar $$pt --remove-files; \
+ fi; \
+ done
mkdir -p archive/packages
mv archive/packages archive/packages-old
mv $(ARCHIVE_TMP)/packages archive/packages
@@ -80,12 +82,11 @@ $(foreach al, $(autoloads), $(eval $(call RULE-srcdeps,
$(al))))
%-autoloads.el:
@echo 'EMACS -f package-generate-autoloads $@'
@cd $(dir $@); \
- $(EMACS) --batch \
- -l $(CURDIR)/admin/archive-contents.el \
- --eval "(archive--refresh-pkg-file)" \
- --eval "(require 'package)" \
- --eval "(package-generate-autoloads '$$(basename $$(pwd)) \
- \"$$(pwd)\")"
+ $(EMACS) -l $(CURDIR)/admin/archive-contents.el \
+ --eval "(archive--refresh-pkg-file)" \
+ --eval "(require 'package)" \
+ --eval "(package-generate-autoloads '$$(basename $$(pwd)) \
+ \"$$(pwd)\")"
# Put into elcs the set of elc files we need to keep up-to-date.
# I.e. one for each .el file except for the -pkg.el, the -autoloads.el, and
@@ -102,7 +103,7 @@ elcs := $(call SET-diff, $(naive_elcs), $(patsubst %.el,
%.elc, $(nbc_els)))
# '(dolist (al (quote ($(patsubst %, "%", $(autoloads))))) (load
(expand-file-name al) nil t))'
%.elc: %.el
@echo 'EMACS -f batch-byte-compile $<'
- @$(EMACS) --batch \
+ @$(EMACS) \
--eval "(setq package-directory-list '(\"$(abspath packages)\"))" \
--eval '(package-initialize)' \
-L $(dir $@) -f batch-byte-compile $<
@@ -124,7 +125,7 @@ $(extra_elcs):; rm $@
# #$(foreach al, $(single_pkgs), $(eval $(call RULE-srcdeps, $(al))))
# %-pkg.el: %.el
# @echo 'EMACS -f package-generate-description-file $@'
-# @$(EMACS) --batch \
+# @$(EMACS) \
# --eval '(require (quote package))' \
# --eval '(setq b (find-file-noselect "$<"))' \
# --eval '(setq d (with-current-buffer b (package-buffer-info)))' \
@@ -138,3 +139,7 @@ all-in-place: $(extra_elcs) $(autoloads) # $(single_pkgs)
############### Rules to prepare the externals ################################
+.PHONY:
+externals:
+ $(EMACS) -l admin/archive-contents.el \
+ -f archive-add/remove/update-externals
diff --git a/admin/archive-contents.el b/admin/archive-contents.el
index 5a5462a..8ef73aa 100644
--- a/admin/archive-contents.el
+++ b/admin/archive-contents.el
@@ -86,40 +86,64 @@ Delete backup files also."
(pp (nreverse packages) (current-buffer))
(write-region nil nil "archive-contents"))))
-(defun batch-prepare-packages ()
+(defconst archive--revno-re "[0-9a-f]+")
+
+(defun archive-prepare-packages (srcdir)
"Prepare the `packages' directory inside the Git checkout.
Expects to be called from within the `packages' directory.
\"Prepare\" here is for subsequent construction of the packages and archive,
so it is meant to refresh any generated files we may need.
Currently only refreshes the ChangeLog files."
+ (setq srcdir (file-name-as-directory (expand-file-name srcdir)))
(let* ((wit ".changelog-witness")
- (prevno (or (with-temp-buffer
- (ignore-errors (insert-file-contents wit))
- (when (looking-at "[1-9][0-9]*\\'")
- (string-to-number (match-string 0))))
- 1))
+ (prevno (with-temp-buffer
+ (ignore-errors (insert-file-contents wit))
+ (if (looking-at (concat archive--revno-re "$"))
+ (match-string 0)
+ (error "Can't find previous revision name"))))
(new-revno
(or (with-temp-buffer
- (call-process "bzr" nil '(t) nil "revno")
- (goto-char (point-min))
- (when (looking-at "[1-9][0-9]*$")
- (string-to-number (match-string 0))))
- (error "bzr revno did not return a number as expected")))
+ (let ((default-directory srcdir))
+ (call-process "git" nil '(t) nil "rev-parse" "HEAD")
+ (goto-char (point-min))
+ (when (looking-at (concat archive--revno-re "$"))
+ (match-string 0))))
+ (error "Couldn't find the current revision's name")))
(pkgs '()))
- (unless (= prevno new-revno)
+ (unless (equal prevno new-revno)
(with-temp-buffer
- (unless (zerop (call-process "bzr" nil '(t) nil "log" "-v"
- (format "-r%d.." (1+ prevno))))
- (error "Error signaled by bzr log -v -r%d.." (1+ prevno)))
+ (let ((default-directory srcdir))
+ (unless (zerop (call-process "git" nil '(t) nil "diff"
+ "--dirstat=cumulative,0"
+ prevno))
+ (error "Error signaled by git diff --dirstat %d" prevno)))
(goto-char (point-min))
- (while (re-search-forward "^ packages/\\([-[:alnum:]]+\\)/" nil t)
- (pushnew (match-string 1) pkgs :test #'equal))))
- (dolist (pkg pkgs)
- (condition-case v
- (if (file-directory-p pkg)
- (archive--make-changelog pkg))
- (error (message "Error: %S" v))))
- (write-region (number-to-string new-revno) nil wit nil 'quiet)))
+ (while (re-search-forward "^[ \t.0-9%]* packages/\\([-[:alnum:]]+\\)/$"
+ nil t)
+ (push (match-string 1) pkgs))))
+ (let ((default-directory (expand-file-name "packages/")))
+ (dolist (pkg pkgs)
+ (condition-case v
+ (if (file-directory-p pkg)
+ (archive--make-changelog pkg (expand-file-name "packages/"
+ srcdir)))
+ (error (message "Error: %S" v)))))
+ (write-region new-revno nil wit nil 'quiet)
+ ;; Also update the ChangeLog of external packages.
+ (let ((default-directory (expand-file-name "packages/")))
+ (dolist (dir (directory-files "."))
+ (and (not (member dir '("." "..")))
+ (file-directory-p dir)
+ (let ((index (expand-file-name
+ (concat "packages/" dir "/.git/index")
+ srcdir))
+ (cl (expand-file-name "ChangeLog" dir)))
+ (and (file-exists-p index)
+ (or (not (file-exists-p cl))
+ (file-newer-than-file-p index cl))))
+ (archive--make-changelog
+ dir (expand-file-name "packages/" srcdir)))))
+ ))
(defun archive--simple-package-p (dir pkg)
"Test whether DIR contains a simple package named PKG.
@@ -131,7 +155,7 @@ Otherwise, return nil."
(let* ((pkg-file (expand-file-name (concat pkg "-pkg.el") dir))
(mainfile (expand-file-name (concat pkg ".el") dir))
(files (directory-files dir nil archive-re-no-dot))
- version description req commentary)
+ version description req)
(dolist (file (prog1 files (setq files ())))
(unless (string-match "\\(?:\\.elc\\|~\\)\\'" file)
(push file files)))
@@ -157,33 +181,13 @@ Otherwise, return nil."
(if requires-str
(setq req (mapcar 'archive--convert-require
(car (read-from-string requires-str))))))
- (setq commentary (lm-commentary))
- (list version description req commentary))))
+ (list version description req))))
((not (file-exists-p pkg-file))
(error "Can find single file nor package desc file in %s" dir)))))
-(defun archive--process-simple-package (dir pkg vers desc req commentary)
+(defun archive--process-simple-package (dir pkg vers desc req)
"Deploy the contents of DIR into the archive as a simple package.
-Rename DIR/PKG.el to PKG-VERS.el, delete DIR, and write the
-package commentary to PKG-readme.txt. Return the descriptor."
- ;; Write the readme file.
- (with-temp-buffer
- (erase-buffer)
- (emacs-lisp-mode)
- (insert (or commentary
- (prog1 "No description"
- (message "Missing commentary in package %s" pkg))))
- (goto-char (point-min))
- (while (looking-at ";*[ \t]*\\(commentary[: \t]*\\)?\n")
- (delete-region (match-beginning 0)
- (match-end 0)))
- (uncomment-region (point-min) (point-max))
- (goto-char (point-max))
- (while (progn (forward-line -1)
- (looking-at "[ \t]*\n"))
- (delete-region (match-beginning 0)
- (match-end 0)))
- (write-region nil nil (concat pkg "-readme.txt")))
+Rename DIR/PKG.el to PKG-VERS.el, delete DIR, and return the descriptor."
;; Write DIR/foo.el to foo-VERS.el and delete DIR
(rename-file (expand-file-name (concat pkg ".el") dir)
(concat pkg "-" vers ".el"))
@@ -208,8 +212,8 @@ package commentary to PKG-readme.txt. Return the
descriptor."
(delete-directory dir t)
(cons (intern pkg) (vector (version-to-list vers) req desc 'single)))
-(defun archive--make-changelog (dir)
- "Export Bzr log info of DIR into a ChangeLog file."
+(defun archive--make-changelog (dir srcdir)
+ "Export Git log info of DIR into a ChangeLog file."
(message "Refreshing ChangeLog in %S" dir)
(let ((default-directory (file-name-as-directory (expand-file-name dir))))
(with-temp-buffer
@@ -219,27 +223,29 @@ package commentary to PKG-readme.txt. Return the
descriptor."
(if (file-readable-p "ChangeLog") (insert-file-contents "ChangeLog"))
(let ((old-md5 (md5 (current-buffer))))
(erase-buffer)
- ;; git --no-pager log --date=short --format="%cd %aN
<%ae>%n%n%w(80,8,8)%B%n" | sed 's/^ /\t/'
- (call-process "bzr" nil (current-buffer) nil
- "log" "--gnu-changelog" ".")
+ (let ((default-directory
+ (file-name-as-directory (expand-file-name dir srcdir))))
+ (call-process "git" nil (current-buffer) nil
+ "log" "--date=short"
+ "--format=%cd %aN <%ae>%n%n%w(80,8,8)%B%n"
+ "."))
+ (tabify (point-min) (point-max))
+ (goto-char (point-min))
+ (while (re-search-forward "\n\n\n+" nil t)
+ (replace-match "\n\n"))
(if (equal old-md5 (md5 (current-buffer)))
(message "ChangeLog's md5 unchanged for %S" dir)
(write-region (point-min) (point-max) "ChangeLog" nil 'quiet)))))))
(defun archive--process-multi-file-package (dir pkg)
"Deploy the contents of DIR into the archive as a multi-file package.
-Rename DIR/ to PKG-VERS/, and write the package commentary to
-PKG-readme.txt. Return the descriptor."
+Rename DIR/ to PKG-VERS/, and return the descriptor."
(let* ((exp (archive--multi-file-package-def dir pkg))
(vers (nth 2 exp))
- (req (mapcar 'archive--convert-require (nth 4 exp)))
- (readme (expand-file-name "README" dir)))
+ (req (mapcar 'archive--convert-require (nth 4 exp))))
(unless (equal (nth 1 exp) pkg)
(error (format "Package name %s doesn't match file name %s"
(nth 1 exp) pkg)))
- ;; Write the readme file.
- (when (file-exists-p readme)
- (copy-file readme (concat pkg "-readme.txt") 'ok-if-already-exists))
(rename-file dir (concat pkg "-" vers))
(cons (intern pkg) (vector (version-to-list vers) req (nth 3 exp) 'tar))))
@@ -253,39 +259,6 @@ PKG-readme.txt. Return the descriptor."
(goto-char (point-min))
(read (current-buffer)))))
-(defun batch-make-site-dir (package-dir site-dir)
- (require 'package)
- (setq package-dir (expand-file-name package-dir default-directory))
- (setq site-dir (expand-file-name site-dir default-directory))
- (dolist (dir (directory-files package-dir t archive-re-no-dot))
- (if (not (file-directory-p dir))
- (message "Skipping non-package file %s" dir)
- (let* ((pkg (file-name-nondirectory dir))
- (autoloads-file (expand-file-name
- (concat pkg "-autoloads.el") dir))
- simple-p version)
- ;; Omit autoloads and .elc files from the package.
- (if (file-exists-p autoloads-file)
- (delete-file autoloads-file))
- (archive--delete-elc-files dir 'only-orphans)
- ;; Test whether this is a simple or multi-file package.
- (setq simple-p (archive--simple-package-p dir pkg))
- (if simple-p
- (progn
- (apply 'archive--write-pkg-file dir pkg simple-p)
- (setq version (car simple-p)))
- (setq version
- (nth 2 (archive--multi-file-package-def dir pkg))))
- (make-symbolic-link (expand-file-name dir package-dir)
- (expand-file-name (concat pkg "-" version)
- site-dir)
- t)
- (let ((make-backup-files nil))
- (package-generate-autoloads pkg dir))
- (let ((load-path (cons dir load-path)))
- ;; FIXME: Don't compile the -pkg.el files!
- (byte-recompile-directory dir 0))))))
-
(defun archive--refresh-pkg-file ()
(let* ((dir (directory-file-name default-directory))
(pkg (file-name-nondirectory dir))
@@ -297,16 +270,6 @@ PKG-readme.txt. Return the descriptor."
;; (message "Not refreshing pkg description of %s" pkg)
)))
-(defun batch-make-site-package (sdir)
- (let* ((dest (car (file-attributes sdir)))
- (pkg (file-name-nondirectory (directory-file-name (or dest sdir))))
- (dir (or dest sdir)))
- (let ((make-backup-files nil))
- (package-generate-autoloads pkg dir))
- (let ((load-path (cons dir load-path)))
- ;; FIXME: Don't compile the -pkg.el files!
- (byte-recompile-directory dir 0))))
-
(defun archive--write-pkg-file (pkg-dir name version desc requires &rest
ignored)
(let ((pkg-file (expand-file-name (concat name "-pkg.el") pkg-dir))
(print-level nil)
@@ -341,7 +304,7 @@ PKG-readme.txt. Return the descriptor."
<meta http-equiv=\"Content-Type\" content=\"text/html; charset=utf-8\">
</head>
<body>
-<h1 align=\"center\">%s</h1>"
+<h1 align=\"center\">%s</h1>\n"
title title))
(defun archive--html-bytes-format (bytes) ;Aka memory-usage-format.
@@ -399,8 +362,33 @@ PKG-readme.txt. Return the descriptor."
(buffer-string)))))))
(defun archive--quote (txt)
- (replace-regexp-in-string "<" "<"
- (replace-regexp-in-string "&" "&" txt)))
+ (replace-regexp-in-string "<" "<"
+ (replace-regexp-in-string "&" "&" txt)))
+
+(defun archive--insert-repolinks (name srcdir mainsrcfile)
+ (let ((url (archive--get-prop "URL" name srcdir mainsrcfile)))
+ (if url
+ (insert (format "<p>Origin: <a href=%S>%s</a></p>\n"
+ url (archive--quote url)))
+ (let* ((externals
+ (with-temp-buffer
+ (insert-file-contents
+ (expand-file-name "../../../elpa/externals-list" srcdir))
+ (read (current-buffer))))
+ (external (eq :external (nth 1 (assoc name externals))))
+ (git-sv "http://git.savannah.gnu.org/")
+ (urls (if external
+ '("cgit/emacs/elpa.git/?h=externals/"
+
"gitweb/?p=emacs/elpa.git;a=shortlog;h=refs/heads/externals/")
+ '("cgit/emacs/elpa.git/tree/packages/"
+ "gitweb/?p=emacs/elpa.git;a=tree;f=packages/"))))
+ (insert (format
+ (concat "<p>Browse repository: <a href=%S>%s</a>"
+ " or <a href=%S>%s</a></p>\n")
+ (concat git-sv (nth 0 urls) name)
+ 'CGit
+ (concat git-sv (nth 1 urls) name)
+ 'Gitweb))))))
(defun archive--html-make-pkg (pkg files)
(let* ((name (symbol-name (car pkg)))
@@ -420,16 +408,10 @@ PKG-readme.txt. Return the descriptor."
(let ((maint (archive--get-prop "Maintainer" name srcdir mainsrcfile)))
(when maint
(insert (format "<p>Maintainer: %s</p>\n" (archive--quote maint)))))
- (let* ((urlkind "Origin")
- (url
- (or (archive--get-prop "URL" name srcdir mainsrcfile)
- (progn
- (setq urlkind "Repository")
- (concat
"http://bzr.sv.gnu.org/lh/emacs/elpa/files/head:/packages/" name)))))
- (insert (format "<p>%s: <a href=%S>%s</a></p>"
- urlkind url (archive--quote url))))
+ (archive--insert-repolinks name srcdir mainsrcfile)
(let ((readme (archive--get-section "Commentary" "README" srcdir
mainsrcfile)))
(when readme
+ (write-region readme nil (concat name "-readme.txt"))
(insert "<h2>Full description</h2><pre>\n" (archive--quote readme)
"\n</pre>\n")))
(unless (< (length files) 2)
@@ -489,7 +471,7 @@ PKG-readme.txt. Return the descriptor."
;;; Maintain external packages.
-(defun archive-add/remove-externals ()
+(defun archive-add/remove/update-externals ()
(let ((exts (with-current-buffer (find-file-noselect "externals-list")
(goto-char (point-min))
(read (current-buffer)))))
@@ -510,8 +492,11 @@ PKG-readme.txt. Return the descriptor."
(progn (delete-directory dir 'recursive t)
(message "Deleted all of %s" dir))
(message "Keeping leftover unclean %s:\n%s" dir status))))))
- (pcase-dolist (`(,dir ,kind ,url) exts)
+ (pcase-dolist (`(,dir ,kind ,_url) exts)
(cond
+ ((eq kind :subtree) nil) ;Nothing to do.
+ ((not (eq kind :external))
+ (message "Unknown external package kind `%S' for %s" kind dir))
((not (file-exists-p dir))
(let* ((branch (concat "externals/" dir))
(output
@@ -521,17 +506,22 @@ PKG-readme.txt. Return the descriptor."
branch (concat "origin/" branch))
(call-process "git" nil t nil "clone"
"--shared" "--branch" branch "../" dir)
- ;; (let ((default-directory (file-name-as-directory
- ;; (expand-file-name dir))))
- ;; (call-process "git" nil t nil "branch"
- ;; "-m" branch "master"))
+ (let ((default-directory (file-name-as-directory
+ (expand-file-name dir))))
+ ;; (call-process "git" nil t nil "branch"
+ ;; "-m" branch "master")
+ (call-process "git" nil t nil "remote"
+ "set-url" "--push" "origin"
+
"git+ssh://git.sv.gnu.org/srv/git/emacs/elpa.git"))
(buffer-string))))
(message "Cloning branch %s:\n%s" dir output)))
((not (file-directory-p (concat dir "/.git")))
(message "%s is in the way of an external, please remove!" dir))
(t
- (let ((default-directory (expand-file-name dir)))
+ (let ((default-directory (file-name-as-directory
+ (expand-file-name dir))))
(with-temp-buffer
+ (message "Running git pull in %S" default-directory)
(call-process "git" nil t nil "pull")
(message "Updated %s:%s" dir (buffer-string))))
))))))
- [elpa] elpa-admin f25cda3 303/357: * GNUmakefile (%.elc): Use load-prefer-newer, (continued)
- [elpa] elpa-admin f25cda3 303/357: * GNUmakefile (%.elc): Use load-prefer-newer, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 21c69f7 308/357: Merge commit 'c304ac9a80ab7e96f705771a3842a48bf5a32c8a', Stefan Monnier, 2020/12/10
- [elpa] elpa-admin cdefe40 311/357: Improve copying of :core files; update .gitignore, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 3ca74c1 011/357: preparing for new rst format document, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin a703bf2 025/357: Merge branch 'master' of github.com:rocky/emacs-load-relative, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin b584cdf 007/357: # Updated., Stefan Monnier, 2020/12/10
- [elpa] elpa-admin bd19e1f 015/357: Start to use load-relative. dbgr-core.el: fix bug in dbgr-term-sentinal, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin a53e009 023/357: Merge branch 'master' of github.com:rocky/emacs-dbgr, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 3c6d0e1 037/357: Copy examples from wcheck-language-data doc to README, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin f8051b4 032/357: Fixes to archive-contents.el and package-update.sh., Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 81a6a01 088/357: Fix up scripts to build archive from Git,
Stefan Monnier <=
- [elpa] elpa-admin daf3553 099/357: Sync from ack/master, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 96a8283 129/357: Merge branch 'master' of git.sv.gnu.org:/srv/git/emacs/elpa, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin adc1e5b 146/357: Merge branch 'master' of git+ssh://git.sv.gnu.org/srv/git/emacs/elpa, Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 88fb267 150/357: Update README to tell how to checkout a single external package., Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 779c1d2 165/357: Add 'packages/loc-changes/' from commit '8447baff7cb4839ef8d1d747a14e5da85d0cee5b', Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 84f4eb7 153/357: * wcheck-mode: New package., Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 97cad46 171/357: Merge commit '3fa56d7925d5b24c5cd11c61bd6b3235e753a055', Stefan Monnier, 2020/12/10
- [elpa] elpa-admin f936e1e 180/357: Merge dummy commit to add the gnorb subtree metadata., Stefan Monnier, 2020/12/10
- [elpa] elpa-admin 5a1a2bc 184/357: Merge commit '32b1944d5f0a65aa10c6768f4865f7ed1de8eb49' as 'packages/pinentry', Stefan Monnier, 2020/12/10
- [elpa] elpa-admin cc33931 222/357: Add test capability to ELPA, Stefan Monnier, 2020/12/10