-
Notifications
You must be signed in to change notification settings - Fork 26
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
zsh issues #50
Comments
Ok, I might take a look sometime.... seems like a reasonable request, and I think I can install zsh on ubuntu, afaik. |
Hi. Ok, I installed zsh, and tried running cltorch, and so far no issues. See below. Can you produce a tiny test-case that demonstrates the issue please?
|
Yes I do, a minimum reproducible case is like this (on OS X 10.11 El Capitan) with
with
So precisely speaking, this is a |
Ah, I dont have Mac OS X ;-) Is this something you might be able to help fix? |
Surely I would love to if I know how, hint? |
Obviously if I knew how, I would have fixed it ;-) But I would imagine it's something to do with env vars. But as to what and how it's hard to say. Things I would try:
If that throws up nothing, you're going to have to start loading stuff, and hack around a bit. If it was me, I'd start writing little c programs probalby, to load stuff from c, and find out what works, and what doesnt. Here's an example I was using to try to fix Mac problems in cltorch yesterday:
Obviously you'll need to modify the names of the lilbraries being loaded and stuff. And you might comment out hte first bit, that loads libPyTorchLua.so and libpaths, or not. Or modify it. Etc. To build it, i would think it's something like:
... and that might be all you need. Oh.. this version actually hard-links with
( Edit: Oh, you'll need those lua headers... which means ... ummm.... you probably need to link with lua library somehow. That bit is always a bit tricky... and so... at this point... you'd have to start thinking about what is failing and where really.... ummm.... |
How about start by trying to load If that works (or if it doesnt), then try maybe loading the lua library, initializing lua ( I cant tell you an exact recipe. Something I often do is, hack around in the lua library itself, build myself. For example, loaindg the library is done by
It might need a certain amount of effort, and time :-P |
So, I added a call to zsh to my travis script https://travis-ci.org/hughperkins/cltorch/builds/112530954#L1268 , which runs edit: sorry, I mean against https://github.com/hughperkins/cltorch/blob/master/src/test/test-zsh.zsh |
Just did a quick comparison in both zsh and bash, see results below. Interestingly
More interestingly, I tried to manually export |
I coped everything from |
Seems it has something to do with What I don't understand is why |
I think you should have an LD_LIBRARY_PATH. On my system:
|
Yep you are correct, both
|
For SIP, what is the recommended approach, instead of using LD_LIBRARY_PATH? |
What happens if you copy everything from |
seems it is likely an Relevant references: http://linuxmafia.com/faq/Admin/ld-lib-path.html https://blogs.oracle.com/ali/entry/avoiding_ld_library_path_the |
Interestingly, if I clear my LD_LIBRARY_PATH, and move my build directoires, everything continues to run:
|
My RPATHs look like:
Per bottom of https://cmake.org/Wiki/CMake_RPATH_handling , yo uwould need to use |
Hahaha, copying everything from Ended up I just symlink it with |
Ok, that's interesting. Not sure that is the most sustainable solution, but good that it is working :-) |
Absolutely! |
Seems a bunch of people had ran into
zsh
not working properly withcltorch
, see #31 and #24, and for now the only solution is switching back tobash
which is less than ideal, just wondering has anyone figured out a solution to this? Thanks!The text was updated successfully, but these errors were encountered: