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

Fixes Issue #163: Modification for DNF package manager #164

Open
wants to merge 1 commit into
base: master
from

Conversation

@LalatenduMohanty
Copy link

LalatenduMohanty commented Nov 15, 2015

Dnf is the new package manager for Fedora.
And it is the default package manager for F22 and F23

Signed-off-by: Lalatendu Mohanty lmohanty@redhat.com

Dnf is the new package manager for Fedora.
And it is the default package manager for F22 and F23

Signed-off-by: Lalatendu Mohanty <lmohanty@redhat.com>
@LalatenduMohanty LalatenduMohanty changed the title Fixes Issue #163: Modification for DNF package manager [Do Not Merge] Fixes Issue #163: Modification for DNF package manager Nov 15, 2015
@LalatenduMohanty LalatenduMohanty changed the title [Do Not Merge] Fixes Issue #163: Modification for DNF package manager Fixes Issue #163: Modification for DNF package manager Nov 15, 2015
@fgrehm

This comment has been minimized.

Copy link
Owner

fgrehm commented Nov 20, 2015

Sorry but as per GH-143 I'm looking for help to maintain this plugin and for now I'm holding back from bringing in changes to the code =/

If you are interested on maintaining the project and keep things moving please shoot me an email or drop a message on GH-143 so I can arrange things.

Cheers 🍻

@athak

This comment has been minimized.

Copy link

athak commented on 081c809 Dec 1, 2015

Hi,
I tried your patch and I think you need to disable yum_cache_dir on fedora. redhat capability is determined only by the existance of /etc/redhat-release, so fedora hosts have it. Without disabling the yum_cache_dir it still errors. I added:

guest_capability 'fedora', 'yum_cache_dir' do
  # Disable Yum on fedora guests
end

and it works fine.

Cheers,
Atha

@LalatenduMohanty

This comment has been minimized.

Copy link
Author

LalatenduMohanty commented Dec 1, 2015

@athak thanks. I also figured that out. I am working a new patch which will address things in a better way I think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.