[anaconda:rhel7/master] Add option help text for --image and --dirinstall flags (#1056791)

Chris Lumens clumens at redhat.com
Thu Jan 30 20:57:01 UTC 2014


> +class HelpTextParser(object):
> +    """Class to parse help text from file and make it available to option
> +       parser.
> +    """
> +
> +    def __init__(self, path):
> +        """ Initializer
> +            :param path: The absolute path to the help text file
> +        """
> +        # path --- the path to the file containing the help text
> +        if not os.path.isabs(path):
> +            raise ValueError("path %s is not an absolute path" % path)
> +        self._path = path
> +
> +        # _help_text --- the help text, as a dictionary of option,
> +        self._help_text = None
> +

I don't know that the comments for these attributes are necessary, but
if so the _help_text one should be complete.

Otherwise, looks fine to me if it looks fine to pylint.

- Chris


More information about the anaconda-patches mailing list