You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/07/16 23:10:57 UTC

[GitHub] [incubator-mxnet] aaronmarkham opened a new pull request #15561: Julia path patch

aaronmarkham opened a new pull request #15561: Julia path patch
URL: https://github.com/apache/incubator-mxnet/pull/15561
 
 
   ## Description ##
   Another followup PR to fix the Julia site build and unblock website publishing...
   Since the website builds so many versions, it does so in their own folders inside `/mxnet/docs/build_version_doc/apache-mxnet/...` and this makes it difficult for Julia to find when it's expecting the binaries in `/work/mxnet/lib`. After digging through the Julia source and testing a bunch of things I found that `LD_LIBRARY_PATH` doesn't seem to do anything, but `MXNET_HOME` is what Julia is looking for. It then manually appends `lib`. 
   Unfortunately, according to the [MXNet environment variables](https://mxnet.incubator.apache.org/versions/master/faq/env_var.html) page, this same env var is used elsewhere to store models.
   
   Unblocking website publishing is my primary goal, not dealing with conflicting environment variables, so I'm patching this for now, but I think the Julia package should updated to use `MXNET_LIBRARY_PATH` or `LD_LIBRARY_PATH` instead. 
   
   My test run used a test CI pipeline that points to my fork:
   http://jenkins.mxnet-ci.amazon-ml.com/job/test-website-build/
   
   The "successful" run was here. It fails because there's no place to publish, which is fine. Everything else worked: http://jenkins.mxnet-ci.amazon-ml.com/job/test-website-build/3/console
   We might want to keep this test pipeline around... it took me awhile to remember how to create one for testing, and I definitely should have done this thorough test in the first place.
   
   It was able to find Julia, and it used the locally compiled binary based off of the target branch: this one.
   
   I'm hoping this is the last surprise that the CI environment is going to give me for getting Julia to publish.
   
   Previous PRs for the Julia docs publishing:
   https://github.com/apache/incubator-mxnet/pull/15554
   https://github.com/apache/incubator-mxnet/pull/15523
   https://github.com/apache/incubator-mxnet/pull/15454

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services