From a95646fc8786590f05c0dae019b9b72ccb5ce912 Mon Sep 17 00:00:00 2001 From: David Douard <david.douard@sdfa3.org> Date: Fri, 29 May 2020 17:14:31 +0200 Subject: [PATCH] Exclude [Skipped]Content.ctime from hash/eq computation this attribute is not an intrinsic property of a content object, so it should not be used when comparing or hashing. --- swh/model/model.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/swh/model/model.py b/swh/model/model.py index 7db255cc..1dc0248c 100644 --- a/swh/model/model.py +++ b/swh/model/model.py @@ -540,7 +540,10 @@ class Content(BaseContent): data = attr.ib(type=Optional[bytes], validator=type_validator(), default=None) ctime = attr.ib( - type=Optional[datetime.datetime], validator=type_validator(), default=None + type=Optional[datetime.datetime], + validator=type_validator(), + default=None, + eq=False, ) @length.validator @@ -600,7 +603,10 @@ class SkippedContent(BaseContent): origin = attr.ib(type=Optional[str], validator=type_validator(), default=None) ctime = attr.ib( - type=Optional[datetime.datetime], validator=type_validator(), default=None + type=Optional[datetime.datetime], + validator=type_validator(), + default=None, + eq=False, ) @reason.validator -- GitLab