Puppet: System Administration Automated

Support

Ticket #1187 (closed defect: fixed)

Opened 8 months ago

Last modified 7 months ago

FreeBSD's pkg_add requires -r and $PACKAGESITE in openbsd package provider

Reported by: trombik Assigned to: luke
Priority: normal Milestone:
Component: library Version: 0.24.4
Severity: normal Keywords: FreeBSD package PACKAGESITE pkg_add
Cc: Triage Stage: Ready for checkin
Attached Patches: Code Complexity: Easy

Description

With this patch, the following code works as expected. 

package { "foo":
  source => "ftp://example.org/pub/FreeBSD/ports/i386/packages-6-stable/Latest/",
  ensure => installed,
  provider => "freebsd"
}

Attachments

patch-lib-puppet-provider-package-freebsd (1.0 kB) - added by trombik on 04/15/08 10:13:44.

Change History

04/11/08 08:05:24 changed by trombik

the revised one also works with "source => /usr/ports/packages/Latest/".

04/12/08 00:40:40 changed by jamtur01

  • keywords changed from FreeBSD to FreeBSD package PACKAGESITE pkg_add.
  • owner changed from community to jamtur01.
  • complexity changed from Unknown to Easy.
  • status changed from new to assigned.
  • stage changed from Unreviewed to Needs more info.

I note you're limiting this to FreeBSD? Does the same behaviour work on Net* and Open*? Or is this simply a FreeBSD requirement?

04/15/08 10:12:49 changed by trombik

  • patch changed from None to Code.

OpenBSD's pkg_add has different meaning of -r option. I don't know at all about NetBSD's.

It is found that FreeBSD's pkg_add requires "-r" even if PKG_PATH contains remote URI. Accutually, pkg_add with -r ignores PKG_PATH.

04/15/08 10:13:44 changed by trombik

  • attachment patch-lib-puppet-provider-package-freebsd added.

04/15/08 10:15:22 changed by trombik

this is FreeBSD specific, so freebsd.rb is the right place to patch.

04/17/08 07:25:55 changed by jamtur01

  • status changed from assigned to new.
  • stage changed from Needs more info to Accepted.

04/19/08 13:14:20 changed by jamtur01

  • owner changed from jamtur01 to luke.
  • stage changed from Accepted to Ready for checkin.

Pushed in commit [b5640a186d0e7103d5d7f5cdfd309342a1634de3] in branch 0.24.x

04/20/08 06:31:50 changed by luke

  • status changed from new to closed.
  • resolution set to fixed.

Merged and pushed.