[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#29856] [PATCH core-updates] guix: python-build-system:, Modify ".py
From: |
Danny Milosavljevic |
Subject: |
[bug#29856] [PATCH core-updates] guix: python-build-system:, Modify ".py" files in-place. |
Date: |
Tue, 2 Jan 2018 18:26:27 +0100 |
Hi Hartmut,
thanks for the review!
On Tue, 2 Jan 2018 17:13:15 +0100
Hartmut Goebel <address@hidden> wrote:
> * it kills "from __future__ import", which must be the first import
> statement (or even the first statement after any doc-string) to work.
... oops.
> Thus I suggest aiming to implement the solution discussed in that thread
> (see esp.
> <https://lists.gnu.org/archive/html/guix-devel/2017-11/msg00041.html>.
I like that approach. Nice...
> Beside of this, the patch suffers from some more issues. Sorry to say :-(
>
> * When converting PYTHONPATH into a list of python strings, these need
> to be quoted properly.
I agree.
> * The description (commit-message) of the patch is much to terse. It
> should describe the the reason and implications. Esp. it should
> describe the case this is fixing.
Sure.