Re: [spice-server v2 2/2] build: Prepare for 0.14.1 release

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

 



On Mon, Aug 13, 2018 at 06:34:19AM -0400, Frediano Ziglio wrote:
> > 
> > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> > ---
> >  NEWS         | 33 +++++++++++++++++++++++++++++++++
> >  configure.ac |  2 +-
> >  2 files changed, 34 insertions(+), 1 deletion(-)
> > 
> > diff --git a/NEWS b/NEWS
> > index 51bb4a2e6..1577b013f 100644
> > --- a/NEWS
> > +++ b/NEWS
> > @@ -1,3 +1,36 @@
> > +Major Changes in 0.14.1:
> > +========================
> > +
> > +The main change in this release is the addition of a new protocol extension
> > +in order to support streaming the remote display as a video stream rather
> > than
> > +going through the QXL protocol. Together with spice-streaming-agent, and/or
> > with
> > +more work on the qemu/spice-server side, this should allow streaming of 3D
> > +accelerated VMs in the future. At this point, this part of spice-server is
> > +still work in progress.
> > +
> > +* add new org.spice-space.stream.0 channel used for passing an encoded video
> > +* stream from the guest to the client
> > +* add support for TCP_CORK to reduce the amount of packets that we send
> > +* fix cursor related migration crash
> > +* fix regression causing sound recording to be muted after
> > +  client disconnection/reconnection (introduced in 0.13.90)
> > +* fix regression in corner cases where images could be sent uncompressed
> > +  when they used to be compressed with QUIC
> > +* disable TLS 1.0 support
> > +* CELT 0.5.1 support is now disabled by default. If celt051-devel is
> > installed
> > +  at build-time, --enable-celt051/--disable-celt051 must be explicitly
> > specified
> > +* drop support for unsupported OpenSSL version. OpenSSL 1.0.0 or newer is
> > now
> > +  required
> > +* bumped minimum required glib version to 2.32
> > +* endianness fixes
> > +* (small) leak fixes
> > +* usual round of code cleanups
> > +* not directly related to the release, but the upstream git repository is
> > now
> > +* hosted on gitlab.freedesktop.org
> 
> Extra star (*) at the beginning of the line. The "not directly related to the release"
> looks like a comment for patch reviewer like "should we have this line or
> completely remove it?".

It's a comment meant for the reader of the NEWS file, as imo it's useful
information, but has nothing to do with 0.14.1.

> 
> > +
> > +* red-parse-qxl: Avoid invalid flag usage ?
> > +
> 
> I think people would be more interesting on the result of this and
> the reason. In this case to avoid very suboptimal case (worth mentioned probably
> Windows users?). Also the comment is really developer focused, a normal user
> won't read much from this comment (I'm really confused is this file should
> just directed to developers or also to users).

And actually this one was not supposed to be in the patch (I've removed
it for now), but I meant to ask if this is worth documenting it in NEWS
or not. Do we want it? And yes, this cannot be formulated this way, if
we add it to the NEWS file, it will mention the user observable result.
:)

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]