vfox icon indicating copy to clipboard operation
vfox copied to clipboard

Supports setting the vfox configuration path via the `VFOX_HOME` environment variable

Open Chance-fyi opened this issue 1 year ago • 6 comments

close: #45

Chance-fyi avatar Mar 06 '24 06:03 Chance-fyi

This also involves the issue of whether there are permissions. In addition, The default permissions granted now are 0755. Since we are allowed to change VFOX_HOME, we should also consider the permissions.

aooohan avatar Mar 06 '24 06:03 aooohan

This also involves the issue of whether there are permissions. In addition, The default permissions granted now are 0755. Since we are allowed to change VFOX_HOME, we should also consider the permissions.

This is a problem, I will add a check for VFOX_HOME permissions.

Chance-fyi avatar Mar 06 '24 06:03 Chance-fyi

Regarding permissions, currently, only errors related to insufficient permissions are being returned.

As for #51, regarding the potential support for multiple users sharing the VFOX_HOME directory, this would require setting permissions to at least 775. We need to discuss this matter further, perhaps it could be addressed in another PR.

Chance-fyi avatar Mar 07 '24 02:03 Chance-fyi

Regarding permissions, currently, only errors related to insufficient permissions are being returned.

As for #51, regarding the potential support for multiple users sharing the VFOX_HOME directory, this would require setting permissions to at least 775. We need to discuss this matter further, perhaps it could be addressed in another PR.

Yes, so simply changing VFOX_HOME does not seem to be of much use at present. The default 755 permissions still limit this feature.

aooohan avatar Mar 07 '24 02:03 aooohan

Can you add command like vfox config store-path=D:/app/vfox or command vfox config home=D:/app/vfox

cifaz avatar Apr 13 '24 14:04 cifaz

Can you add command like vfox config store-path=D:/app/vfox

cifaz avatar Apr 13 '24 14:04 cifaz