pyobjus icon indicating copy to clipboard operation
pyobjus copied to clipboard

Can't clone pyobjus on Windows, MS-DOS reserved word as a folder

Open KeyWeeUsr opened this issue 7 years ago • 2 comments

git clone https://github.com/kivy/pyobjus

Cloning into 'pyobjus'...
remote: Enumerating objects: 219, done.
remote: Counting objects: 100% (219/219), done.
remote: Compressing objects: 100% (146/146), done.
emote: Total 1944 (delta 146), reused 131 (Receiving objects:  98% (190delta 72), pack-reused 1725
Receiving objects: 100% (1944/1944), 763.17 KiB | 0 bytes/s, done.
Resolving deltas: 100% (1221/1221), done.
Checking connectivity... done.
fatal: cannot create directory at 'objc_classes/aux': Invalid argument
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'

This is happening due to aux being a reserved MS-DOS word and the only workaround is very annoying. In case we want people to contribute to PyOBJus from Windows (e.g. docs if not the code) we'll need to rename it. That folder is most likely used on one of the iOS things hopefully only under Plyer, but I guess that something would also be available under Kivy-iOS, so it'll break things and I don't have any iOS device to test it with.

KeyWeeUsr avatar Nov 09 '18 17:11 KeyWeeUsr

Same problem here, adding Pyobjus for example to Poetry conf breaks operations on Windows, because Poetry will naturally checkout dependencies (even those for other platforms) to perform dependency resolution.

This is a nasty limitation of Windows, but renaming this "aux/" folder in the source tree would solve it easily.

pakal avatar Jun 05 '22 11:06 pakal

Catching up on this. Some ideas:

Can this folder be renamed without breaking iOS? Can the folder be deleted in advance during the installation process, before windows complains about it? Could the folder be named aux_rename and then renamed to aux, if installing on a non-windows environment?

brentpicasso avatar Nov 08 '22 16:11 brentpicasso