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

Updated to Fedora 25 #725

Merged
merged 1 commit into from
Nov 30, 2016
Merged

Updated to Fedora 25 #725

merged 1 commit into from
Nov 30, 2016

Conversation

rickard-von-essen
Copy link
Collaborator

@rickard-von-essen rickard-von-essen commented Nov 22, 2016

Tested x86_64 and i386 builds on:

  • Parallels Desktop for Mac Pro 12.1.1
  • VMware Fusion 7.1.3
  • VirtualBox 5.1.6

Signed-off-by: Rickard von Essen <rickard.von.essen@gmail.com>
@rickard-von-essen
Copy link
Collaborator Author

I would like to drop yum support by default by removing the dnf-yum package. I think that should be installed by the ones that need it. Vagrant should work without yum.

@cheeseplus
Copy link
Contributor

cheeseplus commented Nov 28, 2016

Looks like we're removing yum from all platforms which I don't think should be the case. Can we do this just for 25+?

(ignore this comment, I had not had enough coffee)

@tas50
Copy link
Contributor

tas50 commented Nov 28, 2016

The reason for adding yum was that chef currently doesn't have a DNF provider for packages. That work is happening now, but without this packages won't install

@rickard-von-essen
Copy link
Collaborator Author

The reason for adding yum was that chef currently doesn't have a DNF provider for packages.

So let's keep it for now.

Looks like we're removing DNF from all platforms which I don't think should be the case.

Dnf have been the default since Fedora 22. So it should be possible to remove for all supported versions.

@rickard-von-essen
Copy link
Collaborator Author

rickard-von-essen commented Nov 29, 2016

And I haven't dropped yum yet. That requires removing dnf-yum from the kickstart file.

@cheeseplus cheeseplus added this to the 2.3.1 milestone Nov 30, 2016
@tas50 tas50 merged commit 191c899 into chef:master Nov 30, 2016
@tas50 tas50 removed the Developing label Nov 30, 2016
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.

4 participants