Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

Review request for mesos, Benjamin Hindman and Till Toenshoff.


Repository: mesos


Description
---

See summary.


Diffs
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

(Updated June 15, 2015, 12:11 p.m.)


Review request for mesos, Benjamin Hindman and Till Toenshoff.


Changes
---

Fixed copy-paste error.


Repository: mesos


Description
---

See summary.


Diffs (updated)
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

(Updated June 15, 2015, 12:20 p.m.)


Review request for mesos, Benjamin Hindman and Till Toenshoff.


Changes
---

@Till mentioned that we don't need Python 3 for Mac OS X.


Repository: mesos


Description
---

See summary.


Diffs (updated)
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Till Toenshoff 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/#review87915
---

Ship it!



docs/getting-started.md


Not sure this makes sense for this chapter.



docs/getting-started.md


Also not yours but this does not seem to make any sense. OS X does not need 
Python3 to get installed just like the other distros dont need it. AFAIK we are 
fine with Python 2.7.x on all platforms. Could you please get rid of this part?



docs/getting-started.md


I know this isn't yours but it seems weird to mark C++, Java and Python as 
being code keywords, no?


- Till Toenshoff


On June 15, 2015, 12:20 p.m., Michael Park wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35455/
> ---
> 
> (Updated June 15, 2015, 12:20 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 
> 
> Diff: https://reviews.apache.org/r/35455/diff/
> 
> 
> Testing
> ---
> 
> Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 
> 6.6` and `CentOS 7.1` docker images.
> 
> 
> Thanks,
> 
> Michael Park
> 
>

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/#review87916
---


Patch looks great!

Reviews applied: [35455]

All tests passed.

- Mesos ReviewBot


On June 15, 2015, 12:20 p.m., Michael Park wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35455/
> ---
> 
> (Updated June 15, 2015, 12:20 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 
> 
> Diff: https://reviews.apache.org/r/35455/diff/
> 
> 
> Testing
> ---
> 
> Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 
> 6.6` and `CentOS 7.1` docker images.
> 
> 
> Thanks,
> 
> Michael Park
> 
>

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

(Updated June 15, 2015, 12:36 p.m.)


Review request for mesos, Benjamin Hindman and Till Toenshoff.


Changes
---

Addressed Till's comments.


Repository: mesos


Description
---

See summary.


Diffs (updated)
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

(Updated June 15, 2015, 12:42 p.m.)


Review request for mesos, Benjamin Hindman and Till Toenshoff.


Changes
---

Since we don't download the `maven` binary and un-`tar` it, `tar` is no longer 
a utility package we require.


Repository: mesos


Description
---

See summary.


Diffs (updated)
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

(Updated June 15, 2015, 12:54 p.m.)


Review request for mesos, Benjamin Hindman and Till Toenshoff.


Changes
---

Add `tar` back, since we actually still need it.


Repository: mesos


Description
---

See summary.


Diffs (updated)
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Michael Park 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/
---

(Updated June 15, 2015, 12:58 p.m.)


Review request for mesos, Benjamin Hindman and Till Toenshoff.


Changes
---

`s/#/$/`


Repository: mesos


Description
---

See summary.


Diffs (updated)
-

  docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 

Diff: https://reviews.apache.org/r/35455/diff/


Testing
---

Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 6.6` 
and `CentOS 7.1` docker images.


Thanks,

Michael Park

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35455/#review87922
---


Patch looks great!

Reviews applied: [35455]

All tests passed.

- Mesos ReviewBot


On June 15, 2015, 12:58 p.m., Michael Park wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35455/
> ---
> 
> (Updated June 15, 2015, 12:58 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> See summary.
> 
> 
> Diffs
> -
> 
>   docs/getting-started.md af679665434f38dc999f519b985b5ddb4f782ff8 
> 
> Diff: https://reviews.apache.org/r/35455/diff/
> 
> 
> Testing
> ---
> 
> Followed the instructions line by line with fresh `Ubuntu 14.04`, `CentOS 
> 6.6` and `CentOS 7.1` docker images.
> 
> 
> Thanks,
> 
> Michael Park
> 
>