GNU bug report logs - #31121
[PATCH 1/3] gnu: Add python2-urlgrabber

Previous Next

Package: guix-patches;

Reported by: Tomáš Čech <sleep_walker <at> gnu.org>

Date: Tue, 10 Apr 2018 09:26:01 UTC

Severity: normal

Tags: patch

Done: Tomáš Čech <sleep_walker <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #50 received at 31121 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Tomáš Čech <sleep_walker <at> gnu.org>
Cc: 31121 <at> debbugs.gnu.org
Subject: Re: [bug#31121] [PATCH 3/3] gnu: Add osc.
Date: Thu, 19 Apr 2018 17:41:07 +0200
[Message part 1 (text/plain, inline)]
Tomáš Čech <sleep_walker <at> gnu.org> writes:

> Hi,
>
> On Wed, Apr 18, 2018 at 01:21:02PM +0200, Marius Bakke wrote:
>>Tomáš Čech <sleep_walker <at> gnu.org> writes:
>>
>>>  * gnu/packages/build-tools.scm (osc): New variable.
>>
>>[...]
>>
>>> +(define-public osc
>>> +  (package
>>> +    (name "osc")
>>      ^
>>Indentation off here too.
>
> I'm sorry, I'm a bit confused here. My Emacs indent the code like this
> and indent script you pointed in previous mail doesn't change it for me.
>
> You would like me to reduce one space? Like this?
>
> (define-public osc
>   (package
>    (name "osc")
>
> It seems that this is not unified in the code. Can I somehow teach my
> Emacs indentantation rules?

Whoops.  I didn't actually verify this, it just looked wrong.  If
someone has a dir-locals.el parser for human brains I'd be interested in
buying one.

Trying it locally in emacs shows that your original level was correct
(though I think the other instance was actually off, no?).  Anyway LGTM!
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 7 years and 28 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.