[bug#38326] [PATCH 2/2] gnu: Add emacs-telega.

[Brett Gilio]

Ludovic Courtès <address@hidden> writes:

> Hi Brett,
>
> Brett Gilio <address@hidden> skribis:
>
>>>From c632d01dd48f89dab2822d5a81d091ca68b43c62 Mon Sep 17 00:00:00 2001
>> From: Brett Gilio <address@hidden>
>> Date: Fri, 22 Nov 2019 12:54:44 -0600
>> Subject: [PATCH 2/2] gnu: Add emacs-telega.
>>
>> * gnu/packages/emacs-xyz.scm (emacs-telega): New variable.
>> ---
>>  gnu/packages/emacs-xyz.scm | 58 ++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 58 insertions(+)
>>
>> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
>> index 0caf12a423..7ff53ecf1e 100644
>> --- a/gnu/packages/emacs-xyz.scm
>> +++ b/gnu/packages/emacs-xyz.scm
>> @@ -114,6 +114,7 @@
>>    #:use-module (gnu packages image)
>>    #:use-module (gnu packages linux)
>>    #:use-module (gnu packages libevent)
>> +  #:use-module (gnu packages messaging)
>>    #:use-module (gnu packages music)
>
> We have a problem (not your fault!): applying this hunk on top of
> current master breaks it all, due to circular top-level references in
> the modules.
>
> I haven’t been able to spend much time investigating, but we first need
> to find out where that comes from.
>
> Ludo’.

Ludo,

That is perfectly okay. I have some revisions I will be sharing to this
patch soon anyways.

Regarding the circular top-level references. That is interesting. I was
not familiar with this issue. Is there an open bug report for this? I'd
be willing to investigate it a bit.

It is particularly interesting, because emacs-xyz is not used in the
messaging module, so the circular reference logic is indeed obscured
somewhere.

Best!

-- 
Brett M. Gilio
https://git.sr.ht/~brettgilio/