-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
Make sed do -i '' on OSX #258
Conversation
Merged build triggered. One or more automated tests failed |
Merged build started. One or more automated tests failed |
Jenkins, test this please |
Merged build triggered. One or more automated tests failed |
@nicklan I actually see some other errors due to pushd and popd now:
Haven't looked into it much, see if you can get someone with a Mac to work with. |
Merged build started. One or more automated tests failed |
@mateiz huh, that's odd, it suggests that |
Merged build finished. All automated tests passed. |
All automated tests passed. |
@mateiz okay, I think i figured out the mktemp problem. if you could test it once more that would be great. i'll also try and test it on a mac later today if you're too busy at the moment. |
Merged build triggered. One or more automated tests failed |
Merged build started. One or more automated tests failed |
Merged build finished. One or more automated tests failed |
One or more automated tests failed |
Merged build finished. One or more automated tests failed |
One or more automated tests failed |
So it seems that on OS X, mktemp needs a template: https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man1/mktemp.1.html. Just doing The pushd/popd errors are because mktemp failed, but that was earlier in the log. |
Yeah, have you tried with my latests PR where I added the -t? On Thu, Mar 27, 2014 at 6:08 PM, Matei Zaharia notifications@git.luolix.topwrote:
|
Actually nm, saw you fixed it. This fixes make-distribution.sh, although unfortunately Tachyon as packaged here doesn't properly deploy on OS X. If I go into
However we can fix this in another JIRA, so I'll merge this PR as is. |
(BTW if I run this twice, the "sed" error goes away but I still get "Error: Could not find or load main class tachyon.Format"). |
Oh one other thing, make an account on our JIRA and I can assign https://spark-project.atlassian.net/browse/SPARK-1326 to you. |
Okay will do, and I can look at these other errors tomorrow
|
Thanks, I've now merged this. |
I don't have access to an OSX machine, so if someone could test this that would be great. Author: Nick Lanham <nick@afternight.org> Closes #258 from nicklan/osx-sed-fix and squashes the following commits: a6f158f [Nick Lanham] Also make mktemp work on OSX 558fd6e [Nick Lanham] Make sed do -i '' on OSX (cherry picked from commit 632c322) Signed-off-by: Matei Zaharia <matei@databricks.com>
Some of this is unfortunately a bug in the tachyon scripts. I'll submit a On Thu, Mar 27, 2014 at 6:17 PM, Matei Zaharia notifications@git.luolix.topwrote:
|
I don't have access to an OSX machine, so if someone could test this that would be great. Author: Nick Lanham <nick@afternight.org> Closes apache#258 from nicklan/osx-sed-fix and squashes the following commits: a6f158f [Nick Lanham] Also make mktemp work on OSX 558fd6e [Nick Lanham] Make sed do -i '' on OSX
* Add parameter for driver pod name * Mark KUBERNETES_DRIVER_POD_NAME not being internal. Update docment. * Add test case for driver pod name * Diff driver pod name with appid * replace 'spark.kubernetes.driver.pod.name` with KUBERNETES_DRIVER_POD_NAME * Update readme to complete item
* Add parameter for driver pod name * Mark KUBERNETES_DRIVER_POD_NAME not being internal. Update docment. * Add test case for driver pod name * Diff driver pod name with appid * replace 'spark.kubernetes.driver.pod.name` with KUBERNETES_DRIVER_POD_NAME * Update readme to complete item
[NOSQUASH] Resync kube
…d by users other than 'mapr' (apache#258)
I don't have access to an OSX machine, so if someone could test this that would be great.