inrin/cdist
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix circular dep for CDIST_ORDER_DEPENDENCY

Browse source 
Fixes #756
This commit is contained in:
Darko Poljak 2019-04-14 16:54:59 +02:00
parent d18584b4ff
commit 797522f91e
1 changed files with 23 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
cdist/emulator.py
View file

@ -283,7 +283,8 @@ class Emulator(object):
self.object_source))) self.object_source)))
raise raise
self.log.debug("Recording requirement: %s", requirement) self.log.debug("Recording requirement %s for %s",
requirement, self.cdist_object.name)
# Save the sanitised version, not the user supplied one # Save the sanitised version, not the user supplied one
# (__file//bar => __file/bar) # (__file//bar => __file/bar)
@ -305,13 +306,26 @@ class Emulator(object):
# get the type created before this one ... # get the type created before this one ...
try: try:
lastcreatedtype = typecreationorder[-2].strip() lastcreatedtype = typecreationorder[-2].strip()
if 'require' in self.env: # __object_name is the name of the object whose type
self.env['require'] += " " + lastcreatedtype # manifest is currently executed
__object_name = self.env.get('__object_name', None)
if lastcreatedtype == __object_name:
self.log.debug(("Not injecting require for "
"CDIST_ORDER_DEPENDENCY: %s for %s,"
" %s's type manifest is currently"
" being executed"),
lastcreatedtype,
self.cdist_object.name,
lastcreatedtype)
else: else:
self.env['require'] = lastcreatedtype if 'require' in self.env:
self.log.debug(("Injecting require for " self.env['require'] += " " + lastcreatedtype
"CDIST_ORDER_DEPENDENCY: %s for %s"), else:
lastcreatedtype, self.cdist_object.name) self.env['require'] = lastcreatedtype
self.log.debug(("Injecting require for "
"CDIST_ORDER_DEPENDENCY: %s for %s"),
lastcreatedtype,
self.cdist_object.name)
except IndexError: except IndexError:
# if no second last line, we are on the first type, # if no second last line, we are on the first type,
# so do not set a requirement # so do not set a requirement
@ -360,4 +374,6 @@ class Emulator(object):
# But only if the user hasn't said otherwise. # But only if the user hasn't said otherwise.
# Must prevent circular dependencies. # Must prevent circular dependencies.
if parent.name not in current_object.requirements: if parent.name not in current_object.requirements:
self.log.debug("Recording autorequirement %s for %s",
current_object.name, parent.name)
parent.autorequire.append(current_object.name) parent.autorequire.append(current_object.name)