Skip to content

Commit

Permalink
Merge pull request statsd#331 from fjaeckel/debian-packaging-cleanup
Browse files Browse the repository at this point in the history
Debian packaging cleanup
  • Loading branch information
mrtazz committed Aug 21, 2013
2 parents 1f048ed + 49ac7ca commit 907838a
Show file tree
Hide file tree
Showing 5 changed files with 29 additions and 5 deletions.
15 changes: 14 additions & 1 deletion debian/changelog
Original file line number Diff line number Diff line change
@@ -1,7 +1,20 @@
statsd (0.6.0-1) unstable; urgency=low

* Non-maintainer upload.
* Fix lintian error messages regarding init script removal but not
installing them, using adduser but not defining the dependencies,
using /var/run/ in debian/dirs even though it can be a tmpfs, init script
not depending on $remote_fs, spelling mistakes
* removing depedency on ${misc:Depends} as this got only added as lintian
used to complain about not having it, it's not anymore
* Align version number to git tag versions

-- Frederic Jaeckel <[email protected]> Tue, 20 Aug 2013 14:03:10 +0200

statsd (0.0.6-1) unstable; urgency=low

* Update packaging for 0.0.6
* Bump nodejs dependancy to 0.6 per pcakage.json
* Bump nodejs dependency to 0.6 per package.json

-- Kiall Mac Innes <[email protected]> Thu, 27 Jun 2013 19:17:00 +0100

Expand Down
2 changes: 1 addition & 1 deletion debian/control
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ Build-Depends: debhelper (>= 8.0.0)

Package: statsd
Architecture: all
Depends: ${misc:Depends}, nodejs (>= 0.6)
Depends: nodejs (>= 0.6), adduser
Description: Stats aggregation daemon
A network daemon for aggregating statistics (counters and timers),
rolling them up, then sending them to graphite.
1 change: 0 additions & 1 deletion debian/dirs
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
var/log/statsd
var/run/statsd
5 changes: 5 additions & 0 deletions debian/postinst
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,11 @@
set -e

if [ "$1" = configure ]; then
# Automatically added by dh_installinit
if [ -x "/etc/init.d/statsd" ]; then
update-rc.d statsd defaults >/dev/null
invoke-rc.d statsd start || exit $?
fi

if ! getent passwd _statsd > /dev/null; then
adduser --system --quiet --home /nonexistent --no-create-home \
Expand Down
11 changes: 9 additions & 2 deletions debian/statsd.init
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
#! /bin/sh
### BEGIN INIT INFO
# Provides: statsd
# Required-Start: $network $local_fs
# Required-Stop:
# Required-Start: $remote_fs $network $local_fs
# Required-Stop: $remote_fs
# Default-Start: 2 3 4 5
# Default-Stop: 0 1 6
### END INIT INFO
Expand Down Expand Up @@ -41,6 +41,13 @@ CHDIR="/usr/share/statsd"
# Depend on lsb-base (>= 3.0-6) to ensure that this file is present.
. /lib/lsb/init-functions

# Create PIDDIR on runtime
if [ ! -d /var/run/$NAME ];
then
mkdir /var/run/$NAME
chown $USER /var/run/$NAME
fi

#
# Function that starts the daemon/service
#
Expand Down

0 comments on commit 907838a

Please sign in to comment.