[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#65922] [qt-team PATCH v2 1/4] gnu: qtbase: Do not capture Python (a
From: |
Maxim Cournoyer |
Subject: |
[bug#65922] [qt-team PATCH v2 1/4] gnu: qtbase: Do not capture Python (again). |
Date: |
Sun, 17 Sep 2023 08:55:54 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi Andreas,
Andreas Enge <andreas@enge.fr> writes:
> Am Sat, Sep 16, 2023 at 10:51:22PM -0400 schrieb Maxim Cournoyer:
>> > This one looks good, except for indentation there, I think:
>> >> + (substitute*
>> >> + (search-input-file
>> >> + outputs "lib/qt5/mkspecs/features/uikit/devices.py")
>> >> + (((which "python3"))
>> >> + "/usr/bin/env python3")))))))
>> It's actually valid -- I just doubled check :-)
>
> Hm, I thought that arguments to functions should be aligned, so that
> "(search-..." and "(((which" should start in the same column?
> And that we would indent by either 1 or 2 (according to arcane rules I
> never memorised)?
According to our indentation rules in .dir-locals.el:
--8<---------------cut here---------------start------------->8---
(put 'substitute* 'scheme-indent-function 1)
--8<---------------cut here---------------end--------------->8---
Which, following the Emacs documentation for scheme-indent-function then
lisp-indent-function (C-h f scheme-indent-function RET), says (where N
is the 1 argument):
--8<---------------cut here---------------start------------->8---
* an integer N, meaning indent the first N arguments specially
(like ordinary function arguments), and then indent any further
arguments like a body;
--8<---------------cut here---------------end--------------->8---
The indentation produced is because of that special rule.
--
Thanks,
Maxim
- [bug#65922] [qt-team PATCH 1/4] gnu: qtbase: Do not capture Python (again)., (continued)
[bug#65922] [qt-team PATCH v2 3/4] gnu: qtbase: Reinstate date related tests., Maxim Cournoyer, 2023/09/13
[bug#65922] [qt-team PATCH v3 0/5] Avoid capturing python in qtbase, reinstate tests, Maxim Cournoyer, 2023/09/17
- [bug#65922] [qt-team PATCH v3 5/5] gnu: qtdeclarative: Add a debug output., Maxim Cournoyer, 2023/09/17
- [bug#65922] [qt-team PATCH v3 1/5] gnu: qtbase: Do not capture Python (again)., Maxim Cournoyer, 2023/09/17
- [bug#65922] [qt-team PATCH v3 3/5] gnu: qtbase: Reinstate date related tests., Maxim Cournoyer, 2023/09/17
- [bug#65922] [qt-team PATCH v3 2/5] gnu: qtbase: Add $TZDIR to native search paths., Maxim Cournoyer, 2023/09/17
- [bug#65922] [qt-team PATCH v3 4/5] gnu: qtbase: Disable problematic tests on powerpc64le., Maxim Cournoyer, 2023/09/17