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

fix some cross-compile issues from linux, touch up README #67

Merged
merged 1 commit into from
May 13, 2015

Conversation

markrechler
Copy link
Contributor

Updated the README with some clearer instructions for grabbing releases vs dev and fixed mktemp in dist.sh. The linux version does not auto append random data to the dir so it will bomb.

@markrechler
Copy link
Contributor Author

@jehiah this is ready for review. Thanks.

[![Build Status](https://secure.travis-ci.org/bitly/statsdaemon.png)](http://travis-ci.org/bitly/statsdaemon)

Installing
==========

```bash
go get github.com/bitly/statsdaemon
You can download either pre-compiled binaries or the source of the latest [release](https://github.com/bitly/statsdaemon/releases).
Copy link
Member

Choose a reason for hiding this comment

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

For Installing can we list a 'Current Stable Release: vXXXX' section w/ links to the current release downloads below that, a block for older releases below that pointing to the github release page, and then have a building from source section similar to nsq.io

@jehiah
Copy link
Member

jehiah commented May 13, 2015

Want to update the version to 0.7 in this PR and start drafting release notes?

@markrechler
Copy link
Contributor Author

Adjusted and starting to draft notes, let me know if there are any other changes you can think of.

@jehiah jehiah mentioned this pull request May 13, 2015
markrechler added a commit that referenced this pull request May 13, 2015
fix some cross-compile issues from linux, touch up README
@markrechler markrechler merged commit fcbcfd0 into bitly:master May 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants