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

Add src/duct_hierarchy.edn to :resource-paths #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kwrooijen
Copy link
Contributor

Problem

When building an uberjar all duct_hierarchy.edn files are merged to create a full hierarchy. This works fine until you create an uberjar. Your main project's duct_hierarchy.edn is not included it the merged hierarchy.

Solution

Add src/duct_hierarchy.edn to the uberjar's profile resource-paths. This will make sure it is included when generating the uberjar.

@weavejester
Copy link
Collaborator

This seems like a bug in Leiningen if true. I'll see if I can reproduce this issue.

@kwrooijen
Copy link
Contributor Author

@weavejester My colleagues and I came across this issue again. I think it's because the default duct template has no :source-paths definition, and possibly defaults to ["src"] ? We added our own: :source-paths ["src/clj" "src/cljs"] and it broke. We fixed it after changing this to: ["src/clj" "src/cljs" "src"].

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

Successfully merging this pull request may close these issues.

2 participants