On Oct 13, 2008, at 1:04 PM, Brice Figureau wrote:
>>
>
> Yes. I'm even amazed that it worked.
> The code was like this:
>
> class apt {
> concatenated_file {
>  "preferences":
>    ...before => File[apt_config]
> }
>
> later in the same module
>
> file {
>  "/etc/apt/apt.conf":
>    alias => apt_conf
>    before => Concatenated_file[preferences]
> }
>
> Maybe that was because of the alias, or the fact that it was  
> referencing
> a define?

It's got to either be the define, or something a bit fundamentally  
broken in the graphing.

Note that consistently tracking cycles is often confusing because it  
can appear to be working but then fails when the hashing algorithm  
starts producing a different value (and thus affects sort order for  
otherwise-unsorted resources).

Obviously this should have thrown an error, but considering that I  
wrote the entire graph library (drastically simplified and sped up  
from a different library), I wouldn't be all that surprised if there  
were cases it missed.  Disappointed, but not surprised. :/

>
>> I don't think the graphing has changed at all in 0.24.6, so I'm kinda
>> surprised there's a behaviour change there.
>
> Yes, I was surprised too. If I have time tomorrow, I'll put back a
> 0.24.5 (and the old code) to see if the error pops up.
>
> Also, something I forgot to mention, the 0.24.6-rc1 puppetmaster  
> reports
> its version as 0.24.5. I first thought my upgrade had failed, then I
> checked :-)


We only update the release number on release, mostly because gems suck  
(you can't have letters in release numbers) and my release system is  
stodgy (you can't release a version whose release number is already in  
the file).

-- 
The one thing more difficult than following a regimen is not imposing it
on others. -- Marcel Proust
---------------------------------------------------------------------
Luke Kanies | http://reductivelabs.com | http://madstop.com


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" 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-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to