void-packages icon indicating copy to clipboard operation
void-packages copied to clipboard

New package: libguestfs-tools 1.48.0

Open chexum opened this issue 3 years ago • 5 comments

Testing the changes

  • I tested the changes in this PR: YES

New package

perl-Locale-TextDomain is only introduced for the script in perl.

libguestfs-tools is not technically now, some time ago it was part of libguestfs. Some tools were split into their own packages before libguestfs 1.45.2 and split into this package, see:

https://github.com/libguestfs/libguestfs/commit/733d2182b64df7abc5c5cd7d78177baa6079628c

Without this package, important tools like virt-sysprep were not provided at all.

The package is guestfs-tools but all other distributions align the package name with libguestfs.

Note that the test suite is very resource-intensive, needs to download 100+MB of images, and runs VMs for several of the tests for several minutes, which I don't think are reasonable for commit-time tests. They have been successful otherwise, apart from test-virt-sysprep-script.sh, which has been disabled.

Part of the test suite is downloading a test VM image, which is roughly the purpose of the update-libguestfs-appliance script in the base libguestfs package, however, it's not designed to run as a normal user in the chroot test pass.

(It also refers to an image which is no longer available, but I'm not sure what's the best approach to make it work, as it's only needed for the test suite, which is not nice at all to be run in a CI environment)

Perhaps to be coordinated with @Hoshpak (I don't insist staying the maintainer for these new entries, but don't mind it either)


Local build testing

  • I built this PR locally for my native architecture, (x86_64-glibc)
  • Could not test any other archs, will check the CI

chexum avatar Jun 05 '22 19:06 chexum

Just to reitarate, 22 of the packages documented on https://libguestfs.org/ are no longer actually part of the shipped void libguestfs package. libguestfs-tools may be "new" but all it contains is the remaining tools that were previously part of libguestfs.

chexum avatar Jun 09 '22 05:06 chexum

We generally use upstream names, anything wrong with using guestfs-tools?

Duncaen avatar Jun 27 '22 22:06 Duncaen

We generally use upstream names, anything wrong with using guestfs-tools?

Even the internal library common to libguestfs and guestfs-tools is residing in libguestfs-common. Perhaps the vendor name was chosen to only have a name that's shorter to type, but it's all still for the libguestfs family. Both the two major distributions use the name libguestfs-tools as well. This way it could reduce the confusion of both new users, to learn about the package, and users from other distributions, to find the package they know.

chexum avatar Jun 29 '22 16:06 chexum

Of course I'm not quite sure if the choice of the non-upstream package name libguestfs-tools is correct. However, when trying to find out where did virt-sysprep (for example) has gone from libguestfs, it took a nontrivial amount of time to see that it's now in a package that's not yet shipping. If void were to use the name established by the deb and rpm folks, it at least smooths the transition for people using documentation written with those distributions in mind. Perhaps surprisingly, this includes the upstream web site, so I have a fair point that this is actually the upstream name:

https://libguestfs.org/

sudo yum install libguestfs-tools      # Fedora/RHEL/CentOS
sudo apt-get install libguestfs-tools  # Debian/Ubuntu

Furthermore, it should also make it easier to find it by xbps-query -Rs libguestfs. Would any user think that they need to use xbps-query -Rs guestfs instead? If these points still don't make sense over the upstream naming rule, I'd be happy to oblige, and change the name.

chexum avatar Jul 03 '22 14:07 chexum

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

github-actions[bot] avatar Oct 02 '22 02:10 github-actions[bot]

Yes, I'd still like to have this merged - give me some time to review recent changes

chexum avatar Oct 09 '22 18:10 chexum

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

github-actions[bot] avatar Jan 09 '23 01:01 github-actions[bot]

still working on getting back on my feet and retesting this package, appreciate the patience

chexum avatar Jan 18 '23 18:01 chexum

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

github-actions[bot] avatar Apr 20 '23 01:04 github-actions[bot]

I ended up here looking for a package that might contain virt-filesystems, I guess there hasn't been too much demand for it (^^;

sogaiu avatar Jul 01 '24 04:07 sogaiu