Skip to content

checkout: check if cache.type has changed #1371

Closed
@efiop

Description

@efiop

Same as in #1370 .

Activity

added this to the Queue milestone on Nov 27, 2018
efiop

efiop commented on Aug 21, 2019

@efiop
ContributorAuthor

Related to #2358 where we already do that.

For checkout itself, we need to not rely in self.changed()(at least not in its current form, as it doesn't detect link/protected change) in RemoteBASE.checkout() but actually granularly check links. That could be done by modifying changed() so when it sees that self.state doesn't have a correct entry for the path_info, it goes ahead and not only checks the checksums, but also whether or not file is protected and is of correct link type.

efiop

efiop commented on Nov 11, 2019

@efiop
ContributorAuthor

Fixed by checkout patch from @pared

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @efiop

        Issue actions

          checkout: check if cache.type has changed · Issue #1371 · iterative/dvc