Issue #3398 has been updated by Luke Kanies. Status changed from Accepted to Ready for Testing Assigned to changed from Luke Kanies to Markus Roberts Branch set to luke/tickets/master/3398
I've got a branch up at tickets/master/3398 that does exactly this. It's mostly a code simplification, and should make any code that touches this area much easier. ---------------------------------------- Refactor #3398: File's 'checksum' attribute should be a parameter, not a property http://projects.reductivelabs.com/issues/3398 Author: Luke Kanies Status: Ready for Testing Priority: Normal Assigned to: Markus Roberts Category: file Target version: Rowlf Affected version: 0.25.4 Branch: luke/tickets/master/3398 The 'checksum' attribute on 'file' has always been a property, rather than a parameter, and this fact has always resulted in a bit of torture around its relationship to 'source' and 'content'. We should switch this to a parameter whose sole role is to determine which checksum type we use. The 'content' property already actually speaks in checksums anyway, so we won't have to deal with the full file content anywhere. Note that this is a punt on tracking changes - we'll be removing all of the functionality that originally provided this just for file content. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://reductivelabs.com/redmine/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
