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

Correct FreeBSD Python package name #688

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rtprio
Copy link

@rtprio rtprio commented Feb 13, 2024

Pull Request (PR) description

When trying to install Puppetboard I ran into this module trying to install virtualenv (doesn't exist as a package, only a port) and py3.9-pip (correct name is py39-pip).

Replace out the . in the version name. I did not test this extensively, just enough to get it to compile and install the right package for puppetboard. The origination of of the module doesn't seem to allow an easy way to get a default version based on OS and pass it down to python, pip, etc.

Feedback welcome.

@smortex
Copy link
Member

smortex commented Feb 15, 2024

Regarding the version, I never hit that issue. It happen I use this in my python profile:

class profile::python {
  class { 'python':
    version => '39',
    dev     => 'present',
  }
}

The data type for the version parameter accept a large number of formats, maybe too much.

The other change seems unrelated and break CI. Maybe worth a different PR?

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.

2 participants