Re: [PATCH 0/2] firewall-manager update

[Thomas Woerner <twoerner redhat com>]

Hello Jiri,

On 02/10/2012 04:45 PM, Jiri Popelka wrote:
> The first patch updates FirewallD DBus service&  path&  interface.
> The second one adds calling of FirewallD1.zone.removeInterface method
> during device deactivation.
>
> Jiri Popelka (2):
>    firewall: update FirewallD DBus service&  path&  interface
>    firewall: add remove_from_zone()
>
>   src/firewall-manager/nm-firewall-manager.c |   80 ++++++++++++++++++++++-----
>   src/firewall-manager/nm-firewall-manager.h |   12 +++--
>   src/nm-device.c                            |   10 +++-
>   3 files changed, 82 insertions(+), 20 deletions(-)

that's great.

One thing is still needed: If an interface gets renamed, the new 
interface has to be added first and the old one removed afterwards.
Maybe there is a way to implement this in a simple way.

Thanks,
Thomas