[Kimchi-devel] [PATCH V2] [Wok] Issue #12: wokd start fails if plugin does not have ui/config/tab-ext.xml

[Paulo Vital]

Reviewed-by: Paulo Vital <pvital at linux.vnet.ibm.com>


On 12/21/2015 10:09 AM, archus at linux.vnet.ibm.com wrote:
> From: Archana Singh <archus at linux.vnet.ibm.com>
> 
> Currently, it gets the list of tab-ext.xml
> for all the installed plugins and then
> each file is parsed in for loop.
> Moved code which parse the tab-ext.xml file
> inside try except block to handle IOError.
> On IOError(File not found) continue the for
> loop to parse other plugins tab-ext.xml.
> 
> Signed-off-by: Archana Singh <archus at linux.vnet.ibm.com>
> ---
>  src/wok/utils.py | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/src/wok/utils.py b/src/wok/utils.py
> index 997adf3..0138bd2 100644
> --- a/src/wok/utils.py
> +++ b/src/wok/utils.py
> @@ -109,7 +109,11 @@ def get_all_tabs():
> 
>      tabs = []
>      for f in files:
> -        root = ET.parse(f)
> +        try:
> +            root = ET.parse(f)
> +        except (IOError), e:
> +            wok_log.debug("Unable to load %s", f)
> +            continue
>          tabs.extend([t.text.lower() for t in root.getiterator('title')])
> 
>      return tabs
>