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

Update Debian Package #506

Merged
merged 3 commits into from
Jun 15, 2015
Merged

Conversation

joshughes
Copy link

It is nice to have the ability to create a debian package when your deploying statsd across your infrastructure. I found that the upstart script works but the inti.d script does not... so I removed it.

I also wanted to have an upstart job for the proxy and install the npm dependencies when the package is installed.

The dependencies and version were also update to be in sync with master.

@@ -7,7 +7,7 @@ Build-Depends: debhelper (>= 8.0.0)

Package: statsd
Architecture: all
Depends: nodejs (>= 0.6), adduser
Depends: nodejs (>= 0.8), adduser
Copy link
Member

Choose a reason for hiding this comment

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

can you bump this up to 0.10? We don"t support 0.8 anymore

@xofer xofer mentioned this pull request Jun 9, 2015
@tuxinaut
Copy link
Contributor

👍

@mrtazz
Copy link
Member

mrtazz commented Jun 15, 2015

thanks for the contribution!

mrtazz added a commit that referenced this pull request Jun 15, 2015
@mrtazz mrtazz merged commit 71be4e9 into statsd:master Jun 15, 2015
@roflmao
Copy link
Contributor

roflmao commented Jul 29, 2015

So this makes debian packages not actually have a working initscript since no debian use upstart by default? sysvinit is compat with systemd at least.

coykitten added a commit that referenced this pull request May 2, 2016
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.

5 participants