Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

directory name with space in it #167

Open
slagyr opened this issue Jan 25, 2021 · 2 comments
Open

directory name with space in it #167

slagyr opened this issue Jan 25, 2021 · 2 comments

Comments

@slagyr
Copy link
Owner

slagyr commented Jan 25, 2021

lein spec -a

Changes to the core.clj file do not get reloaded and specs do not rerun. Removing the space in the directory name fixes the problem.

bowling katas.zip

@brandoncorrea
Copy link
Contributor

This looks like an issue with fresh.core line 42, ns-to-file, which is returning URL Encoded file paths.

(file (.getFile url))

One solution would be adding a URLDecoder. This would fix the encoded space and any other URL Encoding issues in directories, but would require a dependency on java.net.

(:import 
    ; ...
    [java.net URLDecoder])
; ...
(file (URLDecoder/decode (.getFile url) "UTF-8"))

@slagyr
Copy link
Owner Author

slagyr commented Dec 20, 2021

Thanks @brandoncorrea. The fresh library should be deprecated since clojure.tools.namespace.dir exists now. The correct fix would be to swap out the use of fresh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants