[TNS-17] doesn't track namespace when ns decl isn't first in file Created: 04/Feb/14 Updated: 19/Sep/14 Resolved: 11/Jul/14
|Reporter:||Trevor Wennblom||Assignee:||Stuart Sierra|
This was first filed for ns-tracker https://github.com/weavejester/ns-tracker/issues/15 but I understand it's more specific to tools.namespace.
(I realize this is likely a very low priority, but it'd still be nice to see fixed if possible. Thanks!)
untracked — (any of the four possibilities when uncommented, for example):
|Comment by Stuart Sierra [ 13/Jun/14 4:43 PM ]|
What's the use case for having a file in which the ns declaration isn't the first non-comment form?
|Comment by Trevor Wennblom [ 13/Jun/14 5:53 PM ]|
In the case that triggered the issue for me, it can be useful to see the load-order of files based on the dependencies given in ns for debugging. That is, to print a message before and after the completion of loading the namespace dependencies.
I'm not sure how difficult or involved the issue would be to solve.
|Comment by Andy Fingerhut [ 13/Jun/14 6:42 PM ]|
Would the :verbose option to require perhaps give you the load-order info you want? Try something like this at a REPL, with a namespace you care about:
In general, I think Stuart's concern would be questions like: How many forms would you expect tools.namespace to skip over before giving up? If it is a fixed number of forms before it gives up, why that number and not some other?
If it is based upon some other condition to give up looking for an ns form, what would that condition be?
Or would you expect it to be able to find the ns form in the body of a 'when' or 'if' form, and not at the top level?
The criteria can easily start to sound kind of arbitrary, and/or complex to implement.
|Comment by Stuart Sierra [ 11/Jul/14 5:04 PM ]|
Fixed on master as of commit 3c08b722. This turned out to be an easy change.
I wouldn't recommend putting anything before the ns declaration, but it's no longer required to be first. A file could also have multiple ns declarations, although clojure.tools.namespace.track will only look at the first one.
|Comment by Stuart Sierra [ 19/Sep/14 9:08 AM ]|
Included in release 0.2.6