[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Supporting multiline Package-Requires header
From: |
Artur Malabarba |
Subject: |
Re: Supporting multiline Package-Requires header |
Date: |
Wed, 12 Aug 2015 15:23:05 +0100 |
>> That said, I don't think the proposed patch makes this situation any worse.
>
> Oh, I missed it. Can you resend it?
My brain probably glitched, I meant "the proposed feature".
There was no patch yet, but here it is now:
---
lisp/emacs-lisp/package.el | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lisp/emacs-lisp/package.el b/lisp/emacs-lisp/package.el
index 9677208..a2027d2 100644
--- a/lisp/emacs-lisp/package.el
+++ b/lisp/emacs-lisp/package.el
@@ -982,7 +982,8 @@ boundaries."
(narrow-to-region start (point))
(require 'lisp-mnt)
;; Use some headers we've invented to drive the process.
- (let* ((requires-str (lm-header "package-requires"))
+ (let* ((requires-str (when-let ((reqs (lm-header-multiline
"package-requires")))
+ (apply #'concat reqs)))
;; Prefer Package-Version; if defined, the package author
;; probably wants us to use it. Otherwise try Version.
(pkg-version
--
2.4.6
- Re: Supporting multiline Package-Requires header, (continued)
- Re: Supporting multiline Package-Requires header, Phillip Lord, 2015/08/10
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/10
- Re: Supporting multiline Package-Requires header, Phillip Lord, 2015/08/10
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/10
- Re: Supporting multiline Package-Requires header, Bozhidar Batsov, 2015/08/10
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/10
- Re: Supporting multiline Package-Requires header, Phillip Lord, 2015/08/11
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/11
- Re: Supporting multiline Package-Requires header, Bozhidar Batsov, 2015/08/12
- Re: Supporting multiline Package-Requires header, Stefan Monnier, 2015/08/12
- Re: Supporting multiline Package-Requires header,
Artur Malabarba <=
- Re: Supporting multiline Package-Requires header, Stefan Monnier, 2015/08/12
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/12
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/13
- Re: Supporting multiline Package-Requires header, Stefan Monnier, 2015/08/13
- Re: Supporting multiline Package-Requires header, Rasmus, 2015/08/13
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/13
- Re: Supporting multiline Package-Requires header, Rasmus, 2015/08/13
- Re: Supporting multiline Package-Requires header, Stefan Monnier, 2015/08/13
- Re: Supporting multiline Package-Requires header, Achim Gratz, 2015/08/17
- Re: Supporting multiline Package-Requires header, Stefan Monnier, 2015/08/18