[debbugs-tracker] bug#24589: closed ([PATCH 0/2] Fix for command-debug-status)

[GNU bug Tracking System]

Your message dated Tue, 04 Oct 2016 17:43:18 +0300
with message-id <address@hidden>
and subject line Re: bug#24555: [PATCH] Remove unused variable 
`command-debug-status'
has caused the debbugs.gnu.org bug report #24555,
regarding [PATCH 0/2] Fix for command-debug-status
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
24555: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=24555
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> [PATCH 0/2] Fix for command-debug-status
>
>
>
>
> Date: 
>
> Sun,  2 Oct 2016 18:59:53 +0200
>
>
>
>
> Hello,
>
> This patch series restore the lost functionality of variable 
> command-debug-status and also deprecate it in order for it to be removed in the 
> future.
>
> Regards,
>
> Philippe Vaucher (2):
>   Restore command-debug-status functionality
>   Deprecate variable command-debug-status
>
>  doc/lispref/debugging.texi | 2 ++
>  lisp/subr.el               | 2 ++
>  src/callint.c              | 5 ++++-
>  3 files changed, 8 insertions(+), 1 deletion(-)
>
> -- 
> 2.10.0
>
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#24555: [PATCH] Remove unused variable `command-debug-status'
>
>
>
>
> Date: 
>
> Tue, 04 Oct 2016 17:43:18 +0300
>
>
>
>
> > From: Philippe Vaucher <address@hidden>
> > Date: Mon, 3 Oct 2016 09:42:14 +0200
> > Cc: Stefan Monnier <address@hidden>, address@hidden
> > 
> > And again new patches without a typo :-)
>
> Thanks, I pushed this to the emacs-25 branch, and I'm marking this
> bug done.
>
> Please note that your patch had a fatal flaw: specbind needs a
> (quoted) symbol, not its value.  Using Vcommand_debug_status there
> produced a broken binary that would display an error message and
> become unresponsive.  See what I actually committed for the details.
>
> Also, the obsolescence warning needed some minor tweaks (it turns out
> that my advice to prepend a semi-colon was a bad idea, as a semi-colon
> and a newline are produced by Emacs automatically).
>
> Please always test the build after you patch it, to make sure the
> behavior is correct and no bugs creep in.
>
> Finally, in the future please provide commit log messages for the
> changes formatted in the ChanegLog style, as described in CONTRIBUTE.
> I wrote them for this commit, please see the commit for the details of
> the formatting we use.
>
> Thanks a lot for working on this.
>
>
> --- End Message ---