summaryrefslogtreecommitdiffstats
path: root/lib/puppet/parser/compiler.rb
Commit message (Collapse)AuthorAgeFilesLines
* Fixing #2423 - no more strange dependency cyclesLuke Kanies2009-07-251-1/+5
| | | | | | | | | | | | | | | We were getting strange dependency cycles because our class structure mirrored our scope structure. We can't change the scope structure without switching from a dynamically scoped language to a lexically scoped language, which is too big of a change to make right now. Instead, I'm changing the resource graph so that all classes default to just having an edge to the 'main' graph. This will be a behaviour change for many, in that you were getting automatic dependencies from this old behaviour, but it will bring consistency. Signed-off-by: Luke Kanies <luke@madstop.com>
* Switching to LoadedCode from ASTSetLuke Kanies2009-07-051-5/+5
| | | | | | | | I also took the opportunity to clean up and simplify the interface to the parts of the parser that interact with this. Mostly it was method renames. Signed-off-by: Luke Kanies <luke@madstop.com>
* Switching to Indirected ActiveRecordLuke Kanies2009-04-221-37/+0
| | | | | | | | | | | This is mostly a configuration change, with some code getting removed. Also adding an extra require in Format; Puppet::Provider requires Puppet::Provider::Confiner, so the constant lookup is weird. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fix #1933 - Inconsistent resource evaluation order in subsequent evaluation runsBrice Figureau2009-02-141-9/+10
| | | | | | | | | | | | | While evaluating the AST, catalog vertices are not always ordered the same way on different run, leading to some tags (which should have been applied in evaluation order) to not be associated with some underlying resources. This changeset change all accesses to resources inside the compiler to always use an ordered (in evaluation order) list of added resources. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Renaming Puppet::Node::Catalog to Puppet::Resource::CatalogLuke Kanies2008-12-181-2/+2
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing an error message to be more clearLuke Kanies2008-05-011-1/+1
|
* Always duplicating resource defaults in the parser, so thatLuke Kanies2008-03-061-1/+0
| | | | | | stacked metaparameter values do not result in all resources that receive a given default also getting those stacked values.
* Fixing #1092 by no longer using the resource reference toLuke Kanies2008-02-281-3/+2
| | | | | | look resources up, which means there's no concern about not finding the resource, which is where the nil was coming from. We now just iterate over the vertices.
* Merge commit 'bartv/pending/fix-997' into 0.24.xLuke Kanies2008-02-121-2/+6
|
* Moving the ast node tests to rspec (which I could haveLuke Kanies2008-02-121-2/+3
| | | | | | | *sworn* I did this weekend). In the process, I fixed a couple of bugs related to differentiating between nodes and classes, and then cleaned up quite a few error messages.
* Removing the last remaining vestiges of GRATR --Luke Kanies2008-02-111-3/+3
| | | | removing the bangs from 'add_vertex!' and 'add_edge!'.
* Changing the name of the Compile class to Compiler,Luke Kanies2008-02-111-0/+467
since it's stupid to have a class named after a verb.