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

[BUG] Repository path issues on Windows #494

Closed
OmarFourati opened this issue Jan 9, 2024 · 5 comments
Closed

[BUG] Repository path issues on Windows #494

OmarFourati opened this issue Jan 9, 2024 · 5 comments
Assignees
Labels
bug Something isn't working softwarecampus Issues related to the Softwarecampus grant

Comments

@OmarFourati
Copy link
Collaborator

Steps to reproduce

  1. Work with repository on windows

Description

  • Expected: works
  • Actual: breaks because colon not valid filename char. on windows.
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/test-A1:C16.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/test-A1:C16.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/test-B1:C2.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/test-B1:C2.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/valid-A1:A4.jv'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/valid-A1:C*.jv'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-range-selector-executor/valid-A1:E4.jv'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-writer-executor/test-A1:C16.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/cell-writer-executor/test-A1:C16.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/column-deleter-executor/test-A1:B2.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/column-deleter-executor/test-A1:B2.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/column-deleter-executor/test-A1:C16.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/column-deleter-executor/test-A1:C16.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/row-deleter-executor/test-A1:B2.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/row-deleter-executor/test-A1:B2.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/row-deleter-executor/test-A1:C16.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/row-deleter-executor/test-A1:C16.xlsx.license'
error: invalid path 'libs/extensions/tabular/exec/test/assets/sheet-picker-executor/test-A1:C16.xlsx'
error: invalid path 'libs/extensions/tabular/exec/test/assets/sheet-picker-executor/test-A1:C16.xlsx.license'
@OmarFourati OmarFourati added bug Something isn't working softwarecampus Issues related to the Softwarecampus grant labels Jan 9, 2024
@OmarFourati OmarFourati self-assigned this Jan 9, 2024
@OmarFourati OmarFourati changed the title [BUG] <name> [BUG] Repository path issues on Windows Jan 9, 2024
@OmarFourati
Copy link
Collaborator Author

OmarFourati commented Jan 9, 2024

Just noticed that the files aren't even to find in my repo even if I am in the main branch.
So cloning the repo on windows just leaves out the whole "test" folder of libs/extensions/tabular/exec/

I installed now a wsl instance in my windows so I just cloned the repo again and i this time actually got all the files.
Updating this is definitely important for Windows users

@joluj
Copy link
Contributor

joluj commented Jan 11, 2024

Windows has a maximum length for a path, by default 260 characters.

Allowing longer paths is described here: https://learn.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation.

TLDR: Run New-ItemProperty -Path "HKLM:\SYSTEM\CurrentControlSet\Control\FileSystem" -Name "LongPathsEnabled" -Value 1 -PropertyType DWORD -Force in a powershell with administrator rights.

@OmarFourati
Copy link
Collaborator Author

OmarFourati commented Jan 11, 2024

I am not sure that this is the problem. The files doesn't even get clone with the repo.
This path for example doesn't have more than 260 characters : libs/extensions/tabular/exec/test/assets/cell-writer-executor/test-A1:C16.xlsx. right ?
I think ( and @rhazn read it somewhere ) that it is about the naming.
image

@joluj
Copy link
Contributor

joluj commented Jan 11, 2024

That's true, thanks for the PR. I have also created #499 to assert that no such error comes up again. It fails now (which is good since there are invalid file names).
After #498 is merged #499 hopefully runs successfully.

@joluj
Copy link
Contributor

joluj commented Jan 18, 2024

closed by #498 and #499

@joluj joluj closed this as completed Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working softwarecampus Issues related to the Softwarecampus grant
Projects
None yet
Development

No branches or pull requests

2 participants