Subject: bug#36301: closed (Re: bug#36301: Package-Requires should support multiple lines)

From: [email-masked] (GNU bug Tracking System)
Date: Sun, 01 Sep 2019 17:35:02 +0000

Your bug report

#36301: Package-Requires should support multiple lines

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
36301: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=36301
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#36301: Package-Requires should support multiple lines
From: Thomas Fitzsimmons <[email-masked]>
To: Noam Postavsky <[email-masked]>
Cc: [email-masked], Glenn Morris <[email-masked]>
Date: Sun, 01 Sep 2019 13:34:23 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

Thomas Fitzsimmons <[email-masked]> writes:

> Noam Postavsky <[email-masked]> writes:
>
>> Thomas Fitzsimmons <[email-masked]> writes:
>>> I wonder if there's a better way to do the equivalent of mapconcat but
>>> produce nil on a nil input sequence.
>>
>> Not exactly, but I think this is good place for and-let*:
>
> Yes, looks good; "make check" passes on these tests, so I pushed this
> new patch.  I'll wait for other test results before closing this.

The Hydra build succeeded too:

https://hydra.nixos.org/build/99577710

so I'm closing this.

Thanks,
Thomas

Subject: Package-Requires should support multiple lines
From: Thomas Fitzsimmons <[email-masked]>
To: [email-masked]
Date: Wed, 19 Jun 2019 21:55:27 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi,

The Package-Requires line in my GNU ELPA package, Excorporate, is much
longer than 80 characters and I would like to wrap it.  It would be nice
if package.el allowed Package-Requires to span multiple lines (see
patch).  It's probably too late to change this but I thought I'd file a
bug report anyway.

Thomas

diff --git a/lisp/emacs-lisp/package.el b/lisp/emacs-lisp/package.el
index 46f7c91272..5c9b12732d 100644
--- a/lisp/emacs-lisp/package.el
+++ b/lisp/emacs-lisp/package.el
@@ -1016,7 +1016,8 @@ package-buffer-info
     (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
+            (mapconcat 'identity (lm-header-multiline "package-requires") " "))
            ;; Prefer Package-Version; if defined, the package author
            ;; probably wants us to use it.  Otherwise try Version.
            (pkg-version