Skip to content
This repository has been archived by the owner on Aug 4, 2022. It is now read-only.

Commit

Permalink
Bug 874210 - Fix pymake stack depth issues when many targets and prer…
Browse files Browse the repository at this point in the history
…equisites are present. r=gps
  • Loading branch information
zseil committed Nov 8, 2013
1 parent 05b1013 commit d386b4f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
9 changes: 0 additions & 9 deletions build/pymake/make.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,15 +17,6 @@

gc.disable()

# This is meant as a temporary workaround until issues with many targets
# and prerequisites is addressed. Bug 874210 tracks.
#
# The default recursion limit for CPython is 1000.
try:
sys.setrecursionlimit(10000)
except Exception:
print >>sys.stderr, 'Unable to increase Python recursion limit.'

pymake.command.main(sys.argv[1:], os.environ, os.getcwd(), cb=sys.exit)
pymake.process.ParallelContext.spin()
assert False, "Not reached"
3 changes: 2 additions & 1 deletion build/pymake/pymake/data.py
Original file line number Diff line number Diff line change
Expand Up @@ -780,7 +780,8 @@ def runnext(self):
return

self.currunning = True
self.rlist.pop(0).runcommands(self.indent, self.commandscb)
rule = self.rlist.pop(0)
self.makefile.context.defer(rule.runcommands, self.indent, self.commandscb)

def commandscb(self, error):
assert error in (True, False)
Expand Down

0 comments on commit d386b4f

Please sign in to comment.