--- Begin Message ---
Subject: |
package-definition-location is wrong for inherited packages |
Date: |
Fri, 29 Apr 2022 11:56:34 -0400 |
Hi Guix,
Here's another problem with source location I've discovered when
attempting to locate the source of `python2-pytest-warnings', which
reads like:
--8<---------------cut here---------------start------------->8---
(define-public python2-pytest-warnings
(package (inherit (package-with-python2
(strip-python2-variant python-pytest-warnings)))
(properties `((superseded . ,python2-pytest)))))
--8<---------------cut here---------------end--------------->8---
The define-public line is at line 2569, but:
--8<---------------cut here---------------start------------->8---
(define package (car (find-packages-by-name "python2-pytest-warnings")))
(package-definition-location package) location->source-properties)
--8<---------------cut here---------------end--------------->8---
returns 2570, the line below.
Thanks,
Maxim
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#55186: package-definition-location is wrong for inherited packages |
Date: |
Mon, 16 May 2022 00:56:19 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hi Ludovic,
Ludovic Courtès <ludo@gnu.org> writes:
> Hi,
>
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> Here's another problem with source location I've discovered when
>> attempting to locate the source of `python2-pytest-warnings', which
>> reads like:
>>
>> (define-public python2-pytest-warnings
>> (package (inherit (package-with-python2
>> (strip-python2-variant python-pytest-warnings)))
>> (properties `((superseded . ,python2-pytest)))))
>
> It works for me:
Thanks for checking, this works for me too now. I think I was amidst
modifications to the code, that induced me in error. I've fixed what I
think was a typo in a comment in the code that made my understanding
difficult in the early commits of https://issues.guix.gnu.org/55424.
Happily closing,
Maxim
--- End Message ---