The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #15763

Remove deprecated methods on Providers

Added by Anonymous over 2 years ago. Updated over 2 years ago.

Status:ClosedStart date:07/31/2012
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:-
Target version:3.0.0
Affected Puppet version: Branch:https://github.com/puppetlabs/puppet/commit/5944580d3dd50b261e25f96c949d34bd4a9a0a19
Keywords:telly_deprecation

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

The make_command_methods and mkmodelmethods methods are deprecated and should be removed.

make_command_methods should be replaced with optional_commands and mkmodelmethods should be replaced with mk_resource_methods.


Related issues

Related to Puppet - Bug #15896: Remove deprecated `make_command_methods` method on Provid... Accepted 08/08/2012

History

#1 Updated by Anonymous over 2 years ago

  • Subject changed from Remove deprecated methods on Provider to Remove deprecated methods on Providers

#2 Updated by Anonymous over 2 years ago

I may be wrong, but from a bit of digging, it looks like make_command_methods was just deprecated in 3.x, but mkmodelmethods was deprecated before (in 2.7.x).

So it looks like make_command_methods should not be removed yet.

#3 Updated by Anonymous over 2 years ago

Made a new ticket (#15896) for the make_command_methods method since it’s not ready to be removed yet.

#4 Updated by Anonymous over 2 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Target version changed from 3.x to 3.0.0
  • Branch set to https://github.com/puppetlabs/puppet/pull/996

#5 Updated by Anonymous over 2 years ago

  • Assignee set to Anonymous

#6 Updated by Anonymous over 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

#7 Updated by Anonymous over 2 years ago

  • Branch changed from https://github.com/puppetlabs/puppet/pull/996 to https://github.com/puppetlabs/puppet/commit/5944580d3dd50b261e25f96c949d34bd4a9a0a19

Here’s the merge with all the commits:

https://github.com/puppetlabs/puppet/commit/f235aa8

#8 Updated by Matthaus Owens over 2 years ago

  • Status changed from Merged - Pending Release to Closed

Released in Puppet 3.0.0-rc4

Also available in: Atom PDF