Github user dragos commented on the pull request:
https://github.com/apache/spark/pull/10370#issuecomment-187137035
OK, I retract that. We need to use `shell=true`, so @jayv if you have that
character escape version, let's go that way.
The reason why it won't work is `extraLibraryPath`: we need to prepend
library path variables, like `LD_LIBRARY_PATH="path1:path2:$LD_LIBRARY_PATH"`.
We could add this binding in the environment that we're building, but without a
shell we can't refer to the old value of `LD_LIBRARY_PATH` (note that this has
to stay an unevaluated reference to `LD_LIBRARY_PATH` because we're preparing
the `CommandInfo` structure in the dispatcher, which is NOT the machine where
the command will be executed).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
