Hi On Fri, Sep 1, 2023 at 9:23 PM Daniel P. Berrangé <berra...@redhat.com> wrote: > > On Wed, Aug 30, 2023 at 01:38:23PM +0400, marcandre.lur...@redhat.com wrote: > > From: Marc-André Lureau <marcandre.lur...@redhat.com> > > > > Move common declarations to console-priv.h, and add a new unit > > console-vc.c which will handle VC/chardev rendering, when pixman is > > available. > > > > (if necessary, the move could be done chunk by chunks) > > > > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> > > --- > > ui/console-priv.h | 43 ++ > > ui/console-vc.c | 1079 ++++++++++++++++++++++++++++++++++++++++++++ > > ui/console.c | 1096 +-------------------------------------------- > > ui/meson.build | 1 + > > 4 files changed, 1126 insertions(+), 1093 deletions(-) > > create mode 100644 ui/console-priv.h > > create mode 100644 ui/console-vc.c > > > > diff --git a/ui/console-vc.c b/ui/console-vc.c > > new file mode 100644 > > index 0000000000..656f099556 > > --- /dev/null > > +++ b/ui/console-vc.c > > @@ -0,0 +1,1079 @@ > > +/* > > + * SPDX-License-Identifier: GPL-2.0-or-later > > + * QEMU VC > > This code is moved out of console.c, which (surprisingly) appears > to have an MIT license header. >
updated > > The include/ui/console.h header meanwhile has no license, so is > valid to assume it is GPL-2.0-or-later I guess. > Do you think I should split this patch in multiple patches? or can you r-b it with the license change? thanks -- Marc-André Lureau
