Merge ~bryce/ubuntu/+source/open-vm-tools:mre-lp1998558-kinetic into ubuntu/+source/open-vm-tools:ubuntu/kinetic-devel

Proposed by Bryce Harrington
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merge reported by: Bryce Harrington
Merged at revision: d6f357cdb92f5b97c3ed5c5228977b2fdf840e62
Proposed branch: ~bryce/ubuntu/+source/open-vm-tools:mre-lp1998558-kinetic
Merge into: ubuntu/+source/open-vm-tools:ubuntu/kinetic-devel
Diff against target: 940 lines (+491/-72)
18 files modified
ReleaseNotes.md (+17/-22)
debian/changelog (+43/-0)
debian/control (+6/-3)
debian/open-vm-tools-desktop.lintian-overrides (+1/-1)
debian/patches/debian/grpc_1.51 (+13/-0)
debian/patches/series (+1/-0)
debian/rules (+0/-2)
open-vm-tools/ChangeLog (+99/-0)
open-vm-tools/configure.ac (+2/-2)
open-vm-tools/lib/include/buildNumber.h (+6/-6)
open-vm-tools/lib/include/conf.h (+6/-0)
open-vm-tools/lib/include/deployPkg/linuxDeployment.h (+20/-1)
open-vm-tools/lib/include/vm_tools_version.h (+14/-7)
open-vm-tools/libDeployPkg/linuxDeployment.c (+177/-20)
open-vm-tools/services/plugins/deployPkg/deployPkg.c (+43/-4)
open-vm-tools/services/plugins/vmbackup/nullProvider.c (+30/-1)
open-vm-tools/services/vmtoolsd/mainPosix.c (+1/-3)
open-vm-tools/tools.conf (+12/-0)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Andreas Hasenack (community) Approve
Canonical Server Pending
Canonical Server Reporter Pending
Canonical Server packageset reviewers Pending
Review via email: mp+441524@code.staging.launchpad.net

Description of the change

This is the MRE for this cycle's update to open-vm-tools.

PPA: https://launchpad.net/~bryce/+archive/ubuntu/open-vm-tools-mre-lp1998558

Autopkgtests are running...
$ ppa tests https://launchpad.net/~bryce/+archive/ubuntu/open-vm-tools-mre-lp1998558

This is pretty straightforward, there was one modification I added to fix a build issues due to missing build-dep libabsl-dev.

I considered rolling in the fix for LP: #1999945, but that's just a cosmetic typo that doesn't seem to cause any problems and wouldn't be accepted as SRU, so I've left it and will tend to it in the next merge.

I verified the package builds and installs/uninstalls in LXC. The service doesn't run in LXC but I assume it'd need a full VM or better since it's a virtualization system itself.

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1, the jammy MP[1] has more comments, but this is essentially the same.

1. https://code.launchpad.net/~bryce/ubuntu/+source/open-vm-tools/+git/open-vm-tools/+merge/441525

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: bryce, ahasenack
Uploaders: bryce, ahasenack
MP auto-approved

review: Approve
Revision history for this message
Bryce Harrington (bryce) wrote :

Thanks for the review. Added the bug ref as discussed in the other MP, and uploaded:

Successfully signed dsc, buildinfo, changes files
Vcs-Git: https://git.launchpad.net/~bryce/ubuntu/+source/open-vm-tools
Vcs-Git-Commit: d6f357cdb92f5b97c3ed5c5228977b2fdf840e62
Vcs-Git-Ref: refs/heads/mre-lp1998558-kinetic
$ dput ubuntu ../open-vm-tools_12.1.5-3~ubuntu0.22.10.1_source.changes
gpg: ../open-vm-tools_12.1.5-3~ubuntu0.22.10.1_source.changes: Valid signature from E603B2578FB8F0FB
gpg: ../open-vm-tools_12.1.5-3~ubuntu0.22.10.1.dsc: Valid signature from E603B2578FB8F0FB
D: Setting host argument.
Checking signature on .changes
Checking signature on .dsc
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading open-vm-tools_12.1.5-3~ubuntu0.22.10.1.dsc: done.
  Uploading open-vm-tools_12.1.5-3~ubuntu0.22.10.1.debian.tar.xz: done.
  Uploading open-vm-tools_12.1.5-3~ubuntu0.22.10.1_source.buildinfo: done.
  Uploading open-vm-tools_12.1.5-3~ubuntu0.22.10.1_source.changes: done.
Successfully uploaded packages.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches