Bug#1138884: dh-elpa: simplify emacs-bundled addon handling

Xiyue Deng <[email protected]> Fri, 12 Jun 2026 23:23:04 -0700

Newsgroups	gmane.linux.debian.devel.emacsen
Message-ID	<87o6hf7yd3.fsf__41512.5921181001$1781331927$gmane$org@debian-hx90.lan>

Sean Whitton <[email protected]> writes:

> Xiyue Deng [11/Jun  7:13pm -07] wrote:
>> Hi Sean,
>>
>> Sean Whitton <[email protected]> writes:
>>
>>> Xiyue Deng [10/Jun 11:23pm -07] wrote:
>>>> So I guess this can really work!
>>>
>>> Nice.
>>>
>>>> Will continue with this after the integration test is merged so that
>>>> we have some test coverage.
>>>
>>> IMO this is more important than that other work and can merge first.
>>
>> Ah I was hoping that the test branch is safer as it doesn't change any
>> functions and can go in first.
>>
>> Anyway, I experimented a change[1] and tested it in the
>> `built-in-addon-handling' branch, and rebased it on to the test branch
>> for testing. One of the test cases has cl-lib in Package-Requires, and
>> the generated diff looked OK[2].
>>
>> So now we only remove `emacs' from the list of dependencies. The code
>> can be further simplified, but I'd like to keep the current structure in
>> case we want to do more later.
>
> I think I'd prefer to strip out everything we don't need anymore -- it's
> always there in the Git history if required again.

Ack. Now I only keep the comparison with "emacs". Also shortened the doc
string. PTAL.

-- 
Regards,
Xiyue Deng

signature.asc (application/pgp-signature, 857 B)