[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Supporting multiline Package-Requires header
From: |
Phillip Lord |
Subject: |
Re: Supporting multiline Package-Requires header |
Date: |
Mon, 10 Aug 2015 13:05:02 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Kaushal <address@hidden> writes:
> I think you meant to prefix the comment chars on each line of the
> multi-line header, correct?
>
> As for preference, I like (1)
>
> ;; Package-Requires: ((pkg-a "4.2.0")
> ;; (pkg-b "1.0") (pkg-c "2.0"))
>
> In (2), it is not clear where the requires end.
I dislike this format, though, because it is too easy to miss of the
last paren and then quite hard to detect (M-x check-parens doesn't check
these parens because they are in comments). This is already a problem
with the current format where there are several dependencies.
I raise possibility (3)
Package-Requires: ((pkg-a "4.2.0) (pkg-b "1.0))
((pkg-c "1.2))
Although I would be concerned that some existing tools would silently
drop the second line. One advantage with (1) is that the first line is
invalid wrt to the existing format. There is something to be send for a
format change that should cleanly break existing tools: fail fast
behaviour seems a good thing. The first packages that go this way on
ELPA will fail on an older versions of Emacs than the first one to have
this support in package.el.
Phil
- Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/10
- Re: Supporting multiline Package-Requires header, Kaushal, 2015/08/10
- Re: Supporting multiline Package-Requires header, Artur Malabarba, 2015/08/10
- Re: Supporting multiline Package-Requires header,
Phillip Lord <=
- 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, 2015/08/12