GiraffaFS / giraffa

Giraffa FileSystem (Slack: giraffa-fs.slack.com)
https://giraffa.ci.cloudbees.com
Apache License 2.0
17 stars 6 forks source link

Replace README.txt with README.md and add "Requirements" section. #191

Closed dsalychev closed 9 years ago

dsalychev commented 9 years ago

Compilation error with protoc version 2.6.1 Sure, I have to compile it with 2.5.0 version, but I think it's ok to mention it in README.txt.

weilintsaiWand commented 9 years ago

How about mentioning hbase-1.0.1 and hadoop-2.5.1?

Btw, I've update wiki page https://github.com/GiraffaFS/giraffa/wiki/Introduction-and-requirements-for-Giraffa https://github.com/GiraffaFS/giraffa/wiki/How-to-Setup,-Build,-and-Use-Giraffa

dsalychev commented 9 years ago

I think, they are hbase-1.0.0+ and hadoop-2.5.0+, aren't they?

shvachko commented 9 years ago

Good idea to update README. Let's say Java 1.7+. Because

shvachko commented 9 years ago

You also need to amend the commit message. It should be formatted as described in step 6 of https://github.com/GiraffaFS/giraffa/wiki/Contribution#sharing-code-changes Also pre-commit hook should help: https://github.com/GiraffaFS/giraffa/wiki/Contribution#configuring-pre-commit-hook

weilintsaiWand commented 9 years ago

Looks good. Only one quick question: I see README.txt is still existed, should we delete it ?

dsalychev commented 9 years ago

I removed it, just forgot to remove it within first commit.

shvachko commented 9 years ago

I just committed this. Congratulations Dmitry!

For the future. It is ok to put multiple commits in the same push request. They will be squashed into a single commit anyways.