[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#771715: pre-approval unblock: monitoring-plugins/2.1-3 (or 2.1.1-1)



Control: tags -1 moreinfo confirmed

On 2014-12-01 21:51, Jan Wagner wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please pre-approval unblock package monitoring-plugins.
> 
> I uploaded the version 2.1-2 which fixed some not unserious issues some
> days ago. Unfortunately this fixes was uncomplete.
> 
> Upstream this issues was addressed in the 'maint' branch[1].
> 
> This upload has 2 patches from upstream 'maint' branch fixing check_ntp
> regressions from 2.1-2 and one small bugfix for check_tcp help output.
> 

The proposed changes looks reasonable and I am ready to accept them via
unstable, provided that they are uploaded prior to Monday, the 8th of
December.

> There is also a upstream release 2.1.1 expected in the next days
> (hopefully tomorrow) including all but only the changed from the
> upstream 'maint' branch. Should I just try to grab this new release or
> stick with 2.1 upstream and just deliver the patches from this branch?
> Anyways this should be the same codebase (after applying patches) in
> both cases.
> 

Assuming it is basically just a version bump on top of these changes,
then it would make sense.  If it has other changes as well, we would
need to review those as well.  Keep in mind that the deadline for having
it reviewed and approved /is still the 5th of December/.
  So hopefully it will not be delayed considerably.  If you go with this
and it has additional changes (beyond simple version/documentation
changes), please attach the new debdiff for review.

> The full debdiff stat is:
> 
> [...]
> 
> Many thanks, Jan.
> [1] https://github.com/monitoring-plugins/monitoring-plugins/commits/maint
> 

Please remove the moreinfo tag when you have either uploaded the
approved changes OR provided a new debdiff for review.

Thanks,
~Niels


Reply to: