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

Pre-Approval for iftop 0.17-8



Hi release team,

I haven't heard back from you, yet, so I guess, it didn't caught your
attention due to a missleading subject ;)

I still think it would be a good idea to have at least the ipv6 and the
segfautl patches applied.  One could discuss about the disabled Shell
functionality, and if it makes it easiert for you, I'm willing to upload
a new iftop version with only the IPv6 and the segfault patch.

Best Regards,
  Alexander

* Alexander Reichle-Schmehl <tolimar@debian.org> [101018 13:07]:

> Overall, I would like to upload with the following changes:
> 
>  debian/patches/07-spelling-and-usage-text.patch    |   87 +
> Fixes some spelling error in the manpage
> 
>  debian/patches/08-segfault-duplicate-options.patch |   55 +
> Fixes #425488/#598376 a sefault when using the same command line
> argument twice.
> 
>  debian/patches/09-incomplete-hash-func.patch       |   51 
> Fixes #595169, which is actually only wishlist, but quite nasty.
> 
>  debian/patches/10-lladdr-inclusion-kfreebsd.patch  |   37 
> Fixes #598367, an usage problem on kfreebds-*
> 
>  debian/patches/11-implement-ipv6-support.patch     | 1120 +++++++++++++++++++++
> Is the missig IPv6 support.
> 
> All these patches have been reviewsed and accepted by upstream in the
> meantime.
> 
> The laste change I seek your approval would be the following:
>  iftop-0.17/debian/rules                            |    3 
>  debian/NEWS                                        |   11 
> 
> Here I disabled an quite unexpected feature of iftop:  It allows you to
> open a subshell.  As iftop is often used in combination with sudo,
> sysadmins might be suprised, to give other users full root by granting
> them access to iftop (granted; only if the don't configure sudo
> properly); but as it is only a minimal change, I hope you allow that,
> too.
> 
> 
> Just for completness, I also changed the following files:
>  debian/watch                                       |    4 
>  iftop-0.17/debian/changelog                        |   27 
>  iftop-0.17/debian/control                          |    2 
>  iftop-0.17/debian/patches/series                   |    5 
> 
> I've uploaded the complete debdiff between 0.17-6 (currently in Squeeze)
> and the proposed 0.17-8 upload at
> http://people.debian.org/~tolimar/tmp/iftop-debdiff_0.17-6_0.17-8.diff.

Attachment: signature.asc
Description: Digital signature


Reply to: