Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10525#issuecomment-168278782
Aha! It appears that there _is_ a portability issue in `sort`, or at least
in how we're using it in the dependency-checking script.
When I run `echo -e "breeze-macros\nbreeze_2.0" | sort` on my Mac, I get
```
breeze-macros
breeze_2.0
```
but I get the opposite ordering when running on AMPLab Jenknis.
On my mac, `sort --version` outputs `sort (GNU coreutils) 5.93`, whereas
AMP Jenkins has `sort (GNU coreutils) 8.4`. But it turns out that this isn't
the problem! It's a locale-dependent issue:
https://stackoverflow.com/questions/28881/why-doesnt-sort-sort-the-same-on-every-machine
We need to set `LC_ALL=C` in order to make things portable. I'm going to
open a hotfix PR to address these issues, then will retest this PR once my fix
is in.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
