Stack overflow on cyclic dependency

Description

If a cyclic dependency is introduced between a namespace and itself, it results in a StackOverflow when calculating transitive-dependencies.

I believe clojure.tools.namespace.dependency/transitive should maintain a set of seen namespaces and check if we have already seen the namespace at each step.

Environment

None

Activity

Show:
Stuart Sierra
January 10, 2014, 5:26 PM

Mark old resolved issues as 'closed'

Stuart Sierra
June 15, 2013, 4:33 PM

Already fixed in 0.2.4-SNAPSHOT by forbidding cyclic dependencies: https://github.com/clojure/tools.namespace/commit/85e73af04fdb497db1600ef3bf21aef5ed676619

Completed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Stuart Sierra

Reporter

Ambrose Bonnaire-Sergeant