Subject: Re: bug#60418: [PATCH] Add :vc keyword to use-package

From: Philip Kaludercic <[email-masked]>
Date: Sun, 07 May 2023 07:34:07 +0000

Eli Zaretskii <[email-masked]> writes:

>> From: Philip Kaludercic <[email-masked]>
>> Cc: [email-masked],  [email-masked],
>>   [email-masked],  [email-masked]
>> Date: Sat, 06 May 2023 18:50:02 +0000
>> 
>> Eli Zaretskii <[email-masked]> writes:
>> 
>> >> This is true, for package-vc-install, but the idea was not do this for
>> >> the :vc keyword to use-package.
>> >
>> > But I wrote the above as a comment to a patch to package-vc.el, not to
>> > use-package.  So why use-package is relevant here?
>> 
>> Because this patch is related to use-package, and a keyword that would
>> allow for use-package to invoke package-vc-install?
>
> Yes, but the doc string in package-vc-install should describe
> everything that package-vc supports, not just the subset we think will
> be useful in use-package via :rev.

In that case there was some other unrelated confusion.

> Anyway, I think we are in agreement, as your last doc string is
> comprehensive enough to make me happy.

OK, great!  Thank you for your input.

-- 
Philip Kaludercic