diff options
| -rw-r--r-- | etc/NEWS | 14 | ||||
| -rw-r--r-- | lisp/emacs-lisp/package.el | 9 |
2 files changed, 20 insertions, 3 deletions
| @@ -1075,6 +1075,20 @@ it can't find the config file. | |||
| 1075 | 1075 | ||
| 1076 | ** Package | 1076 | ** Package |
| 1077 | 1077 | ||
| 1078 | *** Warn if "footer line" is missing, but still install package. | ||
| 1079 | package.el used to refuse to install a package without the so-called | ||
| 1080 | "footer line", which appears at the very end of the file: | ||
| 1081 | |||
| 1082 | ;;; FILENAME ends here | ||
| 1083 | |||
| 1084 | package.el will now install packages without this line, but it will | ||
| 1085 | issue a warning. To avoid this warning, packages should keep the | ||
| 1086 | "footer line". | ||
| 1087 | |||
| 1088 | Note that versions of Emacs older than 27.1 will not only refuse to | ||
| 1089 | install packages without such a line -- they will be unable to parse | ||
| 1090 | package data. It is therefore recommended to keep this line. | ||
| 1091 | |||
| 1078 | *** Change of 'package-check-signature' for packages with multiple sigs | 1092 | *** Change of 'package-check-signature' for packages with multiple sigs |
| 1079 | In previous Emacsen, 't' checked that all signatures are valid. | 1093 | In previous Emacsen, 't' checked that all signatures are valid. |
| 1080 | Now 't' only checks that at least one signature is valid and the new 'all' | 1094 | Now 't' only checks that at least one signature is valid and the new 'all' |
diff --git a/lisp/emacs-lisp/package.el b/lisp/emacs-lisp/package.el index 645e831bcc9..8c8e86aead9 100644 --- a/lisp/emacs-lisp/package.el +++ b/lisp/emacs-lisp/package.el | |||
| @@ -1082,10 +1082,13 @@ boundaries." | |||
| 1082 | (let ((file-name (match-string-no-properties 1)) | 1082 | (let ((file-name (match-string-no-properties 1)) |
| 1083 | (desc (match-string-no-properties 2)) | 1083 | (desc (match-string-no-properties 2)) |
| 1084 | (start (line-beginning-position))) | 1084 | (start (line-beginning-position))) |
| 1085 | ;; The terminating comment format could be extended to accept a | 1085 | ;; This warning was added in Emacs 27.1, and should be removed at |
| 1086 | ;; generic string that is not in English. | 1086 | ;; the earliest in version 31.1. The idea is to phase out the |
| 1087 | ;; requirement for a "footer line" without unduly impacting users | ||
| 1088 | ;; on earlier Emacs versions. See Bug#26490 for more details. | ||
| 1087 | (unless (search-forward (concat ";;; " file-name ".el ends here")) | 1089 | (unless (search-forward (concat ";;; " file-name ".el ends here")) |
| 1088 | (error "Package lacks a terminating comment")) | 1090 | (lwarn '(package package-format) :warning |
| 1091 | "Package lacks a terminating comment")) | ||
| 1089 | ;; Try to include a trailing newline. | 1092 | ;; Try to include a trailing newline. |
| 1090 | (forward-line) | 1093 | (forward-line) |
| 1091 | (narrow-to-region start (point)) | 1094 | (narrow-to-region start (point)) |