-
Notifications
You must be signed in to change notification settings - Fork 14
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
The system cannot find the file specified #44
Comments
hi @byrongibby i will have a look |
Thanks a lot! |
I reran things on an empty project
Produces the same error:
|
I'm using
on Windows 10. |
had a look. the riddley jar looks a bit weird to be honest. this is the list of files in the jar:
note that both I can look into trying to track down the exact cause of why mranderson chokes on this and find a workaround but I would argue that the uberjar should not look like this. Please check if you can have this fixed in riddley. |
Will do. Thank you for the time and effort. Would the following assessment be correct:
That is not to suggest that the mranderson team has any responsibility to fix the issue, I just want to ask if I am seeing this correctly? |
I have posted this issue to the riddley team: ztellman/riddley#31 |
well. I suppose you can formalise this like your two above points. but we are in a bit grey area in terms of who should fix what. I mean apparently the uberjar is working fine so I would not be surprised if this issue did not get much attention at riddley -- unless this causes problems for other tools which i am not aware of. that said I am happy to look into this further to see if there is a quick fix for this. we are also on the green pastures of open source here so you can do the same: I am also happy to review/accept pull requests if you come up with a fix ;) |
It's likely beyond my ability, but maybe that's a perfectly good reason to expand my ability. The the reason I need mranderson is not immediately necessary, so let me give it a go. |
I pushed a fix to a branch: https://github.com/benedekfazekas/mranderson/tree/fix-44 anyways before proceeding would be nice if you tested this.
|
Great! Will do. |
Everything is working as expected
And I am no longer getting package/version conflicts in MATLAB. This has been a huge help to me, thank you very much! I will continue to monitor the issue I posted on riddley and hopefully get the problem sorted out there with the aim of making fix-44 unnecessary in the future. EDIT: So my initial test was flawed, it is not working to hide the dependencies from MATLAB, but I guess the problem is on my side now... |
i keep this open as you still seem to have problems and as the fix is not yet incorporated into master or released. wonder why the hiding/shadowing is not working though... please refer to https://github.com/benedekfazekas/mranderson#usage part of the readme to figure out how to make sure you run your app/repl etc with the inlined deps. |
I made a few stupid mistakes, having resolved them things look like this, I am unable to run the repl or package the JAR without an error from lein:
My
|
can't you just inline |
Okay. On a fresh project As soon as I require
|
you need to clean and rerun inline deps when you edit core.clj |
Like so?
|
not sure you can use with-profile like this. can you pls retry in two separate step? inline first, uberjarring next |
Unfortunately the same result:
|
will have a look on my local |
Thanks, I will check on my Linux box as well. EDIT: It gives the same result... |
i had just a very short time to look at this and did not manage to make this work. BUT I did manage to pass the above error. This basically caused by this line in the project.clj: |
That's good news, looking forward to a solution. |
Due to benedekfazekas/mranderson#44 or similar issue the automatic inlining throws an exception so this was done by hand.
Due to benedekfazekas/mranderson#44 or similar issue the automatic inlining throws an exception so this was done by hand.
Hi
Could someone please help me with the following error:
My
project.clj
looks as follows:The text was updated successfully, but these errors were encountered: