[libvirt] [PATCH v2 0/7] vbox: drop support for many old versions

Daniel P. Berrangé posted 7 patches 5 years ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190412130040.21306-1-berrange@redhat.com
src/vbox/Makefile.inc.am      |    12 -
src/vbox/vbox_CAPI_v4_0.h     |  7451 -----------------------
src/vbox/vbox_CAPI_v4_1.h     |  7882 ------------------------
src/vbox/vbox_CAPI_v4_2.h     |  8855 ---------------------------
src/vbox/vbox_CAPI_v4_2_20.h  |  9001 ----------------------------
src/vbox/vbox_CAPI_v4_3.h     | 10210 -------------------------------
src/vbox/vbox_CAPI_v4_3_4.h   | 10321 --------------------------------
src/vbox/vbox_V4_0.c          |    37 -
src/vbox/vbox_V4_1.c          |    37 -
src/vbox/vbox_V4_2.c          |    13 -
src/vbox/vbox_V4_2_20.c       |    13 -
src/vbox/vbox_V4_3.c          |    13 -
src/vbox/vbox_V4_3_4.c        |    13 -
src/vbox/vbox_XPCOMCGlue.h    |     2 +-
src/vbox/vbox_common.h        |    14 +-
src/vbox/vbox_storage.c       |    14 +-
src/vbox/vbox_tmpl.c          |   218 +-
src/vbox/vbox_uniformed_api.h |    10 +-
18 files changed, 8 insertions(+), 54108 deletions(-)
delete mode 100644 src/vbox/vbox_CAPI_v4_0.h
delete mode 100644 src/vbox/vbox_CAPI_v4_1.h
delete mode 100644 src/vbox/vbox_CAPI_v4_2.h
delete mode 100644 src/vbox/vbox_CAPI_v4_2_20.h
delete mode 100644 src/vbox/vbox_CAPI_v4_3.h
delete mode 100644 src/vbox/vbox_CAPI_v4_3_4.h
delete mode 100644 src/vbox/vbox_V4_0.c
delete mode 100644 src/vbox/vbox_V4_1.c
delete mode 100644 src/vbox/vbox_V4_2.c
delete mode 100644 src/vbox/vbox_V4_2_20.c
delete mode 100644 src/vbox/vbox_V4_3.c
delete mode 100644 src/vbox/vbox_V4_3_4.c
[libvirt] [PATCH v2 0/7] vbox: drop support for many old versions
Posted by Daniel P. Berrangé 5 years ago
Per this link:

  https://www.virtualbox.org/wiki/Download_Old_Builds

    VirtualBox 6.0 (active maintenance)
    VirtualBox 5.2 (active maintenance)
    VirtualBox 5.1 (no longer supported, support ended 2018/04)
    VirtualBox 5.0 (no longer supported, support ended 2017/05)
    VirtualBox 4.3 (no longer supported, support ended 2015/12)
    VirtualBox 4.2 (no longer supported, support ended 2015/12)
    VirtualBox 4.1 (no longer supported, support ended 2015/12)
    VirtualBox 4.0 (no longer supported, support ended 2015/12)

We can certainly drop all the 4.x versions. The host OS those are
officially supported on are very outdated:

eg

  https://www.virtualbox.org/wiki/Download_Old_Builds_4_3

Linux Hosts:

    Ubuntu 13.04 ("Raring") / 13.10 ("Saucy") / 14.04 ("Trusty") / 14.10 ("Utopic")  i386 |  AMD64
    Ubuntu 12.10 ("Quantal")  i386 |  AMD64
    Ubuntu 12.04 ("Precise")  i386 |  AMD64
    Ubuntu 10.04 LTS ("Lucid")  i386 |  AMD64
    Debian 7 ("Wheezy")  i386 | AMD64
    Debian 6 ("Squeeze")  i386 |  AMD64
    openSUSE 12.3 / 13.1  i386 |  AMD64
    openSUSE 11.4 / 12.1 / 12.2  i386 |  AMD64
    SUSE Linux Enterprise Server 11 (SLES11)  i386 |  AMD64
    SUSE Linux Enterprise Server 10 (SLES10)  i386 |  AMD64
    Fedora 18 ("Spherical Cow") / Fedora 19 ("Schrödingers Cat")  i386 |  AMD64
    Fedora 17 ("Beefy Miracle")  i386 |  AMD64
    Mandriva 2011.0  i386 |  AMD64
    Oracle Linux 6 ("OL6") / Red Hat Enterprise Linux 6 ("RHEL6") / CentOS 6  i386 |  AMD64
    Oracle Linux 5 ("OL5") / Red Hat Enterprise Linux 5 ("RHEL5") / CentOS 5  i386 |  AMD64
    All distributions  i386 | AMD64

Although the catch all "All distributions" suggest it might work with
newer distros, they are clearly a second tier from a support POV. All
these explicitly targetted distros are outside libvirt's support policy.

There is a question as to whether to delete 5.0 and 5.1 support from
libvirt.

