Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove config unsupported by fog-libvirt #1835

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

janekzieleznicki
Copy link

fog-libvirt dropped support for libvirt < 1.2.8 in version v0.13.0 This config is no longer supported nor needed

This call generates a warning

Fixes #1831

fog-libvirt dropped support for libvirt < 1.2.8 in version v0.13.0
This config is no longer supported nor needed
@janekzieleznicki janekzieleznicki marked this pull request as ready for review November 18, 2024 18:19
Copy link

A docker image containing the code from this plugin to allow testing locally without installing
can be pulled from: ghcr.io/vagrant-libvirt/vagrant-libvirt:pr-1835-slim

If you need the image with the full dev toolchain, you can instead pull: ghcr.io/vagrant-libvirt/vagrant-libvirt:pr-1835

@abbbi abbbi enabled auto-merge (squash) January 5, 2025 15:23
@abbbi abbbi disabled auto-merge January 5, 2025 15:24
@abbbi abbbi enabled auto-merge January 5, 2025 15:24
@abbbi abbbi requested a review from electrofelix January 5, 2025 15:26
@abbbi
Copy link
Contributor

abbbi commented Jan 5, 2025

@electrofelix anything that speaks against merging this? Ty :-)

@@ -26,8 +26,7 @@ Gem::Specification.new do |s|
s.add_runtime_dependency 'xml-simple'
s.add_runtime_dependency 'diffy'

# Make sure to allow use of the same version as Vagrant by being less specific
s.add_runtime_dependency 'nokogiri', '~> 1.6'
s.add_runtime_dependency 'nokogiri', '< 1.16'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why pin this to < 1.16 while the latest version is 1.18?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logger dependency conflict with Vagrant 2.4.2
3 participants