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

Does not work when specifying path to the project root #20

Closed
Petrulak opened this issue Dec 12, 2017 · 3 comments
Closed

Does not work when specifying path to the project root #20

Petrulak opened this issue Dec 12, 2017 · 3 comments

Comments

@Petrulak
Copy link

It doesn not work if i provide a path (myProject/) to my Kotlin project.
The path in error below is incorrect, should be : /Users/mato/myProject/build/dependency_bubble_graph.html.
instead of
/Users/mato/build/dependency_bubble_graph.html.

Martins-MacBook-Pro:~ mato$sudo daggraph myProject/
Analyzing dagger components and modules..

? What kind of chart do you want to generate? Bubble chart

All done! The chart was saved in /Users/mato/build/dependency_bubble_graph.html. Opening..
The file /Users/mato/build/dependency_bubble_graph.html does not exist.
child_process.js:614
    throw err;
    ^

Error: Command failed: open /Users/mato/build/dependency_bubble_graph.html
The file /Users/mato/build/dependency_bubble_graph.html does not exist.

at checkExecSyncError (child_process.js:574:11)
at execSync (child_process.js:611:13)
at /usr/local/lib/node_modules/daggraph/src/router.js:94:5
at fs.js:1294:7
at FSReqWrap.oncomplete (fs.js:149:15)
@dvdciri
Copy link
Owner

dvdciri commented Dec 12, 2017

Thanks for the issue, apparently there is an issue with the relative path. Creating a PR with a fix right now.

dvdciri added a commit that referenced this issue Dec 12, 2017
@dvdciri
Copy link
Owner

dvdciri commented Dec 12, 2017

@Petrulak Can you pease download develop and check if this issue has been fixed correctly?
Or share more info about it

@dvdciri
Copy link
Owner

dvdciri commented Dec 12, 2017

Also, can you try the same scenario with a project that doesn't contain Kotlin as it is not supported at them moment. So, closing this as it won't work anyway with your example right now.

@dvdciri dvdciri closed this as completed Dec 12, 2017
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