Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fs: transfer: don't raise if link failed with EINVAL #217

Merged
merged 1 commit into from
Jul 30, 2023
Merged

Conversation

efiop
Copy link
Contributor

@efiop efiop commented Jul 30, 2023

Running into this on windows when working on a mapped network share https://support.microsoft.com/en-us/windows/map-a-network-drive-in-windows-29ce55d1-34e3-a7e2-4801-131475f9557d

The error I'm getting is:

OSError: [WinError 1] Incorrect function

which is mapped https://github.com/python/cpython/blob/5113ed7a2b92e8beabebe5fe2f6e856c52fbe1a0/PC/errmap.h#L132 to EINVAL.

Related to iterative/dvc#9780

Running into this on windows when working on a mapped network share https://support.microsoft.com/en-us/windows/map-a-network-drive-in-windows-29ce55d1-34e3-a7e2-4801-131475f9557d

The error I'm getting is:

```
OSError: [WinError 1] Incorrect function
```

which is mapped https://github.com/python/cpython/blob/5113ed7a2b92e8beabebe5fe2f6e856c52fbe1a0/PC/errmap.h#L132 to `EINVAL`.
@efiop efiop added the enhancement New feature or request label Jul 30, 2023
@efiop efiop self-assigned this Jul 30, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 30, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (f48f77c) 64.79% compared to head (d281d67) 64.79%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #217   +/-   ##
=======================================
  Coverage   64.79%   64.79%           
=======================================
  Files          25       25           
  Lines        1943     1943           
  Branches      308      308           
=======================================
  Hits         1259     1259           
  Misses        627      627           
  Partials       57       57           
Files Changed Coverage Δ
src/dvc_objects/fs/generic.py 26.77% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@efiop efiop merged commit b928916 into main Jul 30, 2023
13 checks passed
@daavoo daavoo deleted the efiop-patch-3 branch July 30, 2023 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants