[Spice-devel] [PATCH spice-common 1/2] log: remove deprecated SPICE_ABORT_LEVEL support

[Jonathon Jongsma]

On Wed, 2017-06-14 at 12:00 +0200, Victor Toso wrote:
> Hi,
> 
> On Tue, Jun 13, 2017 at 06:31:38PM +0200, Christophe Fergeau wrote:
> > On Tue, Jun 13, 2017 at 03:47:53PM +0100, Frediano Ziglio wrote:
> > > This feature was marked obsolete by efd1d3cb4d8eee more than
> > > an year ago.
> > 
> > How many stable releases has this been in?
> 
> * Spice-gtk: v0.31, v0.32, v0.33 and I propose v0.34 as final.
> 
>   commit cc8adb63ee5b9a554b9e237f825741cf4fcc2c09
>   Author: Marc-André Lureau <marcandre.lureau at gmail.com>
>   Date:   Thu Mar 10 17:05:20 2016 +0100
> 
>       Update spice-common submodule

Unfortunately, these environment variables never really applied to
spice-gtk (even though it was ostensibly supposed to be a "common"
logging implementation). As I stated in a different email, defining
SPICE_DEBUG_LEVEL only enables debugging for the "Spice" domain, which
is not used by spice-gtk ("GSpice" is the spice-gtk domain). In theory,
SPICE_ABORT_LEVEL could affect spice-gtk, but only if spice-gtk calls
spice_log(), which it generally doesn't. spice-gtk implements its own
SPICE_DEBUG() macro that doesn't use spice_log(). So whether this
variable is kept or removed will not really affect spice-gtk at all.


> 
> But I guess you mean spice-server. That's annoying.
> 
>   commit b206efb30bf41cb2e071f75cee87901cad1c83b7
>   Author: Frediano Ziglio <fziglio at redhat.com>
>   Date:   Tue Feb 2 12:47:40 2016 +0000
> 
>       Update spice-common
> 
>       Needed to have the updated spice_marshaller_get_fd function
> 
> (wingsuit) spice (master 5dc55aa7) $ git describe --contains
> b206efb30b
> v0.13.1~192
> 
> > How many distros have been shipping said stable releases, for how
> > long?
> > 
> > (rethorical questions, the answer is 0).
> 
> Stable releases in spice, I see. IMHO, we should consider 0.13.1 for
> that. Stable releases in spice-server take too long.
> 

Yeah, I agree we probably need a new spice-server release soon.

Jonathon