I think it would be fine to put up the PR as-is, since FreeBSD 9 is EOL.

Thanks for contributing,
Branan Riley
Software Engineer, Puppet

On Thu, May 4, 2017 at 4:12 PM <[email protected]> wrote:

> Hello,
>
> Today I think I found a bug in the ports provider for FreeBSD:
> puppet/lib/puppet/provider/package/ports.rb:7:
>
> :portinfo => "/usr/sbin/pkg_info"
>
> The pkg_info command has been replaced with 'pkg info' as of freebsd 10.0 :
> https://wiki.freebsd.org/PkgPrimer
>
> I've made a simple patch for the issue in ports.rb and freebsd.rb, which
> appears to fix the problem, but would likely break older releases than 10.0
> which use the older pkg_info command.  Can I submit a PR as is or do I need
> to account for older releases?  10.3 is the current oldest supported
> freebsd release.
>
> Thanks!
> Richard
>
> --
> You received this message because you are subscribed to the Google Groups
> "Puppet Developers" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/puppet-dev/1c21454b-7e7e-42a5-b165-429a2ed34237%40googlegroups.com
> <https://groups.google.com/d/msgid/puppet-dev/1c21454b-7e7e-42a5-b165-429a2ed34237%40googlegroups.com?utm_medium=email&utm_source=footer>
> .
> For more options, visit https://groups.google.com/d/optout.
>

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/puppet-dev/CADWDnrmezxM9em6mrV22MXeu54NmQ09hu9GPNgw4E0bTxygELg%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to