42free icon indicating copy to clipboard operation
42free copied to clipboard

False warning about files already existing in sgoinfre

Open loyc12 opened this issue 1 year ago • 2 comments

while running the program (on macos) after having specified to it my sgoinfre dir, it warns me that 'Library/Caches' already exists, despite it not being the case. moreover, whne continuing despite this, it falis to fully move 'Caches', leaves a symbolic link with a tmp name, and gets stuck in a seemingly infinite loop whilst trying to restore what was partially moved...


➜  llord 42free
ERROR: There does not seem to be a sgoinfre directory
available on your campus.
If you are sure there is one, please open an issue on GitHub and mention
the following things:
  - The campus you are on.
  - The path to your sgoinfre directory.
https://github.com/itislu/42free/issues

Do you wish to input the path to your sgoinfre directory manually?
[Y/n]
> y
Please enter the path to your personal sgoinfre directory:
> /System/Volumes/Data/sgoinfre/Perso/llord
 ✔ Directory exists.

                📁  42free  📁


WARNING: The permissions of your personal sgoinfre
directory are not set to 'rwx------'.
They are currently set to ''.
It is highly recommended to change the permissions so that other
students cannot access the files you will move to sgoinfre.
Do you wish to change the permissions of '' to 'rwx------'?
[Y/n]
> y
SUCCESS: The permissions of
'/System/Volumes/Data/sgoinfre/Perso/llord' have been changed to
'rwx------'.
Close all programs first to avoid errors during the move.
To see the manual, run '42free --help'.

The following directories will be moved to sgoinfre:
  ▸ /Users/llord/Library/Caches
  ▸ /Users/llord/Library/Application Support/Code/Cache
  ▸ /Users/llord/Library/Application Support/Code/CachedData
  ▸ /Users/llord/Library/Application Support/Code/CachedExtensionVSIXs
  ▸ /Users/llord/Library/Application Support/Code/Crashpad
  ▸ /Users/llord/Library/Application Support/Code/User/workspaceStorage
  ▸ /Users/llord/Library/Application Support/Code/Service Worker
  ▸ /Users/llord/Library/Application Support/Slack/Cache
  ▸ /Users/llord/Library/Application Support/Slack/Service Worker
  ▸ /Users/llord/Library/Application Support/discord/Cache
  ▸ /Users/llord/Library/Application Support/Google/Chrome/Default/Service Worker

Do you agree with all of those? [Y/n]
> y

[1/11]
'/Users/llord/Library/Caches' ➜ sgoinfre

WARNING: 'Library/Caches' already exists in the
sgoinfre directory.
Do you wish to continue and replace any duplicate files?
[y/N]
> y
Getting the current sizes of the home and sgoinfre directories...
Getting the size of 'Caches'...
Moving 'Caches' to '/Library'...
ERROR: Could not fully move 'Caches' to
'/Library'.
Symbolic link left behind with a tmp name.
0.0B of 2.3GB freed.
Do you wish to restore what was partially moved to the sgoinfre directory
back to the home directory? [y/N]
> y
Restoring 'Caches' to '/Users/llord/Library'...

loyc12 avatar May 10 '24 20:05 loyc12

I noticed that even though you specified the path to your sgoinfre as /System/Volumes/Data/sgoinfre/Perso/llord, it prints:

Moving 'Caches' to '/Library'...

So the path to sgoinfre transformed from /System/Volumes/Data/sgoinfre/Perso/llord to /Library...

itislu avatar May 16 '24 06:05 itislu

@loyc12 With the release of v1.8.0, could you try if this still happens?

itislu avatar May 17 '24 18:05 itislu

In case this still happens please feel free to comment and I will reopen the issue.

itislu avatar Sep 27 '24 00:09 itislu