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.

Sign up for GitHub

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

Jump to bottom

not possible to update/clone the repo on a Windows operating system. #3805

Open
2 of 3 tasks
SirAionTech opened this issue Jan 2, 2024 · 5 comments · Fixed by #3112 or #3806 · May be fixed by #4575
Open
2 of 3 tasks

not possible to update/clone the repo on a Windows operating system. #3805

SirAionTech opened this issue Jan 2, 2024 · 5 comments · Fixed by #3112 or #3806 · May be fixed by #4575
Labels
16.0 bug hotfix needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue.

Comments

@SirAionTech
Copy link
Contributor

SirAionTech commented Jan 2, 2024 •
edited by francesco-ooops
Loading

Hi Guys, thanks for the job.
But since this commit, it's not possible to update/clone the repo on a Windows operating system.

The filename POSTA CERTIFICATA: Invio File 7339338 (base64).txt is incompatible for a NTFS filesystem.

Is it necessary to have ':' in the filename ?
Possible to rename it w/o ':' ?

Thanks.

Originally posted by @mde-spring in #3500 (comment)
@SirAionTech
Copy link
Contributor Author

SirAionTech commented Jan 2, 2024 •
edited
Loading

The current work-around is to avoid such filenames, as done in #3112.

The optimal solution would be to have the CI run in Windows too (asked in OCA/oca-ci#56).

@francesco-ooops
Copy link
Contributor

@mde-spring could you open a PR to fix the filename?

@SirAionTech SirAionTech mentioned this issue Jan 2, 2024
Open
@mde-scopea mde-scopea mentioned this issue Jan 2, 2024
Merged
@mde-scopea
Copy link
Contributor

@francesco-ooops done #3806

This was linked to pull requests Jan 8, 2024
[FIX] l10n_it_fatturapa_pec: Nome file non accettato (Windows) #3112
Merged
[FIX] l10n_it_fatturapa_pec: filename without colon #3806
Merged
@francesco-ooops
Copy link
Contributor

@eLBati @SirAionTech sussiste per la 16?

@eLBati
Copy link
Member

eLBati commented Jan 8, 2024

@eLBati @SirAionTech sussiste per la 16?

sembra di no

@SirAionTech SirAionTech reopened this Dec 11, 2024
@SirAionTech SirAionTech mentioned this issue Dec 11, 2024
Closed
@SirAionTech SirAionTech added needs porting This issue has already been resolved for some version hotfix no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed 12.0 14.0 labels Dec 11, 2024
This was referenced Jan 21, 2025
Closed
Closed
@luca-vercelli luca-vercelli linked a pull request Jan 27, 2025 that will close this issue
Open
@francesco-ooops francesco-ooops linked a pull request Jan 27, 2025 that will close this issue
fatturapa_pec: filename without colon in v16 #3805 #4574 #4575
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
16.0 bug hotfix needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Milestone
No milestone
4 participants
@eLBati @francesco-ooops @mde-scopea @SirAionTech