Skip to content
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

fixed #132 - Update README.md to add more details about installation #133

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

Conversation

snowmantw
Copy link
Contributor

I updated the README.md about how I solved the issue.

@satoryu
Copy link
Member

satoryu commented Jun 4, 2016

You can ignore the failure of the build as we already known this issue and I'm trying to fix it #131 .

@snowmantw
Copy link
Contributor Author

@satoryu Oh okay. I also encountered some local testing errors and I'm tracing them, too. But the issue for this PR is I don't have permission to merge it (only see "Close and comment" and "Comment"). I mean, if you're talking about the CI build, not the build failure I addressed in this PR.

@satoryu
Copy link
Member

satoryu commented Jun 4, 2016

I can see 'Merge pull request' button 👍
I will try to reproduce your situation. after that I will merge this PR.

@@ -33,6 +33,27 @@ You can simply install ROMA and dependency libralies by using a "gems" command o
$ gem install roma
```

### Troubleshooting
Copy link
Member

Choose a reason for hiding this comment

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

I think this part should be a part of Installation and found it from another document. I tried to install roma from this repository like you into Ubuntu 14.04(trusty). lbibz2-dev is not installed in default. so installing libbz2-dev is required before installing roma. so it's not a trouble just a prerequisite for installing roma into Ubuntu.

I will merge your PR with the branch 'master' after update it. after that, I will write installation for tokyocabinet-ruby since it looks hard for visiters to this repository to be aware of another document site...

satoryu added a commit that referenced this pull request Jun 18, 2018
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.

None yet

3 participants