diff options
| author | Giampaolo Lauria <lauria2@yahoo.com> | 2011-07-15 12:07:03 -0400 |
|---|---|---|
| committer | Andrew Tridgell <tridge@samba.org> | 2011-07-21 10:32:24 +1000 |
| commit | f6fa8684896b8f3f9f8b7bd3742c99906973274c (patch) | |
| tree | 633d65aa65aac5ee2d0ae431f549f970c10f4aa7 /source4/scripting/python/samba/netcmd/fsmo.py | |
| parent | 1dfcb019d2dcca1d21ec32ee05bf7da15ca84e9f (diff) | |
| download | samba-f6fa8684896b8f3f9f8b7bd3742c99906973274c.tar.gz samba-f6fa8684896b8f3f9f8b7bd3742c99906973274c.tar.bz2 samba-f6fa8684896b8f3f9f8b7bd3742c99906973274c.zip | |
samba-tool: moved takes_optiongroups definition to Command base class
The option groups should be defined at the Command base class level as they are in common across all samba-tool commands.
Major move advantages:
1. more OOP approach
2. enforcing consistency across commands
3. avoiding the need of declaring for every new command
Signed-off-by: Andrew Tridgell <tridge@samba.org>
Diffstat (limited to 'source4/scripting/python/samba/netcmd/fsmo.py')
| -rw-r--r-- | source4/scripting/python/samba/netcmd/fsmo.py | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/source4/scripting/python/samba/netcmd/fsmo.py b/source4/scripting/python/samba/netcmd/fsmo.py index 3e9067e7a0..aaf7d138e6 100644 --- a/source4/scripting/python/samba/netcmd/fsmo.py +++ b/source4/scripting/python/samba/netcmd/fsmo.py @@ -4,6 +4,7 @@ # # Copyright Nadezhda Ivanova 2009 # Copyright Jelmer Vernooij 2009 +# Copyright Giampaolo Lauria 2011 <lauria2@yahoo.com> # # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -36,12 +37,6 @@ class cmd_fsmo(Command): synopsis = "(show | transfer <options> | seize <options>)" - takes_optiongroups = { - "sambaopts": options.SambaOptions, - "credopts": options.CredentialsOptions, - "versionopts": options.VersionOptions, - } - takes_options = [ Option("--url", help="LDB URL for database or target server", type=str), Option("--force", help="Force seizing of the role without attempting to transfer first.", action="store_true"), |