There are no remainining #ifdef VBOX_API_VERSION conditionals in the
vbox_tmpl.h file related to code differences between 5.0, 5.1 and 5.2,
so keeping them is not a maint burden in code.

The main burden is distributing the large vbox_CAPI_v5_0.h and
vbox_CAPI_v5_1.h files which are ~1 MB in size (uncompressed)
or ~85 kb (xz compressed). IOW versions 5.0 and 5.1 increase
the release tarballs by a small amount, but nothing that is
too unreasonable vs the overall size of 14 MB for libvirt source.

Thus I have kept 5.0 and 5.1 support.

VirtualBox 6.0 came out in Dec 2018 and given past practice it
would not suprise me if that has API changes impacting libvirt.
So some interested party might want to look at supporting that
in libvirt.

In v2:

 - Also remove deleted files from the Makefile
 - Stop including the old deleted 4.0 API file

Daniel P. Berrangé (7):
  vbox: drop support for VirtualBox 4.x releases
  vbox: drop C API definition for release 4.0
  vbox: drop C API definition for release 4.1
  vbox: drop C API definition for release 4.2
  vbox: drop C API definition for release 4.2.20
  vbox: drop C API definition for release 4.3
  vbox: drop C API definition for release 4.3.4

 src/vbox/Makefile.inc.am      |    12 -
 src/vbox/vbox_CAPI_v4_0.h     |  7451 -----------------------
 src/vbox/vbox_CAPI_v4_1.h     |  7882 ------------------------
 src/vbox/vbox_CAPI_v4_2.h     |  8855 ---------------------------
 src/vbox/vbox_CAPI_v4_2_20.h  |  9001 ----------------------------
 src/vbox/vbox_CAPI_v4_3.h     | 10210 -------------------------------
 src/vbox/vbox_CAPI_v4_3_4.h   | 10321 --------------------------------
 src/vbox/vbox_V4_0.c          |    37 -
 src/vbox/vbox_V4_1.c          |    37 -
 src/vbox/vbox_V4_2.c          |    13 -
 src/vbox/vbox_V4_2_20.c       |    13 -
 src/vbox/vbox_V4_3.c          |    13 -
 src/vbox/vbox_V4_3_4.c        |    13 -
 src/vbox/vbox_XPCOMCGlue.h    |     2 +-
 src/vbox/vbox_common.h        |    14 +-
 src/vbox/vbox_storage.c       |    14 +-
 src/vbox/vbox_tmpl.c          |   218 +-
 src/vbox/vbox_uniformed_api.h |    10 +-
 18 files changed, 8 insertions(+), 54108 deletions(-)
 delete mode 100644 src/vbox/vbox_CAPI_v4_0.h
 delete mode 100644 src/vbox/vbox_CAPI_v4_1.h
 delete mode 100644 src/vbox/vbox_CAPI_v4_2.h
 delete mode 100644 src/vbox/vbox_CAPI_v4_2_20.h
 delete mode 100644 src/vbox/vbox_CAPI_v4_3.h
 delete mode 100644 src/vbox/vbox_CAPI_v4_3_4.h
 delete mode 100644 src/vbox/vbox_V4_0.c
 delete mode 100644 src/vbox/vbox_V4_1.c
 delete mode 100644 src/vbox/vbox_V4_2.c
 delete mode 100644 src/vbox/vbox_V4_2_20.c
 delete mode 100644 src/vbox/vbox_V4_3.c
 delete mode 100644 src/vbox/vbox_V4_3_4.c

-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v2 0/7] vbox: drop support for many old versions
Posted by Andrea Bolognani 5 years ago
On Fri, 2019-04-12 at 14:00 +0100, Daniel P. Berrangé wrote:
> Daniel P. Berrangé (7):
>   vbox: drop support for VirtualBox 4.x releases
>   vbox: drop C API definition for release 4.0
>   vbox: drop C API definition for release 4.1
>   vbox: drop C API definition for release 4.2
>   vbox: drop C API definition for release 4.2.20
>   vbox: drop C API definition for release 4.3
>   vbox: drop C API definition for release 4.3.4

I have already provided comments for 1/7 separately.

For the rest of the series,

  Reviewed-by: Andrea Bolognani <abologna@redhat.com>

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v2 0/7] vbox: drop support for many old versions
Posted by Andrea Bolognani 5 years ago
On Mon, 2019-04-15 at 18:06 +0200, Andrea Bolognani wrote:
> On Fri, 2019-04-12 at 14:00 +0100, Daniel P. Berrangé wrote:
> > Daniel P. Berrangé (7):
> >   vbox: drop support for VirtualBox 4.x releases
> >   vbox: drop C API definition for release 4.0
> >   vbox: drop C API definition for release 4.1
> >   vbox: drop C API definition for release 4.2
> >   vbox: drop C API definition for release 4.2.20
> >   vbox: drop C API definition for release 4.3
> >   vbox: drop C API definition for release 4.3.4
> 
> I have already provided comments for 1/7 separately.
> 
> For the rest of the series,
> 
>   Reviewed-by: Andrea Bolognani <abologna@redhat.com>

I see you pushed the series already. Can you please update the
release notes to mention the change?

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list