Bug #2995

UTF-8 isn't needed so missing/faulty iconv shouldn't cause pson-feature failure

Added by Markus Roberts about 2 years ago. Updated about 2 years ago.

Status:Closed Start date:12/29/2009
Priority:Normal Due date:
Assignee:Jesse Wolfe % Done:

0%
Category:serialization
Target version:0.25.2
Affected Puppet version:0.25.1rc2 Branch:http://github.com/jes5199/puppet/tree/ticket/0.25.x/2995
Keywords:
Votes: 0

Description

We don’t actually depend on UTF-8 support so it’s absence should not abort the loading of the PSON feature.

History

#1
Updated by Markus Roberts about 2 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Branch set to http://github.com/MarkusQ/puppet/tree/ticket/0.25.x/2995

#2
Updated by James Turnbull about 2 years ago

  • Status changed from In Topic Branch Pending Review to Closed

Pushed in commit:7e2b1e91e6930364c4b246a03a3f1c70b40da7f0 in branch 0.25.x

#3
Updated by Markus Roberts about 2 years ago

  • Status changed from Closed to Code Insufficient
  • Assignee changed from Markus Roberts to Jesse Wolfe

Looks like we may have jumped the gun on this one, or there may be some additional complication.

#4
Updated by Jesse Wolfe about 2 years ago

  • Status changed from Code Insufficient to In Topic Branch Pending Review
  • Branch changed from http://github.com/MarkusQ/puppet/tree/ticket/0.25.x/2995 to http://github.com/jes5199/puppet/tree/ticket/0.25.x/2995

Markus’s patch exposed a dependency on library load order that was causing Puppet::Util::Log to raise an error. I’ve removed the dependency of Puppet::Type from Puppet::Util::Log.

#5
Updated by Markus Roberts about 2 years ago

  • Status changed from In Topic Branch Pending Review to Ready For Checkin

Mark Plaskin confirmed that this resolved the remaining issue.

#6
Updated by James Turnbull about 2 years ago

  • Status changed from Ready For Checkin to Closed

Pushed in commit:7e2e12be7827c2adb64a192b5b7176c7c541af44 in branch 0.25.x

Also available in: Atom PDF