FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Incorrect dir test in TransferManager · Issue #208 · csvpath/csvpath · GitHub

Incorrect dir test in TransferManager #208

Description

Description:

In TransferManager._path_to_transfer_to() look for this block ~318:

        p = result.csvpath.config.transfer_root
        rp = Nos(p).join(f)
        sep = Nos(rp).sep
        rd = rp[0 : rp.rfind(sep)]
        #
        # should this be dir_exists()?
        #
        if not Nos(rd).exists():
            Nos(rd).makedir()
        return rp

The if not Nos(rd).exists(): should almost certainly be if not Nos(rd).dir_exists():. This is because blob stores handle dirs differently from traditional physical filesystems.

Activity

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

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL