[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: |
Tue, 11 Aug 2015 22:48:05 +0100 |
>>> What is the failure behaviour of package.el for this at the moment?
>>
>> Package.el will signal an error during installation if it's not a
>> valid sexp. The error itself will depend on what the problem is.
>> Here's what you get if you miss a closing paren for instance:
>> package-read-from-string: End of file during parsing
>
> My issue with this is that, iff I see this error, my first port of call
> is going to be check-parens, and generally look for unbalanced parens in
> the relevant file. And there are not going to be any, because in the
> file, there is no invalid sexp. There is a commented out, pseudo sexp.
> [...]
> Just my thoughts, I will leave the issue in your hands from there.
Thanks for the thoughts. I do think package.el could do better error
reporting in these situations (and I might take that up too).
That said, I don't think the proposed patch makes this situation any worse.
- 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, 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 <=
- 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
- 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