RF: remove explicit hash=True from @attr.s

See #430 (comment) for more reasoning -- objects should still be hashable
ReproNim · Jun 17, 2019 · fae7a7e · fae7a7e
1 parent b49f7d6
commit fae7a7e
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/reproman/distributions/debian.py b/reproman/distributions/debian.py
@@ -51,7 +51,7 @@
 
 # TODO: flyweight/singleton ?
 # To make them hashable we need to freeze them... not sure if we are ready:
-#@attr.s(cmp=True, hash=True, frozen=True)
+#@attr.s(cmp=True, frozen=True)
 @attr.s
 class APTSource(SpecObject):
     """APT origin information
@@ -69,7 +69,7 @@ class APTSource(SpecObject):
 _register_with_representer(APTSource)
 
 
-@attr.s(slots=True, frozen=True, hash=True)
+@attr.s(slots=True, frozen=True)
 class DEBPackage(Package):
     """Debian package information"""
     name = attrib(default=attr.NOTHING)

diff --git a/reproman/distributions/docker.py b/reproman/distributions/docker.py
@@ -23,7 +23,7 @@
 from ..utils import attrib
 
 
-@attr.s(slots=True, frozen=True, hash=True)
+@attr.s(slots=True, frozen=True)
 class DockerImage(Package):
     """Docker image information"""
     id = attrib(default=attr.NOTHING)

diff --git a/reproman/distributions/redhat.py b/reproman/distributions/redhat.py
@@ -45,7 +45,7 @@ class RPMSource(SpecObject):
 _register_with_representer(RPMSource)
 
 
-@attr.s(slots=True, frozen=True, hash=True)
+@attr.s(slots=True, frozen=True)
 class RPMPackage(Package):
     """Redhat package information"""
     name = attrib(default=attr.NOTHING)

diff --git a/reproman/distributions/singularity.py b/reproman/distributions/singularity.py
@@ -26,7 +26,7 @@
 from ..utils import attrib, md5sum, chpwd
 
 
-@attr.s(slots=True, frozen=True, hash=True)
+@attr.s(slots=True, frozen=True)
 class SingularityImage(Package):
     """Singularity image information"""
     md5 = attrib(default=attr.NOTHING)