Fix version comparison

[rochacbruno]

Compara versions using ansible.module_utils.compat.version.LooseVersion

closes #334

[rochacbruno]

packaging is added by ansible https://github.com/ansible/ansible/blob/devel/requirements.txt#L9

Not sure why the CI is failing, do we need to include it elsewhere?

[sean-m-sullivan]

Clean, thanks!

[rochacbruno]

@sean-m-sullivan CI is failing because the packaging dependency cannot be found

[sean-m-sullivan]

it looks like this dependency is not part of the ansible install. need it in requirements.txt in the base directory.

[rochacbruno]

ALternativelly we can create our own implementation of the class Version and not depend on external library.

class Version:
    def __init__(self, version_str):
        components = list(map(int, version_str.split('.')))
        self.major = components[0]
        self.minor = components[1] if len(components) > 1 else 0
        self.patch = components[2] if len(components) > 2 else 0

    def __str__(self):
        return f"{self.major}.{self.minor}.{self.patch}"

    def __eq__(self, other):
        if not isinstance(other, Version):
            return NotImplemented
        return (self.major, self.minor, self.patch) == (other.major, other.minor, other.patch)

    def __lt__(self, other):
        if not isinstance(other, Version):
            return NotImplemented
        return (self.major, self.minor, self.patch) < (other.major, other.minor, other.patch)

    def __le__(self, other):
        if not isinstance(other, Version):
            return NotImplemented
        return (self < other) or (self == other)

    def __gt__(self, other):
        if not isinstance(other, Version):
            return NotImplemented
        return not (self <= other)

    def __ge__(self, other):
        if not isinstance(other, Version):
            return NotImplemented
        return not (self < other)

    def __ne__(self, other):
        if not isinstance(other, Version):
            return NotImplemented
        return not (self == other)

# Example usage:
version1 = Version("4.2")
version2 = Version("5.1.3")

print(version1 < version2)  # Output: True
print(version1 >= version2)  # Output: False

[Tompage1994]

sanity check is stating that this needs to be with the imports further down below the documentation segment

[Tompage1994]

Plus the same for all the other modules

[rochacbruno]

Trying a simpler implementation on #337

[rochacbruno]

Closing in favor of #337