Update README for Mac OS X

[ghost]

The README.md file omitted instructions for how to install the Aspell library for Mac OS X users.

[yorickpeterse]

Is there a particular reason why the installation command is mentioned both under "Requirements" and "Usage"? Personally I think it's better to have a separate section in the README (under "Requirements") that lists the installation commands for Aspell. Something like the following:

## Installing Aspell

* Arch Linux: `sudo pacman -S aspell`
* OS X: `brew install aspell --lang=en`

The way it's currently done in this patch is a bit confusing.

[ghost]

Hi, I wasn't sure about the proper convention. I agree that it is better in just the requirements section.

[ghost]

Should I submit another pull request with your suggestion? This is my first
pull request, and it would be awesome if I actually contributed something
that would be meaningful enough to be approved.

On Wed, Feb 27, 2013 at 4:24 PM, Yorick Peterse notifications@github.comwrote:

Is there a particular reason why the installation command is mentioned
both under "Requirements" and "Usage"? Personally I think it's better to
have a separate section in the README (under "Requirements") that lists the
installation commands for Aspell. Something like the following:
Installing Aspell

• Arch Linux: sudo pacman -S aspell
• OS X: brew install aspell --lang=en

The way it's currently done in this patch is a bit confusing.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-14200868
.

[yorickpeterse]

You can just use this pull request for it, there's also no need to rebase/reset anything :)

[yorickpeterse]

Why is the note "Dutch and Greek language packs for Aspell (only when testing the code)" under "Installing Aspell"? It's a requirement, not an installation step.

[ghost]

I initially thought that having all information regarding Aspell
installation in one section was correct. I now see how this presumption is
incorrect. I just moved the bullet point on language packs from Installing
Aspell back to Requirements.

On Sun, Mar 3, 2013 at 1:50 PM, Yorick Peterse notifications@github.comwrote:

Why is the note "Dutch and Greek language packs for Aspell (only when
testing the code)" under "Installing Aspell"? It's a requirement, not an
installation step.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/5#issuecomment-14352258
.

[yorickpeterse]

Thanks!