search

cargomedia / debian-packages

Custom debian packages
8 stars 5 forks source link

Update janus and plugins #101

Closed ppp0 closed 8 years ago

ppp0 commented 8 years ago

@kris-lab this compiles without problems. Please review.

Old and new cfg file diffs: none

kris-lab commented 8 years ago

looks good. let's wait for plugins, update them with new janus and release all together!

tomaszdurka commented 8 years ago

@ppp0 please also update init script to consider pidfile when checking status of the service.

Replace

status_of_proc "$DAEMON_NAME" "$DAEMON" && exit 0 || exit $?

with

status_of_proc -p "$PIDFILE" "$DAEMON_NAME" "$DAEMON" && exit 0 || exit $?
njam commented 8 years ago

@ppp0 @kris-lab let's create a DEB with current versions

Waiting for https://github.com/cargomedia/janus-gateway-rtpbroadcast/issues/77

kris-lab commented 8 years ago

@ppp0 can we stick to this version 08888212c99f715061bcc833f47dd36aabb382ed?

See also:

ppp0 commented 8 years ago

@kris-lab please review

ppp0 commented 8 years ago

Updated init script according to https://github.com/cargomedia/debian-packages/pull/101#issuecomment-194231607

ppp0 commented 8 years ago

@kris-lab please review

kris-lab commented 8 years ago
kris-lab commented 8 years ago

lgtm! as discussed let's merge and deploy tomorrow.

ppp0 commented 8 years ago

@kris-lab thanks! May be double-check again?

Merging (and syncing with repo) will happen tomorrow during maintenance window for bulldogs