Markus Roberts wrote:
>> It is indeed deprecated... as of ActiveRecord 2.2.x This ticket was
>> filed for AR 2.1.x, and I was under the impression Puppet supported
>> that - I see now that the wiki (
>> http://reductivelabs.com/trac/puppet/wiki/UsingStoredConfiguration
>> ) says it requires at least 2.2.2 . If that's true, then let's
>> withdraw this patch and ask people to upgrade ActiveRecord.
>
> Don't forget to add a note to that effect to the ticket and change
> it from "Investigating" to "Rejected."

Is the wiki really authoritative here?  I thought that 2.1 was still
supported based on Luke's change in 1a5c5b3 (Fixing #2508 - removing
mention of ActiveRecord 2.3), which changed:

-                    raise "StoreConfigs not supported without ActiveRecord 
2.3" unless Puppet.features.rails?
+                    raise "StoreConfigs not supported without ActiveRecord 2.1 
or higher" unless Puppet.features.rails?

If the wiki is accurate and 0.24.8 requires <= 2.2.2 and 0.25.x
requires >= 2.2.2, it seems like it will be difficult to support both
Fedora and RHEL/CentOS, as Fedora is at 2.3.4 and EPEL is at 2.1.1.

For distribution packagers, asking folks to install via gem is not an
option.  And if EPEL can only support 0.24.8, updating Fedora to
0.25.1 gets fun, as many users likely have the puppetmaster on 
RHEL/CentOS.

Please forgive me if I'm misunderstanding.

-- 
Todd        OpenPGP -> KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
The most overlooked advantage to owning a computer is that if they
foul up, there's no law against whacking them around a little.
    -- Eric Porterfield

Attachment: pgpdTmR4okz0g.pgp
Description: PGP signature

Reply via email to