Skip to content

Commit

Permalink
do not memoized PackageRecord.combined_depends
Browse files Browse the repository at this point in the history
Change combined_depends from a memoizedproperty to a property.
Memoization was causing problems with conda-build when building against
a matrix using a conda_build_config.yaml file,
c.f. conda/conda-build#3735
  • Loading branch information
jjhelmus committed Sep 26, 2019
1 parent 16bafde commit 9798874
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions conda/models/records.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
from .._vendor.auxlib.entity import (BooleanField, ComposableField, DictSafeMixin, Entity,
EnumField, IntegerField, ListField, NumberField,
StringField)
from .._vendor.auxlib.decorators import memoizedproperty
from .._vendor.boltons.timeutils import dt_to_timestamp, isoparse
from ..base.context import context
from ..common.compat import isiterable, itervalues, string_types, text_type
Expand Down Expand Up @@ -319,7 +318,7 @@ def is_unmanageable(self):

timestamp = TimestampField()

@memoizedproperty
@property
def combined_depends(self):
from .match_spec import MatchSpec
result = {ms.name: ms for ms in MatchSpec.merge(self.depends)}
Expand Down

0 comments on commit 9798874

Please sign in to comment.