Hi Simon,
On 2023-03-07 13:16, Simon Tournier wrote:
Toggle quote (30 lines)
> Hi,
>
> Cool you had the number 62000. ;-)
>
> On Mon, 06 Mar 2023 at 12:48, Bruno Victal <mirai@makinata.eu> wrote:
>> Suspected file: .dir-locals.el
>>
>> Using 'package' as a field in define-configuration results in
>> inconsistent indentation:
>>
>> --8<---------------cut here---------------start------------->8---
>> (define-configuration/no-serialization mympd-configuration
>> (package
>> (file-like mympd) ;;;; <<<<<<<<<<<<<<<<<<<<< notice how it's indented with 2 spaces
>> "The package object of the myMPD server."
>> empty-serializer)
>>
>> (shepherd-requirement
>> (list-of-symbol '()) ;;;; <<<<<<<<<<<<<<<<<<<<< vs 1 space
>> "This is a list of symbols naming Shepherd services that this service
>> will depend on."
>> empty-serializer)
>>
>> ;; ...
>> )
>> --8<---------------cut here---------------end--------------->8---
>
> What do you mean by inconsistent? Do you think that
> ’sheperd-requirement’ should be management by .dir-locals.el?
It's not 'shepherd-requirement' that should be managed by .dir-locals.el, it could have been named something else.
Compare with 'mympd-ip-acl':
Toggle snippet (10 lines)
(define-configuration/no-serialization mympd-ip-acl
(allow
(list-of-string '())
"Allowed IP addresses.")
(deny
(list-of-string '())
"Disallowed IP addresses."))
Toggle quote (3 lines)
> Well, since ’sheperd-requirement’ is not in .dir-locals.el, the
> indentation seems expected. What do you consider as a bug?
Having a field named 'package' within define-configuration causes incorrect indentation.
Cheers,
Bruno.