Skip to content

spells out macOS instructions #53

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

AndrewSB
Copy link

@AndrewSB AndrewSB commented Mar 1, 2021

took me 10 minutes to figure out why cabal configure wasn't putting the binary into ~/.cabal/bin. putting these instructions here to make it clear to people who haven't used cabal before

took me 10 minutes to figure out why `cabal configure` wasn't putting the binary into `~/.cabal/bin`. putting these instructions here to make it clear to people who haven't used cabal before
using <a href="http://brew.sh">Homebrew</a> or <a href="https://www.macports.org">MacPorts</a>. For Homebrew, you can execute:</p>
<code class="command">brew install pkgconfig</code><br>
<code class="command">brew install pcre</code><br>
<p>or for MacPorts:</p>
<code class="command">sudo port install pkgconfig</code><br>
<code class="command">sudo port install pcre</code><br>
<p>Then, you can compile arbtt using the usual cabal commands.</p>
<code class="command">cabal configure; cabal build; cabal install</code><br>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It may be misleading to present this as the single method for installing arbtt using Homebrew. The cabal install arbtt command mentioned above in the "From Hackage:" block also works on macOS. Here's what I do:

brew install pkg-config pcre cabal-install
cabal update
cabal install arbtt

I realize your change matches the existing description, so it's not at all incorrect — but if you'd like, we could use this opportunity to also mention the simpler install-from-hackage method as an option in macOS.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The exisitng instructions are pretty-outdates, and not relaly modern. @waldyrious ’s instructions are better… would you care to put yours into a PR?

Copy link
Author

@AndrewSB AndrewSB Mar 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@waldyrious that's actually what i did as well, i didn't realize installing implicitly configured and built. I can update my PR to read like that. should i also remove the MacPorts instructions, or leave them up?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a macports user so I can't comment on whether those instructions work or not. I'd rather keep them and let people raise an issue if there's a problem, than remove potentially working instructions just because. But that's just my opinion :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants