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 at 5:26 PM

Mark old resolved issues as 'closed'

Stuart Sierra June 15, 2013 at 4:33 PM

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

Completed

Details

Assignee

Reporter

Priority

Created June 15, 2013 at 6:24 AM
Updated January 10, 2014 at 5:26 PM
Resolved January 10, 2014 at 5:26 PM