Github user ryan-williams commented on the pull request:
https://github.com/apache/spark/pull/2859#issuecomment-59799405
Interesting. What do you mean by "both links"? The current links to
`building-spark.html`, as well as the `building-with-maven.html` links I've
submitted here? The former currently 404, so keeping them in the README if we
are going to the trouble of changing it doesn't make sense to me.
I see now that `README.md` is not up-to-date, but that was not at all
apparent when I was getting set up with Spark over the weekend :-\ Seems like
the README should be kept consistent with the source tree that it is committed
with, and that can be decoupled from coarser per-release website refreshes.
Could we add a couple commands explaining that `sbt` is blessed now, and
showing how to use it? Otherwise maybe the README should just be removed.
---
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
