bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#58383: 29.0.50; Make it easier to invert vc-prepare-patches-separate


From: Philip Kaludercic
Subject: bug#58383: 29.0.50; Make it easier to invert vc-prepare-patches-separately
Date: Sun, 13 Nov 2022 18:17:43 +0000

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Philip Kaludercic <philipk@posteo.net>
>> Cc: spwhitton@spwhitton.name,  58383@debbugs.gnu.org
>> Date: Sun, 13 Nov 2022 16:45:26 +0000
>> 
>> Eli Zaretskii <eliz@gnu.org> writes:
>> 
>> > Can we take this opportunity to get rid of the passive voice and
>> > clarify the doc string, please?
>> 
>> Done:
>
> Thanks, and the same with package-vc-prepare-patch, please?
>
> Btw, this kind of reformatting:
>
>> -SUBJECT and REVISIONS are passed on to `vc-prepare-patch', which see.
>> -PKG must be a package description.
>> -Interactively, prompt for PKG, SUBJECT, and REVISIONS.  However,
>> -if the current buffer has marked commit log entries, REVISIONS
>> -are the tags of the marked entries, see `log-view-get-marked'."
>> +SUBJECT and REVISIONS are passed on to `vc-prepare-patch', which
>> +see.  PKG must be a package description.  Interactively, prompt
>> +for PKG, SUBJECT, and REVISIONS.  When invoked with a numerical
>> +prefix argument, the last N revisions will be used.  When invoked
>> +interactively in a Log View buffer with marked revisions, those
>> +revisions will be used."
>
> is IMO for the worse: when the sentence starting with "Interactively"
> begins a new line, it stands out, which helps users find the most
> relevant parts faster.

Addressed both issues here:

Attachment: 0001-Have-vc-prepare-patch-handle-prefix-arguments.patch
Description: Text Data

That being said, the patch now relies on changes made in
scratch/package-vc-fixes.  I can "backport" it onto master, or push it
in a few days when the issues related to that branch are resolved.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]