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

[🚀 | Feature request]: CLI PARSEC_DATA_DIR & PARSEC_CACHE_DIR config var #9340

Open
Landeers opened this issue Jan 14, 2025 · 0 comments · May be fixed by #9263
Open

[🚀 | Feature request]: CLI PARSEC_DATA_DIR & PARSEC_CACHE_DIR config var #9340

Landeers opened this issue Jan 14, 2025 · 0 comments · May be fixed by #9263
Assignees
@FirelightFlagboy
Labels
A-Client Area: Parsec desktop application C-Feature-Request Category: A feature request for Parsec I-CLI Impact: Command-line interface I-Rust Impact: Rust-related stuff

Comments

@Landeers
Copy link
Contributor

What you would like to accomplish

https://scille.slack.com/archives/C20C5ECJH/p1735035228752629

How the feature should work

Today, for the CLI, the cache and data folders are located in the $HOME and cannot be modified. This is a problem when the $HOME directory doesn't (or shouldn't) exist.

The purpose of this issue is to add 2 options to the CLI --data-dir and --cache-dir as --config-dir already exists. And env var should also works.

May be fixed by #9263

jerome@fedora:~$ PARSEC_DATA_DIR=/home/jerome/test_data_dir/ PARSEC_CONFIG_DIR=/home/jerome/test_config_dir/ ./parsec invite claim --addr "parsec3://trial.parsec.cloud/trial-jerome_land-1734528219439?a=claim_user&p=xBB7kAnFABlLK_NGeLM0xGae"

jerome@fedora:~$ PARSEC_DATA_DIR=/home/jerome/test_data_dir/ PARSEC_CONFIG_DIR=/home/jerome/test_config_dir/ ./parsec workspace list --device 964
Enter password for the device:
Found 1 workspace(s)
3882622ac9df41e5abfca833c54289cf - test: manager

jerome@fedora:~$ PARSEC_DATA_DIR=/home/jerome/test_data_dir/ PARSEC_CONFIG_DIR=/home/jerome/test_config_dir/ ./parsec workspace sync --workspace 3882622ac9df41e5abfca833c54289cf --device 964
Enter password for the device:
Workspace test has been synced

jerome@fedora:~$ cd test_data_dir/
jerome@fedora:~/test_data_dir$ ls
964c4efd23e04acdbe72e597f2f23606
jerome@fedora:~/test_data_dir$ cd 964c4efd23e04acdbe72e597f2f23606/
jerome@fedora:~/test_data_dir/964c4efd23e04acdbe72e597f2f23606$ ls
user_data-v1.sqlite

jerome@fedora:~/.local/share/parsec3/964c4efd23e04acdbe72e597f2f23606$ ls -la
total 112
drwxr-xr-x. 1 jerome jerome   146 26 déc.  09:19 .
drwxr-xr-x. 1 jerome jerome   128 26 déc.  09:19 ..
drwxr-xr-x. 1 jerome jerome    98 26 déc.  09:19 3882622ac9df41e5abfca833c54289cf
-rw-r--r--. 1 jerome jerome 57344 26 déc.  09:19 certificates-v1.sqlite
-rw-r--r--. 1 jerome jerome 49152 26 déc.  09:19 user_data-v1.sqlite

Describe alternatives you have considered

No response
@Landeers Landeers added B-Triage Blocker: The issue is awaiting triage C-Feature-Request Category: A feature request for Parsec labels Jan 14, 2025
@FirelightFlagboy FirelightFlagboy changed the title [🚀 | Feature request]: CLI PARSEC_DATA_DIR & PARSEC_CACHE_DIR config var [🚀 | Feature request]: CLI PARSEC_DATA_DIR & PARSEC_CACHE_DIR config var Jan 15, 2025
@FirelightFlagboy FirelightFlagboy added I-Rust Impact: Rust-related stuff A-Client Area: Parsec desktop application I-CLI Impact: Command-line interface and removed B-Triage Blocker: The issue is awaiting triage labels Jan 15, 2025
@FirelightFlagboy FirelightFlagboy linked a pull request Jan 15, 2025 that will close this issue
Add data_dir option to command using a client #9263
Draft
@FirelightFlagboy FirelightFlagboy self-assigned this Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FirelightFlagboy FirelightFlagboy

Labels
A-Client Area: Parsec desktop application C-Feature-Request Category: A feature request for Parsec I-CLI Impact: Command-line interface I-Rust Impact: Rust-related stuff
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add data_dir option to command using a client Scille/parsec-cloud
2 participants
@Landeers @FirelightFlagboy